Merge pull request #117 from kinotio/develop

fix: remove pdf extension from export filename

Author: andostronaut

mod_test.ts

@@ -39,14 +39,6 @@ Deno.test(
 	},
 )
 
-Deno.test(
-	'generateFileName should return correct filename for pdf type',
-	() => {
-		const timestamp = /^prefix_\d{4}-\d{2}-\d{2}_\d{2}-\d{2}-\d{2}\.pdf$/
-		assert.assert(generateFileName('prefix', 'pdf').match(timestamp))
-	},
-)
-
 Deno.test('generateFileName should handle empty prefix', () => {
 	const tm = getTimestamp({ type: 'log' })
 	assert.assertEquals(generateFileName('', 'log'), `_${tm}.log`)

src/utils.ts

@@ -24,7 +24,7 @@ const getTimestamp = ({ type = 'log' }: { type: 'log' | 'pdf' }): string => {
 		: `${year}-${month}-${day}_${hours}-${minutes}-${seconds}`
 }
 
-const generateFileName = (prefix: string, ext: 'log' | 'pdf'): string => {
+const generateFileName = (prefix: string, ext: 'log'): string => {
 	const timestamp = getTimestamp({ type: ext })
 	return `${prefix}_${timestamp}.${ext}`
 }