Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix tests #90

Merged
merged 1 commit into from
Feb 10, 2019
Merged

Fix tests #90

merged 1 commit into from
Feb 10, 2019

Conversation

bcomnes
Copy link
Contributor

@bcomnes bcomnes commented Feb 10, 2019

No description provided.

@bcomnes bcomnes merged commit b6a2d8c into master Feb 10, 2019
@bcomnes bcomnes deleted the fix-tests branch February 10, 2019 23:02
@ungoldman
Copy link
Member

FYI I think tests breaking were because of this: tape-testing/tape#458

You need to do an explicit t.plan() or t.end(), even in the context of just subtests like we had here.

Discovered this bug when CI tests started failing on another project and I couldn't figure out why. It's a bug fix but introduced in a minor, so lots of breaking everywhere.

@bcomnes
Copy link
Contributor Author

bcomnes commented Feb 11, 2019

🤔 alrighty then. Erm, thats weird, seems like more of pretty big behavior change than a 'bug' fix. Whatever.

(Thank you for the info 🙏)

@goto-bus-stop
Copy link
Member

oh, that's probably also why they were failing in nanomorph. i pinned tape to 4.9.x there because i didn't want to look into it, lol

good to know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants