test: simplify vm-module-errors test

Use `assert.rejects` to test error cases. This simplifies the test. PR-URL: #27123 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Yuta Hiroto <hello@hiroppy.me> Reviewed-By: James M Snell <jasnell@gmail.com>
nodejs · Apr 10, 2019 · 3d6533ea02d53a11f550769c70028d00b8d2d391 · 3d6533e
1 parent eb8a51a
commit 3d6533ea02d53a11f550769c70028d00b8d2d391
Showing with 13 additions and 36 deletions.

+13 −36 test/parallel/test-vm-module-errors.js
diff --git a/test/parallel/test-vm-module-errors.js b/test/parallel/test-vm-module-errors.js
@@ -8,26 +8,6 @@ const assert = require('assert');
 
 const { SourceTextModule, createContext } = require('vm');
 
-async function expectsRejection(fn, settings) {
-  const validateError = common.expectsError(settings);
-  // Retain async context.
-  const storedError = new Error('Thrown from:');
-  try {
-    await fn();
-  } catch (err) {
-    try {
-      validateError(err);
-    } catch (validationError) {
-      console.error(validationError);
-      console.error('Original error:');
-      console.error(err);
-      throw storedError;
-    }
-    return;
-  }
-  assert.fail('Missing expected exception');
-}
-
 async function createEmptyLinkedModule() {
   const m = new SourceTextModule('');
   await m.link(common.mustNotCall());
@@ -57,19 +37,19 @@ async function checkArgType() {
   for (const invalidLinker of [
     0, 1, undefined, null, true, 'str', {}, Symbol.iterator
   ]) {
-    await expectsRejection(async () => {
+    await assert.rejects(async () => {
       const m = new SourceTextModule('');
       await m.link(invalidLinker);
     }, {
       code: 'ERR_INVALID_ARG_TYPE',
-      type: TypeError
+      name: 'TypeError'
     });
   }
 }
 
 // Check methods/properties can only be used under a specific state.
 async function checkModuleState() {
-  await expectsRejection(async () => {
+  await assert.rejects(async () => {
     const m = new SourceTextModule('');
     await m.link(common.mustNotCall());
     assert.strictEqual(m.linkingStatus, 'linked');
@@ -78,7 +58,7 @@ async function checkModuleState() {
     code: 'ERR_VM_MODULE_ALREADY_LINKED'
   });
 
-  await expectsRejection(async () => {
+  await assert.rejects(async () => {
     const m = new SourceTextModule('');
     m.link(common.mustNotCall());
     assert.strictEqual(m.linkingStatus, 'linking');
@@ -94,15 +74,14 @@ async function checkModuleState() {
     code: 'ERR_VM_MODULE_NOT_LINKED'
   });
 
-  await expectsRejection(async () => {
+  await assert.rejects(async () => {
     const m = new SourceTextModule('import "foo";');
     try {
       await m.link(common.mustCall(() => ({})));
     } catch {
       assert.strictEqual(m.linkingStatus, 'errored');
       m.instantiate();
     }
-    assert.fail('Unreachable');
   }, {
     code: 'ERR_VM_MODULE_NOT_LINKED'
   });
@@ -124,15 +103,15 @@ async function checkModuleState() {
     await m.evaluate();
   }
 
-  await expectsRejection(async () => {
+  await assert.rejects(async () => {
     const m = new SourceTextModule('');
     await m.evaluate();
   }, {
     code: 'ERR_VM_MODULE_STATUS',
     message: 'Module status must be one of instantiated, evaluated, and errored'
   });
 
-  await expectsRejection(async () => {
+  await assert.rejects(async () => {
     const m = new SourceTextModule('');
     await m.evaluate(false);
   }, {
@@ -141,7 +120,7 @@ async function checkModuleState() {
              'Received type boolean'
   });
 
-  await expectsRejection(async () => {
+  await assert.rejects(async () => {
     const m = await createEmptyLinkedModule();
     await m.evaluate();
   }, {
@@ -157,7 +136,7 @@ async function checkModuleState() {
     message: 'Module status must be errored'
   });
 
-  await expectsRejection(async () => {
+  await assert.rejects(async () => {
     const m = await createEmptyLinkedModule();
     m.instantiate();
     await m.evaluate();
@@ -175,7 +154,7 @@ async function checkModuleState() {
     message: 'Module status must not be uninstantiated or instantiating'
   });
 
-  await expectsRejection(async () => {
+  await assert.rejects(async () => {
     const m = await createEmptyLinkedModule();
     m.namespace;
   }, {
@@ -186,20 +165,19 @@ async function checkModuleState() {
 
 // Check link() fails when the returned module is not valid.
 async function checkLinking() {
-  await expectsRejection(async () => {
+  await assert.rejects(async () => {
     const m = new SourceTextModule('import "foo";');
     try {
       await m.link(common.mustCall(() => ({})));
     } catch (err) {
       assert.strictEqual(m.linkingStatus, 'errored');
       throw err;
     }
-    assert.fail('Unreachable');
   }, {
     code: 'ERR_VM_MODULE_NOT_MODULE'
   });
 
-  await expectsRejection(async () => {
+  await assert.rejects(async () => {
     const c = createContext({ a: 1 });
     const foo = new SourceTextModule('', { context: c });
     await foo.link(common.mustNotCall());
@@ -210,12 +188,11 @@ async function checkLinking() {
       assert.strictEqual(bar.linkingStatus, 'errored');
       throw err;
     }
-    assert.fail('Unreachable');
   }, {
     code: 'ERR_VM_MODULE_DIFFERENT_CONTEXT'
   });
 
-  await expectsRejection(async () => {
+  await assert.rejects(async () => {
     const erroredModule = new SourceTextModule('import "foo";');
     try {
       await erroredModule.link(common.mustCall(() => ({})));