(fix): Warnings error should not show on HTTP 204 - case is expected

Author: ArneTR

frontend/js/compare.js

@@ -25,7 +25,11 @@ const fetchWarningsForRuns = async (ids) => {
             const data = await makeAPICall('/v1/warnings/' + id);
             if (data?.data) warnings.push(...data.data);
         } catch (err) {
-            showNotification('Could not get warnings data from API', err);
+            if (err instanceof APIEmptyResponse204) {
+                console.log('No warnings where present in API response. Skipping error as this is allowed case.')
+            } else {
+                showNotification('Could not get warnings data from API', err);
+            }
         }
     }
     return warnings;

frontend/js/stats.js

@@ -593,8 +593,13 @@ const fetchAndFillWarnings = async (url_params) => {
         warnings = await makeAPICall('/v1/warnings/' + url_params['id'])
         if (!warnings || warnings?.data?.length === 0) return;
     } catch (err) {
-        showNotification('Could not get warnings data from API', err);
+        if (err instanceof APIEmptyResponse204) {
+            console.log('No warnings where present in API response. Skipping error as this is allowed case.')
+        } else {
+            showNotification('Could not get warnings data from API', err);
+        }
         return;
+
     }
 
     const container = document.querySelector('#run-warnings');