Skip to content

Commit

Permalink
test: pummel test fixes
Browse files Browse the repository at this point in the history
This fixes some variable redeclarations in pummel tests and, in the
process, gets `test-stream-pipe-multi.js` to run again. It had been
failing to run.

PR-URL: #4998
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
Trott authored and Myles Borins committed Feb 18, 2016
1 parent 72d38a4 commit 663e852
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 7 deletions.
4 changes: 2 additions & 2 deletions test/pummel/test-crypto-dh.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ var hashes = {
modp18 : 'a870b491bbbec9b131ae9878d07449d32e54f160'
};

for (var name in hashes) {
for (const name in hashes) {
var group = crypto.getDiffieHellman(name);
var private_key = group.getPrime('hex');
var hash1 = hashes[name];
Expand All @@ -40,7 +40,7 @@ for (var name in hashes) {
assert.equal(group.getGenerator('hex'), '02');
}

for (var name in hashes) {
for (const name in hashes) {
// modp1 is 768 bits, FIPS requires >= 1024
if (name == 'modp1' && common.hasFipsCrypto)
continue;
Expand Down
5 changes: 3 additions & 2 deletions test/pummel/test-exec.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,14 @@ var common = require('../common');
var assert = require('assert');
var exec = require('child_process').exec;

var SLEEP3_COMMAND;
if (!common.isWindows) {
// Unix.
var SLEEP3_COMMAND = 'sleep 3';
SLEEP3_COMMAND = 'sleep 3';
} else {
// Windows: `choice` is a command built into cmd.exe. Use another cmd process
// to create a process tree, so we can catch bugs related to it.
var SLEEP3_COMMAND = 'cmd /c choice /t 3 /c X /d X';
SLEEP3_COMMAND = 'cmd /c choice /t 3 /c X /d X';
}


Expand Down
6 changes: 3 additions & 3 deletions test/pummel/test-stream-pipe-multi.js
Original file line number Diff line number Diff line change
Expand Up @@ -49,11 +49,11 @@ process.on('exit', function() {
assert.equal(cnt, rclosed, 'readable streams closed');
});

for (var i = 0; i < chunkSize; i++) {
chunkSize[i] = i % 256;
for (let i = 0; i < chunkSize; i++) {
data[i] = i;
}

for (var i = 0; i < cnt; i++) {
for (let i = 0; i < cnt; i++) {
var r = new FakeStream();
r.on('close', function() {
console.error(this.ID, 'read close');
Expand Down

0 comments on commit 663e852

Please sign in to comment.