Skip to content

Commit

Permalink
test: remove unused deprecation code
Browse files Browse the repository at this point in the history
Currently there are two tests that specify a third argument, a
deprecation code string, when calling common.expectWarning. The
function only takes two arguments and this third argument is not used.

This commit removes the deprecation code.

PR-URL: #19317
Reviewed-By: Yuta Hiroto <hello@hiroppy.me>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Khaidi Chu <i@2333.moe>
Reviewed-By: Jackson Tian <shyvo1987@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
  • Loading branch information
danbev committed Mar 15, 2018
1 parent 536190f commit 3ad7c1a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
3 changes: 1 addition & 2 deletions test/parallel/test-process-assert.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,7 @@ const assert = require('assert');

common.expectWarning(
'DeprecationWarning',
'process.assert() is deprecated. Please use the `assert` module instead.',
'DEP0100'
'process.assert() is deprecated. Please use the `assert` module instead.'
);

assert.strictEqual(process.assert(1, 'error'), undefined);
Expand Down
3 changes: 1 addition & 2 deletions test/parallel/test-process-env-deprecation.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,7 @@ common.expectWarning(
'DeprecationWarning',
'Assigning any value other than a string, number, or boolean to a ' +
'process.env property is deprecated. Please make sure to convert the value ' +
'to a string before setting process.env with it.',
'DEP0104'
'to a string before setting process.env with it.'
);

process.env.ABC = undefined;
Expand Down

0 comments on commit 3ad7c1a

Please sign in to comment.