1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: correct order for assert.strictEqual for inspector-helper test

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/23468
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
This commit is contained in:
Maggie Nolan 2018-10-12 09:54:24 -07:00 committed by Rich Trott
parent c23d2505b3
commit ffa2710cea
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
test/common/inspector-helper.js
View File

@ -63,7 +63,7 @@ function parseWSFrame(buffer) {
if (buffer[0] === 0x88 && buffer[1] === 0x00) { if (buffer[0] === 0x88 && buffer[1] === 0x00) {
return { length: 2, message, closed: true }; return { length: 2, message, closed: true };
} }
assert.strictEqual(0x81, buffer[0]); assert.strictEqual(buffer[0], 0x81);
let dataLen = 0x7F & buffer[1]; let dataLen = 0x7F & buffer[1];
let bodyOffset = 2; let bodyOffset = 2;
if (buffer.length < bodyOffset + dataLen) if (buffer.length < bodyOffset + dataLen)
@ -251,7 +251,7 @@ class InspectorSession {
assert.strictEqual(scriptPath.toString(), assert.strictEqual(scriptPath.toString(),
expectedScriptPath.toString(), expectedScriptPath.toString(),
`${scriptPath} !== ${expectedScriptPath}`); `${scriptPath} !== ${expectedScriptPath}`);
assert.strictEqual(line, location.lineNumber); assert.strictEqual(location.lineNumber, line);
return true; return true;
} }
} }