Running a test to see if the favicon still gets rendered.

Author: cccs-nr

index.html

@@ -6,7 +6,7 @@
     <meta name="viewport" content="width=360, initial-scale=1.0" />
     <meta name="theme-color" content="#000000" />
     <meta name="description" content="Assemblyline 4 React UI" />
-    <link id="favicon" rel="icon" href="favicon.ico?query=test" />
+    <link id="favicon" rel="icon" href="favicon.ico" />
     <link rel="manifest" href="manifest.json" />
     <title>Assemblyline 4</title>
   </head>

src/helpers/utils.test.ts

@@ -236,7 +236,7 @@ describe('Test `resetFavicon`', () => {
     favicon.setAttribute('href', initial);
     expect(favicon.getAttribute('href')).toBe(initial);
     resetFavicon();
-    expect(favicon.getAttribute('href')).toBe(`${defaultURL}/favicon.ico?query=test`);
+    expect(favicon.getAttribute('href')).toBe(`${defaultURL}/favicon.ico`);
   });
 });
 
@@ -257,7 +257,7 @@ describe('Test `setNotifyFavicon`', () => {
     favicon.setAttribute('href', initial);
     expect(favicon.getAttribute('href')).toBe(initial);
     setNotifyFavicon();
-    expect(favicon.getAttribute('href')).toBe(`${defaultURL}/favicon_done.ico?query=test`);
+    expect(favicon.getAttribute('href')).toBe(`${defaultURL}/favicon_done.ico`);
   });
 });
 

src/helpers/utils.ts

@@ -131,7 +131,7 @@ export function humanReadableNumber(num: number | null) {
 export function resetFavicon() {
   const favicon: HTMLLinkElement = document.querySelector('#favicon');
   // TODO: remove query test
-  favicon.href = `/favicon.ico?query=test`;
+  favicon.href = `/favicon.ico`;
 }
 
 /**
@@ -144,7 +144,7 @@ export function resetFavicon() {
 export function setNotifyFavicon() {
   const favicon: HTMLLinkElement = document.querySelector('#favicon');
   // TODO: remove query test
-  favicon.href = `/favicon_done.ico?query=test`;
+  favicon.href = `/favicon_done.ico`;
 }
 
 /**