From 4e7ae082c5491d5a99082bbb2fd923fef359150d Mon Sep 17 00:00:00 2001
From: pyoor <jkratzer@mozilla.com>
Date: Mon, 26 Jan 2026 12:44:52 -0500
Subject: [PATCH] feat: modify template to accept basename instead of full
 filename

---
 .../src/components/Bugs/PublicationForm.vue   |  4 +--
 server/frontend/src/helpers.js                | 26 ++++++++-----------
 server/frontend/tests/helpers.test.js         |  8 +++---
 3 files changed, 16 insertions(+), 22 deletions(-)

diff --git a/server/frontend/src/components/Bugs/PublicationForm.vue b/server/frontend/src/components/Bugs/PublicationForm.vue
index 4997d77d..588ac7da 100644
--- a/server/frontend/src/components/Bugs/PublicationForm.vue
+++ b/server/frontend/src/components/Bugs/PublicationForm.vue
@@ -516,9 +516,7 @@
 
         <div v-if="isBugTemplateCreation" class="row">
           <div class="form-group col-md-6">
-            <label for="testcase_filename"
-              >Filename that will be used for the testcase</label
-            >
+            <label for="testcase_filename">Testcase basename</label>
             <input
               id="testcase_filename"
               v-model="fileName"
diff --git a/server/frontend/src/helpers.js b/server/frontend/src/helpers.js
index 6d5d10bc..da9ac5b5 100644
--- a/server/frontend/src/helpers.js
+++ b/server/frontend/src/helpers.js
@@ -222,12 +222,12 @@ export function formatMonthly(d) {
 }
 
 /**
- * Parse a testcase file path into basename and extension
- * @param {string} testcasePath - Full path to the testcase file (e.g. "/path/to/test.txt")
- * @param {string|null} templateFilename - Optional template filename to use instead of original
- * @returns {{basename: string, extension: string|null}} Object with basename and extension
+ * Parse a testcase file path into basename and extension.
+ * @param {string} testcasePath - Testcase file path.
+ * @param {string|null} templateBasename - Optional template basename.
+ * @returns {{basename: string, extension: string|null}} Object with basename and extension.
  */
-export function parseFilename(testcasePath, templateFilename = null) {
+export function parseFilename(testcasePath, templateBasename = null) {
   const pathParts = testcasePath.split("/");
   const originalFilename = pathParts[pathParts.length - 1];
   const originalParts = originalFilename.split(".");
@@ -238,12 +238,8 @@ export function parseFilename(testcasePath, templateFilename = null) {
     : null;
 
   let basename;
-  if (templateFilename) {
-    const templateParts = templateFilename.split(".");
-    basename =
-      templateParts.length > 1
-        ? templateParts.slice(0, -1).join(".")
-        : templateParts[0];
+  if (templateBasename) {
+    basename = templateBasename;
   } else {
     basename = hasExtension
       ? originalParts.slice(0, -1).join(".")
@@ -254,10 +250,10 @@ export function parseFilename(testcasePath, templateFilename = null) {
 }
 
 /**
- * Build a filename from basename and extension
- * @param {string} basename - The base filename without extension
- * @param {string|null} extension - Optional file extension (without dot)
- * @returns {string} Complete filename with extension if provided
+ * Build a filename from basename and extension.
+ * @param {string} basename - The file basename.
+ * @param {string|null} extension - Optional file extension.
+ * @returns {string}
  */
 export function buildFilename(basename, extension) {
   return extension ? `${basename}.${extension}` : basename;
diff --git a/server/frontend/tests/helpers.test.js b/server/frontend/tests/helpers.test.js
index 3a455117..89b10524 100644
--- a/server/frontend/tests/helpers.test.js
+++ b/server/frontend/tests/helpers.test.js
@@ -5,13 +5,13 @@ describe("parseFilename", () => {
     [
       "testcase with file extension",
       "tests/test.txt",
-      "testcase.zip",
+      "testcase",
       { basename: "testcase", extension: "txt" },
     ],
     [
       "testcase without an extension",
       "tests/original",
-      "template_name.html",
+      "template_name",
       { basename: "template_name", extension: null },
     ],
     [
@@ -38,8 +38,8 @@ describe("parseFilename", () => {
       null,
       { basename: "test.min", extension: "js" },
     ],
-  ])("%s", (_description, testcasePath, templateFilename, expected) => {
-    const result = parseFilename(testcasePath, templateFilename);
+  ])("%s", (_description, testcasePath, templateBasename, expected) => {
+    const result = parseFilename(testcasePath, templateBasename);
     expect(result).toEqual(expected);
   });
 });