Skip to content

Fix 11826 #11920

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
clarenced wants to merge 9 commits into from
Closed

Fix 11826 #11920

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
86ff5b0
test: break up the tests in separate file
clarenced Mar 16, 2017
ef6819c
test: lint error correction
clarenced Mar 17, 2017
d1d1a7d
test:lint error correction
clarenced Mar 18, 2017
aeaf585
test: documentation for function childShouldNotThrowAndAbort added
clarenced Mar 19, 2017
8b663fa
test: old file test-domain-abort-on-uncaught.js deleted
clarenced Mar 19, 2017
6156f98
test: documentation fix and msg error update in childShouldNotThrowAn…
clarenced Mar 20, 2017
c78ca3b
test: jslint correction
clarenced Mar 28, 2017
38ac775
test: remove unnecessary indentation
clarenced Apr 2, 2017
4bb01a7
test: blank line between childShouldNotThrowAndAbort and following pa…
clarenced Apr 17, 2017
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: lint error correction
  • Loading branch information
@clarenced
clarenced committed Apr 17, 2017
commit ef6819c5464b1cad7d697f4cbb1d3a9a8879b575
2 changes: 1 addition & 1 deletion test/common.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -260,7 +260,7 @@ exports.childShouldThrowAndAbort = function() {

exports.childShouldNotThrowAndAbort = () => {
let testCmd = '';
if(!exports.isWindows){
if (!exports.isWindows) {
testCmd += 'ulimit -c 0 && ';
}

Expand Down
5 changes: 2 additions & 3 deletions test/parallel/test-domain-abort-on-uncaught-0.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@
const common = require('../common');
const assert = require('assert');
const domain = require('domain');
const child_process = require('child_process');

let errorHandlerCalled = false;

let nextTick = () => {
const nextTick = () => {
const d = domain.create();

d.once('error', (err) => {
Expand All @@ -21,7 +20,7 @@ let nextTick = () => {
d.run(() => {
process.nextTick(() => {
throw new Error('exceptional!');
})
});
});
};

Expand Down
3 changes: 1 addition & 2 deletions test/parallel/test-domain-abort-on-uncaught-1.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@
const common = require('../common');
const assert = require('assert');
const domain = require('domain');
const child_process = require('child_process');

let errorHandlerCalled = false;

let test = () => {
const test = () => {
const d = domain.create();

d.once('error', (err) => {
Expand Down
3 changes: 1 addition & 2 deletions test/parallel/test-domain-abort-on-uncaught-10.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@
const common = require('../common');
const assert = require('assert');
const domain = require('domain');
const child_process = require('child_process');

let errorHandlerCalled = false;

let test = () => {
const test = () => {
const d = domain.create();
const d2 = domain.create();

Expand Down
3 changes: 1 addition & 2 deletions test/parallel/test-domain-abort-on-uncaught-11.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@
const common = require('../common');
const assert = require('assert');
const domain = require('domain');
const child_process = require('child_process');

let errorHandlerCalled = false;

let test = () => {
const test = () => {
const d = domain.create();
const d2 = domain.create();

Expand Down
3 changes: 1 addition & 2 deletions test/parallel/test-domain-abort-on-uncaught-12.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@
const common = require('../common');
const assert = require('assert');
const domain = require('domain');
const child_process = require('child_process');

let errorHandlerCalled = false;

let test = () => {
const test = () => {
const d = domain.create();
const d2 = domain.create();

Expand Down
3 changes: 1 addition & 2 deletions test/parallel/test-domain-abort-on-uncaught-2.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@
const common = require('../common');
const assert = require('assert');
const domain = require('domain');
const child_process = require('child_process');

let errorHandlerCalled = false;

let test = () => {
const test = () => {
const d = domain.create();

d.once('error', (err) => {
Expand Down
3 changes: 1 addition & 2 deletions test/parallel/test-domain-abort-on-uncaught-3.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@
const common = require('../common');
const assert = require('assert');
const domain = require('domain');
const child_process = require('child_process');

let errorHandlerCalled = false;

let test = () => {
const test = () => {
const d = domain.create();

d.once('error', (err) => {
Expand Down
1 change: 0 additions & 1 deletion test/parallel/test-domain-abort-on-uncaught-4.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
const common = require('../common');
const assert = require('assert');
const domain = require('domain');
const child_process = require('child_process');

let errorHandlerCalled = false;

Expand Down
3 changes: 1 addition & 2 deletions test/parallel/test-domain-abort-on-uncaught-5.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@
const common = require('../common');
const assert = require('assert');
const domain = require('domain');
const child_process = require('child_process');

let errorHandlerCalled = false;

let test = () => {
const test = () => {
const d = domain.create();

d.once('error', (err) => {
Expand Down
10 changes: 5 additions & 5 deletions test/parallel/test-domain-abort-on-uncaught-6.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,24 +7,24 @@
const common = require('../common');
const assert = require('assert');
const domain = require('domain');
const child_process = require('child_process');
const net = require('net');

let errorHandlerCalled = false;

let test = () => {
const test = () => {
const d = domain.create();

d.once('error', (err) => {
errorHandlerCalled = true;
});

d.run(function() {
const server = net.createServer(function(conn) {
const server = net.createServer((conn) => {
conn.pipe(conn);
});
server.listen(0, common.localhostIPv4, function() {
server.listen(0, common.localhostIPv4, () => {
const conn = net.connect(this.address().port, common.localhostIPv4);
conn.once('data', function() {
conn.once('data', () => {
throw new Error('ok');
});
conn.end('ok');
Expand Down
3 changes: 1 addition & 2 deletions test/parallel/test-domain-abort-on-uncaught-7.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@
const common = require('../common');
const assert = require('assert');
const domain = require('domain');
const child_process = require('child_process');

let errorHandlerCalled = false;

let test = () => {
const test = () => {
const d = domain.create();
const d2 = domain.create();

Expand Down
21 changes: 10 additions & 11 deletions test/parallel/test-domain-abort-on-uncaught-8.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,23 +7,22 @@
const common = require('../common');
const assert = require('assert');
const domain = require('domain');
const child_process = require('child_process');

let errorHandlerCalled = false;

let test = () => {
const d = domain.create();
const d2 = domain.create();
const test = () => {
const d = domain.create();
const d2 = domain.create();

d2.on('error', function errorHandler() {
errorHandlerCalled = true;
});
d2.on('error', function errorHandler() {
errorHandlerCalled = true;
});

d.run(function() {
d2.run(function() {
throw new Error('boom!');
});
d.run(function() {
d2.run(function() {
throw new Error('boom!');
});
});
};


Expand Down
3 changes: 1 addition & 2 deletions test/parallel/test-domain-abort-on-uncaught-9.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@
const common = require('../common');
const assert = require('assert');
const domain = require('domain');
const child_process = require('child_process');

let errorHandlerCalled = false;

let test = () => {
const test = () => {
const d = domain.create();
const d2 = domain.create();

Expand Down