Skip to content

[v9.x backport] Various http2 changes #19579

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.

Sign up for GitHub

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

Closed
targos wants to merge 6 commits into from
Closed

[v9.x backport] Various http2 changes #19579

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
b4a99a7
test: http2 errors on req.close()
trivikr Feb 18, 2018
b59f421
test: add more asserts to `test-internal-errors`
refack Jun 19, 2017
fd94671
test: fix common.expectsError
refack Jun 14, 2017
8e5fd45
test: fix wrong error classes passed in as type
BridgeAR Dec 11, 2017
c0a9336
test: http2 stream.respond() error checks
trivikr Feb 19, 2018
a6785dc
http2: refer to stream errors by name
addaleax Feb 23, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test: add more asserts to test-internal-errors 
PR-URL: #13686
Fixes: #13682
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
  • Loading branch information
@refack @targos
refack authored and targos committed Mar 24, 2018
commit b59f42184f39e96d532edf138eaf24f014523f30
3 changes: 3 additions & 0 deletions test/parallel/test-internal-errors.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -175,6 +175,7 @@ common.expectsError(() => {
message: /^Error for testing 2/ });
}, {
code: 'ERR_ASSERTION',
type: assert.AssertionError,
message: /.+ does not match \S/
});

Expand Down Expand Up @@ -225,6 +226,7 @@ common.expectsError(
() => errors.message('ERR_INVALID_URL_SCHEME', [[]]),
{
code: 'ERR_ASSERTION',
type: assert.AssertionError,
message: /^At least one expected value needs to be specified$/
});

Expand All @@ -239,6 +241,7 @@ common.expectsError(
() => errors.message('ERR_MISSING_ARGS'),
{
code: 'ERR_ASSERTION',
type: assert.AssertionError,
message: /^At least one arg needs to be specified$/
});

Expand Down