From 3ad7c1ae9778fd9a61b4e99eeab4291827700d55 Mon Sep 17 00:00:00 2001 From: Daniel Bevenius Date: Tue, 13 Mar 2018 06:56:06 +0100 Subject: [PATCH] test: remove unused deprecation code MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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: https://github.com/nodejs/node/pull/19317 Reviewed-By: Yuta Hiroto Reviewed-By: Colin Ihrig Reviewed-By: Luigi Pinca Reviewed-By: Khaidi Chu Reviewed-By: Jackson Tian Reviewed-By: James M Snell Reviewed-By: Tobias Nießen --- test/parallel/test-process-assert.js | 3 +-- test/parallel/test-process-env-deprecation.js | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/test/parallel/test-process-assert.js b/test/parallel/test-process-assert.js index 74792eebb7b..659fa8ed7ab 100644 --- a/test/parallel/test-process-assert.js +++ b/test/parallel/test-process-assert.js @@ -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); diff --git a/test/parallel/test-process-env-deprecation.js b/test/parallel/test-process-env-deprecation.js index 68817b320b4..310023fba77 100644 --- a/test/parallel/test-process-env-deprecation.js +++ b/test/parallel/test-process-env-deprecation.js @@ -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;