Remove assert.expect
from tests
#398
Labels
good first issue
Good for newcomers
help wanted
Extra attention is needed
internal
Internal changes that don't affect public API
Nearly all of our tests call assertions synchronously - that is, they aren't wrapped in callbacks or other scenarios where it's possible that one or more of the assertions won't get called. As a result, using
assert.expect
becomes unnecessary, since we'll never have a scenario where only some of the assertions get run. We should removeassert.expect
from all tests of this nature to make our tests less brittle to future changes.The text was updated successfully, but these errors were encountered: