test that optArg throws on bad arguments

isaacs · Jan 19, 2023 · 89b38cf · 89b38cf
1 parent 1b95def
commit 89b38cf
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -10,10 +10,8 @@ export interface RimrafOptions {
   maxBackoff?: number
 }
 
-/* c8 ignore start */
 const typeOrUndef = (val: any, t: string) =>
   typeof val === 'undefined' || typeof val === t
-/* c8 ignore stop */
 
 export const isRimrafOptions = (o: any): o is RimrafOptions =>
   !!o &&
@@ -25,15 +23,13 @@ export const isRimrafOptions = (o: any): o is RimrafOptions =>
   typeOrUndef(o.backoff, 'number') &&
   typeOrUndef(o.maxBackoff, 'number')
 
-/* c8 ignore start */
 export const assertRimrafOptions: (o: any) => void = (
   o: any
 ): asserts o is RimrafOptions => {
   if (!isRimrafOptions(o)) {
     throw new Error('invalid rimraf options')
   }
 }
-/* c8 ignore stop */
 
 import { rimrafManual, rimrafManualSync } from './rimraf-manual.js'
 import { rimrafMoveRemove, rimrafMoveRemoveSync } from './rimraf-move-remove.js'

diff --git a/test/opt-arg.js b/test/opt-arg.js
@@ -3,3 +3,8 @@ const optArg = require('../dist/cjs/src/opt-arg.js').default
 const opt = { a: 1 }
 t.equal(optArg(opt), opt, 'returns object if provided')
 t.same(optArg(), {}, 'returns new object otherwise')
+
+t.throws(() => optArg(true))
+t.throws(() => optArg(null))
+t.throws(() => optArg('hello'))
+t.throws(() => optArg({ maxRetries: 'banana' }))