@guardian/ophan-tracker-js 2.2.0 → 2.2.2

package/NPM-dist/attention.js

@@ -1,5 +1,5 @@
 // Generated by CoffeeScript 2.7.0
-import transmit from './transmit';
+import transmit from './transmit.js';
 import components from './components.js';
 // The upworthy definition of attention minutes, from
 // http://upworthy.github.io/2014/06/implementing-attention-minutes-part-1/ :

package/NPM-dist/campaign.js

@@ -1,5 +1,5 @@
 // Generated by CoffeeScript 2.7.0
-import transmit from './transmit';
+import transmit from './transmit.js';
 import perf from './perf.js';
 const start = perf?.timing?.domComplete ?? performance.now();
 // DFP ads

package/NPM-dist/click-path-capture.js

@@ -1,5 +1,5 @@
 import ophan from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 /**
  * Finds the closest ancestor anchor (`<a>`) element of the given element.
  *
@@ -43,6 +43,10 @@ if (typeof document.addEventListener === 'function') {
     document.addEventListener('click', function (e) {
         let anchorTarget, info;
         anchorTarget = validAncestorAnchorElement(e.target);
+        if (!anchorTarget) {
+            // If no anchor element was clicked, exit early.
+            return;
+        }
         info = {
             from: [location.protocol, '//', location.host, location.pathname].join(''),
             to: anchorTarget ? anchorTarget.href : void 0,

package/NPM-dist/contribution.js

@@ -1,6 +1,6 @@
 // Generated by CoffeeScript 2.7.0
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 import attention from './attention.js';
 import visibility from './visibility.js';
 import './click-path-capture.js';

package/NPM-dist/core.js

@@ -1,5 +1,5 @@
 let init, sendInitialEvent, storeDataToSendOnNextEvent, hasProp = {}.hasOwnProperty;
-import transmit from './transmit';
+import transmit from './transmit.js';
 import visibility from './visibility.js';
 import adblockDetection from './adblock-detection.js';
 const VERSION = 17;

package/NPM-dist/embed.js

@@ -1,6 +1,6 @@
 // Generated by CoffeeScript 2.7.0
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 core.init('embed');
 export default {
     record: transmit.sendMore,

package/NPM-dist/fb-instant.js

@@ -1,6 +1,6 @@
 // Generated by CoffeeScript 2.7.0
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 core.init('facebook-instant-article');
 export default {
     viewId: transmit.viewId,

package/NPM-dist/holidays.js

@@ -1,6 +1,6 @@
 // Generated by CoffeeScript 2.7.0
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 core.init('holidays');
 export default {
     record: transmit.sendMore,

package/NPM-dist/interactive.js

@@ -1,5 +1,5 @@
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 import attention from './attention.js';
 import './click-path-capture.js';
 if (window.guardian?.ophan) {

package/NPM-dist/jobs-courses.js

@@ -1,6 +1,6 @@
 // Generated by CoffeeScript 2.7.0
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 core.init('jobs-courses');
 export default {
     record: transmit.sendMore,

package/NPM-dist/jobs.js

@@ -1,6 +1,6 @@
 // Generated by CoffeeScript 2.7.0
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 core.init('jobs');
 export default {
     record: transmit.sendMore,

package/NPM-dist/manage-my-account.js

@@ -1,6 +1,6 @@
 // Generated by CoffeeScript 2.7.0
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 import attention from './attention.js';
 import visibility from './visibility.js';
 import './click-path-capture.js';

package/NPM-dist/membership.js

@@ -1,6 +1,6 @@
 // Generated by CoffeeScript 2.7.0
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 import attention from './attention.js';
 import visibility from './visibility.js';
 import './click-path-capture.js';

package/NPM-dist/ng.js

@@ -1,5 +1,5 @@
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 import attention from './attention.js';
 import visibility from './visibility.js';
 import privatebrowsing from './privatebrowsing.js';

package/NPM-dist/privatebrowsing.js

@@ -1,4 +1,4 @@
-import transmit from './transmit';
+import transmit from './transmit.js';
 /*
     TO-DO:
     This function could probably be simplified with an async function.

package/NPM-dist/r2.js

@@ -1,6 +1,6 @@
 // Generated by CoffeeScript 2.7.0
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 import attention from './attention.js';
 import visibility from './visibility.js';
 import './click-path-capture.js';

package/NPM-dist/smart-news.js

@@ -1,6 +1,6 @@
 // Generated by CoffeeScript 2.7.0
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 import attention from './attention.js';
 import visibility from './visibility.js';
 core.init('smart-news');

package/NPM-dist/support.js

@@ -1,5 +1,5 @@
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 import attention from './attention.js';
 import visibility from './visibility.js';
 import './click-path-capture.js';

package/NPM-dist/tests/click-path-capture.test.js

@@ -1,12 +1,17 @@
 import clickPathCapture, { validAncestorAnchorElement, getContainingComponent } from '../click-path-capture.js';
+import transmit from '../transmit.js';
+import ophan from '../core.js';
 const { getDataLinkNames } = clickPathCapture;
 describe('click-path-capture.js', () => {
     beforeAll(() => {
         document.body.innerHTML = `
             <div data-component="component1">
-                <a href="#" data-link-name="link1">
-                    <span id="testElement1"></span>
-                </a>
+                <div id="testElement4">
+                    <a href="#" data-link-name="link1" id="link1">
+                        <span id="testElement1"></span>
+                    </a>
+                    <span id="nonLinkElement">Non-link element</span>
+                </div>
             </div>
             <div id="testElement2" data-component="component2"></div>
         `;
@@ -41,4 +46,26 @@ describe('click-path-capture.js', () => {
             expect(getDataLinkNames(element)).toEqual([]);
         });
     });
+    describe('click event handling', () => {
+        let transmitSendMoreSpy, ophanStoreDataToSendOnNextEventSpy;
+        beforeEach(() => {
+            transmitSendMoreSpy = jest.spyOn(transmit, 'sendMore').mockImplementation(() => { });
+            ophanStoreDataToSendOnNextEventSpy = jest.spyOn(ophan, 'storeDataToSendOnNextEvent').mockImplementation(() => { });
+        });
+        afterEach(() => {
+            jest.clearAllMocks();
+        });
+        test('should send request and store data when a link is clicked', () => {
+            const element = document.getElementById('testElement1');
+            element.click();
+            expect(transmitSendMoreSpy).toHaveBeenCalled();
+            expect(ophanStoreDataToSendOnNextEventSpy).toHaveBeenCalled();
+        });
+        test('should not send request when a non-link element within a container is clicked', () => {
+            const element = document.getElementById('testElement4');
+            element.click();
+            expect(transmitSendMoreSpy).not.toHaveBeenCalled();
+            expect(ophanStoreDataToSendOnNextEventSpy).not.toHaveBeenCalled();
+        });
+    });
 });

package/NPM-dist/witness.js

@@ -1,6 +1,6 @@
 // Generated by CoffeeScript 2.7.0
 import core from './core.js';
-import transmit from './transmit';
+import transmit from './transmit.js';
 core.init('witness');
 export default {
     record: transmit.sendMore,

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@guardian/ophan-tracker-js",
-	"version": "2.2.0",
+	"version": "2.2.2",
 	"type": "module",
 	"engines": {
 		"node": ">=16"
@@ -31,6 +31,7 @@
 	"scripts": {
 		"build-for-cdn": "NODE_ENV=production npx rollup -c",
 		"build-for-npm": "tsc",
+		"pretest": "tsc",
 		"test": "jest",
 		"format": "npx prettier --write ."
 	},