★ wanayoo — archive 1999 https://github.com/nodejs/node/commit/cd1aa2b0b5Nouvelle recherche | Portail wanayoo
Skip to content
Permalink
Browse files

test: fix http2-binding strictEqual order

Switched the order of arguments for strictEqual checks inside of
test/paralell/test-http2-binding.js

PR-URL: #24616
Refs: nodejsjp#1
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
  • Loading branch information...
dominikeinkemmer authored and rvagg committed Nov 24, 2018
1 parent 82ef618 commit cd1aa2b0b5ebb693ba079c919bfb06b0eee5098d
Showing with 4 additions and 4 deletions.
  1. +4 −4 test/parallel/test-http2-binding.js
@@ -233,16 +233,16 @@ const defaultSettings = {

for (const name of Object.keys(constants)) {
if (name.startsWith('HTTP_STATUS_')) {
assert.strictEqual(expectedStatusCodes[name], constants[name],
assert.strictEqual(constants[name], expectedStatusCodes[name],
`Expected status code match for ${name}`);
} else if (name.startsWith('HTTP2_HEADER_')) {
assert.strictEqual(expectedHeaderNames[name], constants[name],
assert.strictEqual(constants[name], expectedHeaderNames[name],
`Expected header name match for ${name}`);
} else if (name.startsWith('NGHTTP2_')) {
assert.strictEqual(expectedNGConstants[name], constants[name],
assert.strictEqual(constants[name], expectedNGConstants[name],
`Expected ng constant match for ${name}`);
} else if (name.startsWith('DEFAULT_SETTINGS_')) {
assert.strictEqual(defaultSettings[name], constants[name],
assert.strictEqual(constants[name], defaultSettings[name],
`Expected default setting match for ${name}`);
}
}

0 comments on commit cd1aa2b

Please sign in to comment.
You can’t perform that action at this time.