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 test fn's being deleted too early for browser reporter #2066

Merged
merged 1 commit into from
Jan 26, 2016
Merged

Fix test fn's being deleted too early for browser reporter #2066

merged 1 commit into from
Jan 26, 2016

Conversation

danielstjules
Copy link
Contributor

Fixes the issue reported here: #2040 (comment)

screen shot 2016-01-25 at 11 07 05 pm

Review on Reviewable

@danielstjules danielstjules changed the title Fix test fn's being deleted to early for browser reporter Fix test fn's being deleted too early for browser reporter Jan 26, 2016
danielstjules added a commit that referenced this pull request Jan 26, 2016
Fix test fn's being deleted too early for browser reporter
@danielstjules danielstjules merged commit 9f6cb87 into mochajs:master Jan 26, 2016
@danielstjules danielstjules deleted the dstjules/fix-browser branch January 26, 2016 07:33
@chrmod
Copy link

chrmod commented Jan 26, 2016

confirming that fix works.

Though there is something strange about it as I see one failed test but two error reports (on html reporter) - previously I've seen two failed tests and one report.

Have noticed that on master 'fail' hook is called multiple times.

@danielstjules
Copy link
Contributor Author

@chrmod Yup, that's fixed via #2068 :)

@chrmod
Copy link

chrmod commented Jan 27, 2016

great job ! :-)

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.

2 participants