Skip to content

Commit

Permalink
test: remove unreachable code
Browse files Browse the repository at this point in the history
There is no way a line can be called after throwing an exception.
  • Loading branch information
targos committed Aug 20, 2015
1 parent 057126a commit 249db38
Show file tree
Hide file tree
Showing 6 changed files with 1 addition and 8 deletions.
2 changes: 0 additions & 2 deletions test/message/throw_custom_error.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,3 @@ console.error('before');

// custom error throwing
throw ({ name: 'MyCustomError', message: 'This is a custom message' });

console.error('after');
2 changes: 0 additions & 2 deletions test/message/throw_non_error.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,3 @@ console.error('before');

// custom error throwing
throw ({ foo: 'bar' });

console.error('after');
1 change: 0 additions & 1 deletion test/parallel/test-domain.js
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,6 @@ expectCaught++;
// as a callback instead.
function fn(er) {
throw new Error('This function should never be called!');
process.exit(1);
}

var bound = d.intercept(fn);
Expand Down
1 change: 0 additions & 1 deletion test/parallel/test-file-write-stream.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ file
})
.on('error', function(err) {
throw err;
console.error('error!', err.stack);
})
.on('drain', function() {
console.error('drain!', callbacks.drain);
Expand Down
1 change: 0 additions & 1 deletion test/parallel/test-http-content-length.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@ var server = http.createServer(function(req, res) {
break;
default:
throw new Error('Unreachable');
break;
}

receivedRequests++;
Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-listen-fd-detached-inherit.js
Original file line number Diff line number Diff line change
Expand Up @@ -64,8 +64,8 @@ function test() {
// Then output the child's pid, and immediately exit.
function parent() {
var server = net.createServer(function(conn) {
throw new Error('Should not see connections on parent');
conn.end('HTTP/1.1 403 Forbidden\r\n\r\nI got problems.\r\n');
throw new Error('Should not see connections on parent');
}).listen(PORT, function() {
console.error('server listening on %d', PORT);

Expand Down

0 comments on commit 249db38

Please sign in to comment.