[MWPW-186048] Updated LANA logs to use severity levels correctly (#5416)

* updated LANA logs to use the severity parameter

* fixed unit tests

Author: robert-bogos

libs/blocks/adobetv/adobetv.js

@@ -3,7 +3,10 @@ import { createTag, getConfig, getFederatedContentRoot } from '../../utils/utils
 
 let captionsLangMapPromise = null;
 
-const logError = (msg, error) => window.lana.log(`${msg}: ${error}`);
+const logError = (msg, error) => window.lana.log(`${msg}: ${error}`, {
+  tags: 'adobetv',
+  severity: 'error',
+});
 
 const updateCaptionsLang = (url, geo, captionsLangMap) => {
   if (geo && captionsLangMap) {

libs/blocks/article-feed/article-helpers.js

@@ -135,7 +135,10 @@ export async function loadTaxonomy() {
         a.href = tax.link;
       } else {
         // eslint-disable-next-line no-console
-        window.lana.log(`Trying to get a link for an unknown topic: ${topic} (current page)`, { tags: 'article-feed' });
+        window.lana.log(
+          `Trying to get a link for an unknown topic: ${topic} (current page)`,
+          { tags: 'article-feed', severity: 'warning' },
+        );
         a.href = '#';
       }
       delete a.dataset.topicLink;

libs/blocks/article-header/article-header.js

@@ -9,7 +9,7 @@ async function validateAuthorUrl(url) {
   const resp = await fetch(`${url.toLowerCase()}.plain.html`);
   if (!resp?.ok) {
     /* c8 ignore next 3 */
-    window.lana?.log(`Could not retrieve metadata for ${url}`, { tags: 'article-header' });
+    window.lana?.log(`Could not retrieve metadata for ${url}`, { tags: 'article-header', severity: 'critical' });
     return null;
   }
 

libs/blocks/caas/utils.js

@@ -678,7 +678,10 @@ async function getLingoSiteLocale(origin, path, fqdn = 'www.adobe.com') {
       });
     return lingoSiteMapping;
   } catch (e) {
-    window.lana?.log('Failed to load lingo-site-mapping.json:', e);
+    window.lana?.log(`Failed to load lingo-site-mapping.json: ${e}`, {
+      tags: 'caas',
+      severity: 'error',
+    });
   }
   return lingoSiteMapping;
 }

libs/blocks/event-rich-results/event-rich-results.js

@@ -16,7 +16,7 @@ function logNullValues(obj) {
   Object.keys(obj).forEach((key) => {
     const value = obj[key];
     if (!value || value === '') {
-      window.lana.log(`Event property ${key} is not defined`, { tags: 'event-rich-results' });
+      window.lana.log(`Event property ${key} is not defined`, { tags: 'event-rich-results', severity: 'warning' });
     }
     logNullValues(value);
   });

libs/blocks/faas/utils.js

@@ -248,7 +248,10 @@ const beforeSubmitCallback = () => {
       }),
     })
       .catch((error) => {
-        window.lana.log(`AA Sandbox Error: ${error.reason || error.error || error.message || error}`, { tags: 'faas', errorType: 'i' });
+        window.lana.log(
+          `AA Sandbox Error: ${error.reason || error.error || error.message || error}`,
+          { tags: 'faas', errorType: 'i', severity: 'error' },
+        );
       });
   }
 };

libs/blocks/fragment/fragment.js

@@ -127,7 +127,10 @@ export default async function init(a) {
   }
 
   if (isCircularRef(relHref)) {
-    window.lana?.log(`ERROR: Fragment Circular Reference loading ${a.href}`);
+    window.lana?.log(`Fragment Circular Reference loading ${a.href}`, {
+      tags: 'fragment',
+      severity: 'error',
+    });
     return;
   }
 
@@ -264,7 +267,10 @@ export default async function init(a) {
       return;
     }
     const message = `Could not get ${shouldFetchMepLingo ? 'mep-lingo ' : ''}fragment: ${resourcePath}.plain.html`;
-    window.lana?.log(message);
+    window.lana?.log(message, {
+      tags: 'fragment',
+      severity: 'error',
+    });
     return;
   }
 
@@ -275,7 +281,10 @@ export default async function init(a) {
 
   const sections = doc.querySelectorAll('body > div');
   if (!sections.length) {
-    window.lana?.log(`Could not make fragment: ${resourcePath}.plain.html`);
+    window.lana?.log(`Could not make fragment: ${resourcePath}.plain.html`, {
+      tags: 'fragment',
+      severity: 'error',
+    });
     return;
   }
 

libs/blocks/global-footer/global-footer.js

@@ -174,6 +174,7 @@ class Footer {
       error.tags = 'global-footer';
       error.url = url;
       error.errorType = 'e';
+      error.severity = 'error';
       lanaLog({ message: error.message, ...error });
       const { onFooterError } = getConfig();
       onFooterError?.(error);
@@ -272,6 +273,7 @@ class Footer {
         e: `${file.statusText} url: ${file.url}`,
         tags: 'global-footer',
         errorType: 'i',
+        severity: 'error',
       });
     }
     const content = await file.text();
@@ -320,7 +322,11 @@ class Footer {
     try {
       url = new URL(regionSelector.href);
     } catch (e) {
-      lanaLog({ message: `Could not create URL for region picker; href: ${regionSelector.href}`, tags: 'global-footer', errorType: 'e' });
+      lanaLog({
+        message: `Could not create URL for region picker; href: ${regionSelector.href}`,
+        tags: 'global-footer',
+        severity: 'critical',
+      });
       return this.elements.regionPicker;
     }
 
@@ -565,7 +571,12 @@ export default function init(block) {
     if (isDarkMode()) block.classList.add('feds--dark');
     return footer;
   } catch (e) {
-    lanaLog({ message: 'Could not create footer', e });
+    lanaLog({
+      message: 'Could not create footer',
+      e,
+      tags: 'global-footer',
+      severity: 'critical',
+    });
     return null;
   }
 }

libs/blocks/global-navigation/features/aside/aside.js

@@ -6,7 +6,11 @@ export default async function decorateAside({ headerElem, fedsPromoWrapper, prom
   const onError = () => {
     fedsPromoWrapper?.remove();
     headerElem?.classList.remove('has-promo');
-    lanaLog({ message: 'Gnav Promo fragment not replaced, potential CLS' });
+    lanaLog({
+      message: 'Gnav Promo fragment not replaced, potential CLS',
+      tags: 'aside',
+      severity: 'warning',
+    });
     return '';
   };
 

libs/blocks/global-navigation/features/breadcrumbs/breadcrumbs.js

@@ -86,7 +86,13 @@ const createWithBase = async (el) => {
     element.querySelector('ul')?.prepend(...base.querySelectorAll('li'));
     return createBreadcrumbs(element);
   } catch (e) {
-    lanaLog({ e, message: 'Breadcrumbs failed fetching base', tags: 'gnav-breadcrumbs', errorType: 'i' });
+    lanaLog({
+      e,
+      message: 'Breadcrumbs failed fetching base',
+      tags: 'gnav-breadcrumbs',
+      errorType: 'i',
+      severity: 'error',
+    });
     return null;
   }
 };
@@ -115,7 +121,12 @@ export default async function init(el) {
     setBreadcrumbSEO(breadcrumbsEl);
     return breadcrumbsEl;
   } catch (e) {
-    lanaLog({ e, message: 'Breadcrumbs failed rendering', tags: 'gnav-breadcrumbs', errorType: 'e' });
+    lanaLog({
+      e,
+      message: 'Breadcrumbs failed rendering',
+      tags: 'gnav-breadcrumbs',
+      severity: 'error',
+    });
     return null;
   }
 }