Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

fix: fix broken stats test #236

Merged
merged 1 commit into from
Mar 13, 2018
Merged

fix: fix broken stats test #236

merged 1 commit into from
Mar 13, 2018

Conversation

vmx
Copy link
Contributor

@vmx vmx commented Mar 13, 2018

One test broke due to c4934ca 1.

If promises are used, you needn't pass done as parameter of
the callback in.

@vmx vmx requested a review from daviddias March 13, 2018 14:55
@ghost ghost assigned vmx Mar 13, 2018
@ghost ghost added the in progress label Mar 13, 2018
Copy link
Contributor

@hacdias hacdias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for noticing this 😄

@vmx
Copy link
Contributor Author

vmx commented Mar 13, 2018

@MichaelMure See this PR for future reference. I also wasn't aware that passing in the done argument could be an issue.

Tests broke due to c4934ca [1].

If promises are used, you needn't pass `done` as parameter of
the callback in.

[1]: c4934ca#diff-0a6449ecfa8b9e3d807f53dde24eca71R66
@daviddias daviddias merged commit fcb8341 into master Mar 13, 2018
@ghost ghost removed the in progress label Mar 13, 2018
@daviddias daviddias deleted the fix-stats-test branch March 13, 2018 20:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants