tree e33ab6438a7b2c70835640c11b814c6ae5599f91
parent 4646852d05f35bdc10b19e144a82b6cecad1b987
author Nate Bosch <nbosch@google.com> 1621995871 -0700
committer GitHub <noreply@github.com> 1621995871 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJgrbFfCRBK7hj4Ov3rIwAARUkIAHw0GX4vOeSfnsnBtGRmu2XU
 Z0YXoe9t50aYbcIi4MC31pYT6FVdHtCDTYNRtiU0Po4b1/+SZPql73+208Q1AZJa
 yPm6+jGf2eH9YcfzgQYDcv0RsUBbFQ8mjJo37wTAk1APZy8A36ANhl8P0aqyz+qv
 VMeK7qaplSJ/an1j7oCPFDdl0vgZ94LKmubCv5pw0zF6m86+nLyfwODMqP838oQd
 IrCExkJWUQupVevigbdrNXFJ3y/d87rSA/oMDsXCHm16/YSzCPShv8vcD7A0DofT
 OLjzWO5nMSlE/4LJIMSiyZ3J9osIlxSUR9hwH11CdtBBP0+UB7Z/gDiVvzvHILk=
 =vsQ2
 -----END PGP SIGNATURE-----
 

Fix test for wrapping in error zone (#109)

When this test was written it had a bug - it should not pass
`when: false` because that argument makes the method ignore the
`errorZone` argument entirely. Omitting the argument allows the default
`errorZone: true` behavior. The bug in the test was masked by the fact
that `package:test` had previously run tests wrapped in a chain
capturing zone by default.

Remove the `when` argument to correct the test. Add an `onError`
argument to the `.then` call to make the test stronger and assert that
the resulting future does not complete at all - either with a value or
an error. Also add the same check that the future does not complete on
another test for error blocking.