Skip to content

Commit

Permalink
test: removed deprecation tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Tiriel committed Jan 11, 2018
1 parent c341f93 commit e93767f
Showing 1 changed file with 0 additions and 24 deletions.
24 changes: 0 additions & 24 deletions test/parallel/test-require-deps-deprecation.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,37 +5,13 @@ const assert = require('assert');
// The v8 modules when imported leak globals. Disable global check.
common.globalCheck = false;

const deprecatedModules = [
'node-inspect/lib/_inspect',
'node-inspect/lib/internal/inspect_client',
'node-inspect/lib/internal/inspect_repl',
'v8/tools/SourceMap',
'v8/tools/codemap',
'v8/tools/consarray',
'v8/tools/csvparser',
'v8/tools/logreader',
'v8/tools/profile',
'v8/tools/profile_view',
'v8/tools/splaytree',
'v8/tools/tickprocessor',
'v8/tools/tickprocessor-driver'
];

// Newly added deps that do not have a deprecation wrapper around it would
// throw an error, but no warning would be emitted.
const deps = [
'acorn/dist/acorn',
'acorn/dist/walk'
];

const throwingModules = () => {
for (const m of deprecatedModules) {
require(m);
}
};

assert.throws(throwingModules, ReferenceError);

// Instead of checking require, check that resolve isn't pointing toward a
// built-in module, as user might already have node installed with acorn in
// require.resolve range.
Expand Down

0 comments on commit e93767f

Please sign in to comment.