test: improve coverage of SourceTextModule getters

nodejs · Jan 21, 2021 · 5a34564 · 5a34564
1 parent e279304
commit 5a34564
Showing 1 changed file with 14 additions and 4 deletions.
diff --git a/test/parallel/test-vm-module-errors.js b/test/parallel/test-vm-module-errors.js
@@ -235,16 +235,26 @@ function checkInvalidCachedData() {
 }
 
 function checkGettersErrors() {
+  const expecteError = {
+    code: 'ERR_VM_MODULE_NOT_MODULE',
+    message: /Provided module is not an instance of Module/
+  };
   const getters = ['identifier', 'context', 'namespace', 'status', 'error'];
   getters.forEach((getter) => {
     assert.throws(() => {
       // eslint-disable-next-line no-unused-expressions
       Module.prototype[getter];
-    }, {
-      code: 'ERR_VM_MODULE_NOT_MODULE',
-      message: /Provided module is not an instance of Module/
-    });
+    }, expecteError);
+    assert.throws(() => {
+      // eslint-disable-next-line no-unused-expressions
+      SourceTextModule.prototype[getter];
+    }, expecteError);
   });
+  // `dependencySpecifiers` getter is just part of SourceTextModule
+  assert.throws(() => {
+    // eslint-disable-next-line no-unused-expressions
+    SourceTextModule.prototype.dependencySpecifiers;
+  }, expecteError);
 }
 
 const finished = common.mustCall();