Skip to content

v6.x: backport test: fix test-buffer-slow#10459

Merged
targos merged 1 commit into
nodejs:v6.x-stagingfrom
targos:backport-9809-v6.x
Dec 26, 2016
Merged

v6.x: backport test: fix test-buffer-slow#10459
targos merged 1 commit into
nodejs:v6.x-stagingfrom
targos:backport-9809-v6.x

Conversation

@targos

@targos targos commented Dec 26, 2016

Copy link
Copy Markdown
Member

Backport of #9809

@targos targos added buffer Issues and PRs related to the buffer subsystem. test Issues and PRs related to the tests. v6.x labels Dec 26, 2016
@nodejs-github-bot nodejs-github-bot added test Issues and PRs related to the tests. v6.x labels Dec 26, 2016

@addaleax addaleax left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM if CI is green

@targos

targos commented Dec 26, 2016

Copy link
Copy Markdown
Member Author

Fix incorrect use of string instead of RegExp in `throws` assertions.

PR-URL: nodejs#9809
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Prince John Wesley <princejohnwesley@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
@targos targos force-pushed the backport-9809-v6.x branch from 13e8cfe to 70d3b80 Compare December 26, 2016 13:10
@targos

targos commented Dec 26, 2016

Copy link
Copy Markdown
Member Author

CI failed because I cherry-picked #10146 instead of #10149 before making the PR.
New CI: https://ci.nodejs.org/job/node-test-pull-request/5589/

@targos targos merged commit 70d3b80 into nodejs:v6.x-staging Dec 26, 2016
@targos targos deleted the backport-9809-v6.x branch December 26, 2016 14:38
@targos

targos commented Dec 26, 2016

Copy link
Copy Markdown
Member Author

landed in 70d3b80

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buffer Issues and PRs related to the buffer subsystem. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants