fix: remove pdf extension from export filename

kinotiodotio · Jan 11, 2025 · 42f412e · 42f412e
1 parent 05a9512
commit 42f412e
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 9 deletions.
diff --git a/mod_test.ts b/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`)

diff --git a/src/utils.ts b/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}`
 }