test: improve test coverage SourceMap API

juanarbol · targos · commit 94a877b0a4f0 · 2021-05-01T09:44:32.000+02:00
PR-URL: #36089 Reviewed-By: Zeyu Yang <himself65@outlook.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
diff --git a/test/parallel/test-source-map-api.js b/test/parallel/test-source-map-api.js
@@ -1,11 +1,26 @@
 // Flags: --enable-source-maps
 'use strict';
 
-require('../common');
+const common = require('../common');
 const assert = require('assert');
 const { findSourceMap, SourceMap } = require('module');
 const { readFileSync } = require('fs');
 
+// It should throw with invalid args.
+{
+  [1, true, 'foo'].forEach((invalidArg) =>
+    assert.throws(
+      () => new SourceMap(invalidArg),
+      {
+        code: 'ERR_INVALID_ARG_TYPE',
+        name: 'TypeError',
+        message: 'The "payload" argument must be of type object.' +
+               common.invalidArgTypeHelper(invalidArg)
+      }
+    )
+  );
+}
+
 // findSourceMap() can lookup source-maps based on URIs, in the
 // non-exceptional case.
 {
