Skip to content
This repository has been archived by the owner on Oct 15, 2020. It is now read-only.

Commit

Permalink
test: Suppress debugger test and fixed jslint
Browse files Browse the repository at this point in the history
* Suppress running debugger test
* Fixed jslint warning that came post-merge
* Currently there are 18 failures. `parallel/test-util-inspect-simd`
fails because chakracore has not implemented all the SIMD objects
that test uses. `parallel/test-vm-low-stack-space` is also a new
failure that expects vm to run script after we catch `out of stack
space` exception. I will debug this further to see gap in chakrashim
but for now, letting this test to fail with node+chakracore.

PR-URL: #74
Reviewed-By: Jianchun Xu <Jianchun.Xu@microsoft.com>
  • Loading branch information
kunalspathak committed Jun 2, 2016
1 parent 5556a39 commit 407e16c
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
6 changes: 6 additions & 0 deletions test/parallel/test-debug-port-numbers.js
Expand Up @@ -11,6 +11,12 @@ const spawn = require('child_process').spawn;
// instead of just the debugger process.
const detached = !common.isWindows;

if (common.isChakraEngine) {
console.log('1..0 # Skipped: This test is disabled for chakra engine ' +
'because debugger support is not implemented yet.');
return;
}

const children = [];
for (let i = 0; i < 4; i += 1) {
const port = common.PORT + i;
Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-debug-signal-cluster.js
Expand Up @@ -9,7 +9,7 @@ if (common.isChakraEngine) {
return;
}

const PORT_MIN = common.PORT + 1; // The fixture uses common.PORT.
const port = common.PORT + 1; // The fixture uses common.PORT.
var args = ['--debug-port=' + port,
common.fixturesDir + '/clustered-server/app.js'];
var options = { stdio: ['inherit', 'inherit', 'pipe', 'ipc'] };
Expand Down

0 comments on commit 407e16c

Please sign in to comment.