Changes asked in code review

@vates/parse-duration/index.js

@@ -6,12 +6,11 @@ exports.parseDuration = value => {
   if (typeof value === 'number') {
     return value
   }
-  if (typeof value !== 'string' || value === '') {
-    throw new TypeError(`not a valid duration: ${value}`)
+  if (typeof value === 'string' && value !== '') {
+    const duration = ms(value)
+    if (duration !== undefined) {
+      return duration
+    }
   }
-  const duration = ms(value)
-  if (duration === undefined) {
-    throw new TypeError(`not a valid duration: ${value}`)
-  }
-  return duration
+  throw new TypeError(`not a valid duration: ${value}`)
 }

@vates/parse-duration/package.json

@@ -27,6 +27,7 @@
     "ms": "^2.1.2"
   },
   "scripts": {
-    "postversion": "npm publish --access public"
+    "postversion": "npm publish --access public",
+    "test": "node--test"
   }
 }

@vates/parse-duration/parse-duration.test.mjs

@@ -15,8 +15,9 @@ describe('parseDuration()', () => {
     assert.strictEqual(parseDuration(input), expected)
   })
 
-  it('should throw an error for an undefined duration', () => {
-    const input = undefined
-    assert.throws(() => parseDuration(input), { message: 'not a valid duration: undefined' })
+  it('should throw an error for an invalid duration', () => {
+    ;[undefined, '', 'invalid duration'].forEach(input => {
+      assert.throws(() => parseDuration(input), { message: `not a valid duration: ${input}` })
+    })
   })
 })

CHANGELOG.unreleased.md

@@ -40,7 +40,6 @@
 
 <!--packages-start-->
 
-- @vates/parse-duration minor
 - @xen-orchestra/lite minor
 - @xen-orchestra/web minor
 - @xen-orchestra/web-core minor