test: improve coverage for process.umask

evanlucas · MylesBorins · commit 1fe3e866cffe · 2017-11-28T13:10:30.000+09:00
This ensures that process.umask() throws with the correct error when invalid inputs are supplied. PR-URL: #16188 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
diff --git a/test/parallel/test-umask.js b/test/parallel/test-umask.js
@@ -19,3 +19,11 @@ assert.strictEqual(parseInt(mask, 8), process.umask(old));
 assert.strictEqual(old, process.umask());
 // 2. If the test fails, process.umask() will return 0
 assert.strictEqual(old, process.umask());
+
+assert.throws(() => {
+  process.umask({});
+}, /argument must be an integer or octal string/);
+
+assert.throws(() => {
+  process.umask('123x');
+}, /invalid octal string/);
