test: add else and error case for TextDecoder

add test for tinyurl.com/codeandlearn-encoding-1 add test for tinyurl.com/codeandlearn-encoding-2 PR-URL: #24162 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
nodejs · Jan 12, 2019 · e21d784 · e21d784
1 parent 33fd13c
commit e21d784
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/test/parallel/test-whatwg-encoding-textdecoder-fatal.js b/test/parallel/test-whatwg-encoding-textdecoder-fatal.js
@@ -91,3 +91,15 @@ bad.forEach((t) => {
   assert(!new TextDecoder().fatal);
   assert(new TextDecoder('utf-8', { fatal: true }).fatal);
 }
+
+{
+  const notArrayBufferViewExamples = [false, {}, 1, '', new Error()];
+  notArrayBufferViewExamples.forEach((invalidInputType) => {
+    common.expectsError(() => {
+      new TextDecoder(undefined, null).decode(invalidInputType);
+    }, {
+      code: 'ERR_INVALID_ARG_TYPE',
+      type: TypeError
+    });
+  });
+}
diff --git a/test/parallel/test-whatwg-encoding-textdecoder.js b/test/parallel/test-whatwg-encoding-textdecoder.js
@@ -76,6 +76,14 @@ if (common.hasIntl) {
     });
 }
 
+// Test TextDecoder, label undefined, options null
+{
+  const dec = new TextDecoder(undefined, null);
+  assert.strictEqual(dec.encoding, 'utf-8');
+  assert.strictEqual(dec.fatal, false);
+  assert.strictEqual(dec.ignoreBOM, false);
+}
+
 // Test TextDecoder, UTF-16le
 {
   const dec = new TextDecoder('utf-16le');