MWPW-138026 Add tags to lana logs

adobecom · Dec 7, 2023 · 32a71d6 · 32a71d6
1 parent 3833c81
commit 32a71d6
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 10 deletions.
diff --git a/blocks/faas-decode/faas-decode.js b/blocks/faas-decode/faas-decode.js
@@ -6,8 +6,7 @@ export default async function init(el) {
   const resp = await fetch(url);
 
   if (!resp?.ok) {
-    // eslint-disable-next-line no-console
-    console.log(`Error fetching data from url: ${url}`);
+    window.lana?.log(`Error fetching data from url: ${url}`, { tags: 'info, faas-decode' });
     return;
   }
 

diff --git a/blocks/tree-view/tree-view.js b/blocks/tree-view/tree-view.js
@@ -11,8 +11,7 @@ export const isCurrentPage = (link) => {
 
     if (isBacomHost && url.pathname.replace('.html', '') === currentPath) return true;
   } catch (e) {
-    // eslint-disable-next-line no-console
-    console.log('Tree View error:', e);
+    window.lana?.log(`Tree View error:${e.message}`, { tags: 'info, tree-view' });
   }
 
   return false;

diff --git a/scripts/scripts.js b/scripts/scripts.js
@@ -189,6 +189,6 @@ const miloLibs = setLibs(LIBS);
     }
   }
   setConfig({ ...CONFIG, miloLibs });
-  loadLana({ clientId: 'bacom' });
+  loadLana({ clientId: 'bacom', tags: 'info' });
   await loadArea();
 }());
diff --git a/test/blocks/faas-decode/faas-decode.test.js b/test/blocks/faas-decode/faas-decode.test.js
@@ -4,9 +4,11 @@ import init from '../../../blocks/faas-decode/faas-decode.js';
 import { setLibs } from '../../../scripts/utils.js';
 import waitForElement from '../../helpers/waitForElement.js';
 
+window.lana = { log: () => {} };
+
 describe('FaaS Decode', () => {
   before(() => {
-    sinon.spy(console, 'log');
+    sinon.stub(window.lana, 'log');
     setLibs('/libs');
   });
 
@@ -18,7 +20,7 @@ describe('FaaS Decode', () => {
     document.body.innerHTML = '<div class="faas-decode"><div><div><a href="/404"></a></div></div></div>';
     const el = document.querySelector('.faas-decode');
     await init(el);
-    expect(console.log.args[0][0]).to.include('Error fetching data from url:');
+    expect(window.lana.log.args[0][0]).to.include('Error fetching data from url:');
   });
 
   it('creates a table', async () => {

diff --git a/test/blocks/tree-view/tree-view.test.js b/test/blocks/tree-view/tree-view.test.js
@@ -5,6 +5,7 @@ import { setLibs } from '../../../scripts/utils.js';
 
 const { default: init, isCurrentPage } = await import('../../../blocks/tree-view/tree-view.js');
 
+window.lana = { log: () => {} };
 setLibs('libs');
 
 describe('Tree View', () => {
@@ -38,10 +39,10 @@ describe('Tree View', () => {
   });
 
   it('isCurrentPage catches error', () => {
-    sinon.spy(console, 'log');
+    sinon.stub(window.lana, 'log');
     isCurrentPage('/relative-link');
-    expect(console.log.args[0][0]).to.equal('Tree View error:');
-    console.log.restore();
+    expect(window.lana.log.args[0][0]).to.contain('Tree View error:');
+    window.lana.log.restore();
   });
 
   describe('accordion', () => {