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

test: re-enable assert_throws_stack tests for node-chakracore #504

Conversation

mike-kaufman
Copy link
Contributor

  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Copy link
Contributor

@kfarnung kfarnung left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@jackhorton jackhorton left a comment

Choose a reason for hiding this comment

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

Why did this originally fail?

@mike-kaufman
Copy link
Contributor Author

Why did this originally fail?

The test changed upstream to output some additional output, so baseline was different. I just disabled this during the merge just to get merge unblocked.

@mike-kaufman mike-kaufman merged commit 857281b into nodejs:master Mar 28, 2018
@mike-kaufman mike-kaufman deleted the mkaufman-enable-assert-throws-stack-test branch March 28, 2018 20:30
@kfarnung kfarnung mentioned this pull request Apr 24, 2018
4 tasks
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.

4 participants