fix: add test coverage for --hls option without argument

Previously missing test for the case where --hls is provided but no
directory argument is given. Also refined validation to be more precise
about what constitutes an invalid directory path.

Author: limulus

src/bin/tcx2webvtt.spec.ts

@@ -295,13 +295,12 @@ describe('tcx2webvtt CLI', { timeout: 10000 }, () => {
       expect(mockProcess.exit).toHaveBeenCalledWith(1)
     })
 
-    it('should error when HLS directory path is whitespace only', async () => {
-      const tcxFile = join(__dirname, '../../fixtures/tcx/concept2.tcx')
-      mockProcess.argv.push('--hls', '   ', tcxFile)
+    it('should error when --hls option has no argument', async () => {
+      mockProcess.argv.push('--hls')
 
       await main(mockProcess)
 
-      expect(stderr).toContain('HLS directory path cannot be empty')
+      expect(stderr).toContain("Option '--hls <value>' argument missing")
       expect(mockProcess.exit).toHaveBeenCalledWith(1)
     })
 

src/bin/tcx2webvtt.ts

@@ -91,7 +91,7 @@ export async function main(proc: ProcessLike) {
     }
 
     // Validate HLS directory argument
-    if (values.hls !== undefined && !values.hls.trim()) {
+    if (values.hls !== undefined && values.hls === '') {
       proc.stderr.write('Error: HLS directory path cannot be empty\n')
       proc.exit(1)
     }