Fix/adoptium 2086 uniformize extension of proposed download (#2088)

* [adoptium-2086] Uniformize extension of proposed download * Fix tests to be all greeen * Keep same notation without ';' at end of line
adoptium · Aug 11, 2023 · cd9603e · cd9603e
1 parent 47cb127
commit cd9603e
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 14 deletions.
diff --git a/...s/MarketplaceDownloadTable/__tests__/__snapshots__/MarketplaceDownloadTable.test.tsx.snap b/...s/MarketplaceDownloadTable/__tests__/__snapshots__/MarketplaceDownloadTable.test.tsx.snap
@@ -226,7 +226,7 @@ exports[`MarketplaceDownloadTable component > renders correctly 1`] = `
                       />
                     </svg>
                      
-                    tar.gz
+                    .tar.gz
                   </a>
                 </td>
               </tr>
@@ -407,7 +407,7 @@ exports[`MarketplaceDownloadTable component > renders correctly 1`] = `
                       />
                     </svg>
                      
-                    tar.gz
+                    .tar.gz
                   </a>
                 </td>
               </tr>

diff --git a/src/hooks/__tests__/__snapshots__/fetchTemurinArchive.test.tsx.snap b/src/hooks/__tests__/__snapshots__/fetchTemurinArchive.test.tsx.snap
@@ -11,7 +11,7 @@ exports[`getAssetsForVersion > returns valid JSON - ea 1`] = `
             {
               "architecture": "arch_mock",
               "checksum": "checksum_mock",
-              "extension": "tar.gz",
+              "extension": ".tar.gz",
               "link": "https://link_mock/",
               "os": "os_mock",
               "size": 0,
@@ -39,7 +39,7 @@ exports[`getAssetsForVersion > returns valid JSON - with installers 1`] = `
             {
               "architecture": "arch_mock",
               "checksum": "checksum_mock",
-              "extension": "tar.gz",
+              "extension": ".tar.gz",
               "installer_checksum": "installer_checksum_mock",
               "installer_extension": ".msi",
               "installer_link": "https://installer_link_mock/",
@@ -71,7 +71,7 @@ exports[`getAssetsForVersion > returns valid JSON - with release notes 1`] = `
             {
               "architecture": "arch_mock",
               "checksum": "checksum_mock",
-              "extension": "tar.gz",
+              "extension": ".tar.gz",
               "link": "https://link_mock/",
               "os": "os_mock",
               "size": 0,
@@ -100,7 +100,7 @@ exports[`getAssetsForVersion > returns valid JSON - with source 1`] = `
             {
               "architecture": "arch_mock",
               "checksum": "checksum_mock",
-              "extension": "tar.gz",
+              "extension": ".tar.gz",
               "link": "https://link_mock/",
               "os": "os_mock",
               "size": 0,
@@ -129,7 +129,7 @@ exports[`getAssetsForVersion > returns valid JSON 1`] = `
             {
               "architecture": "arch_mock",
               "checksum": "checksum_mock",
-              "extension": "tar.gz",
+              "extension": ".tar.gz",
               "link": "https://link_mock/",
               "os": "os_mock",
               "size": 0,

diff --git a/src/hooks/__tests__/__snapshots__/fetchTemurinReleases.test.tsx.snap b/src/hooks/__tests__/__snapshots__/fetchTemurinReleases.test.tsx.snap
@@ -7,7 +7,7 @@ exports[`loadLatestAssets > returns valid JSON + installer 1`] = `
     "binaries": [
       {
         "checksum": "checksum_mock",
-        "extension": "tar.gz",
+        "extension": ".tar.gz",
         "installer_checksum": "installer_checksum_mock",
         "installer_extension": ".msi",
         "installer_link": "https://installer_link_mock/",
@@ -33,7 +33,7 @@ exports[`loadLatestAssets > returns valid JSON 1`] = `
     "binaries": [
       {
         "checksum": "checksum_mock",
-        "extension": "tar.gz",
+        "extension": ".tar.gz",
         "link": "https://link_mock/",
         "size": 0,
         "type": "JDK",
@@ -55,7 +55,7 @@ exports[`loadLatestAssets > source image is processed correctly 1`] = `
     "binaries": [
       {
         "checksum": "checksum_mock",
-        "extension": "tar.gz",
+        "extension": ".tar.gz",
         "link": "https://link_mock/",
         "size": 0,
         "type": "JDK",

diff --git a/src/util/__tests__/fetchExtension.test.tsx b/src/util/__tests__/fetchExtension.test.tsx
@@ -3,11 +3,11 @@ import { describe, expect, it } from 'vitest'
 
 describe("fetchExtension", () => {
     it("should convert file.tar.gz to tar.gz", () => {
-        expect(fetchExtension("file.tar.gz")).toBe("tar.gz");
+        expect(fetchExtension("file.tar.gz")).toBe(".tar.gz");
     });
 
     it("should convert file.tar.xz to tar.xz", () => {
-        expect(fetchExtension("file.tar.xz")).toBe("tar.xz");
+        expect(fetchExtension("file.tar.xz")).toBe(".tar.xz");
     });
 
     it("should convert file.zip to .zip", () => {

diff --git a/src/util/fetchExtension.tsx b/src/util/fetchExtension.tsx
@@ -2,9 +2,9 @@ export function fetchExtension (filename: string) {
   let extension = `.${filename.split('.').pop()}`
   // Workaround to prevent extension returning as .gz
   if (extension === '.gz') {
-    extension = 'tar.gz'
+    extension = '.tar.gz'
   } else if (extension === '.xz') {
-    extension = 'tar.xz'
+    extension = '.tar.xz'
   }
   return extension
 }