Now calling cross domain linker once

- have added a test, which appears to produce the result we want - now calling the linker code once, but with multiple domains
alphagov · Aug 30, 2019 · 5162640 · 5162640
1 parent b1224e2
commit 5162640
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/app/assets/javascripts/analytics/init.js.erb b/app/assets/javascripts/analytics/init.js.erb
@@ -40,7 +40,7 @@
 
     // Make interface public for virtual pageviews and events
     GOVUK.analytics = analytics;
-    GOVUK.analytics.addLinkedTrackerDomain(secondaryId, 'govuk', 'design-system.service.gov.uk');
+    GOVUK.analytics.addLinkedTrackerDomain(secondaryId, 'govuk', 'design-system.service.gov.uk, passport.service.gov.uk, apply-for-eu-settled-status.homeoffice.gov.uk');
   } else {
     GOVUK.analytics = dummyAnalytics
   }

diff --git a/spec/javascripts/analytics_toolkit/analytics.spec.js b/spec/javascripts/analytics_toolkit/analytics.spec.js
@@ -399,5 +399,19 @@ describe('GOVUK.Analytics', function () {
       expect(allArgs).toContain(['test.set', 'displayFeaturesTask', null])
       expect(allArgs).toContain(['test.send', 'pageview'])
     })
+
+    it('adds multiple linked domains to universal analytics', function () {
+      analytics.addLinkedTrackerDomain('1234', 'test', 'www.example.com, www.something.com')
+
+      var allArgs = window.ga.calls.allArgs()
+      expect(allArgs).toContain(['create', '1234', 'auto', {'name': 'test'}])
+      expect(allArgs).toContain(['require', 'linker'])
+      expect(allArgs).toContain(['test.require', 'linker'])
+      expect(allArgs).toContain(['linker:autoLink', ['www.example.com, www.something.com']])
+      expect(allArgs).toContain(['test.linker:autoLink', ['www.example.com, www.something.com']])
+      expect(allArgs).toContain(['test.set', 'anonymizeIp', true])
+      expect(allArgs).toContain(['test.set', 'displayFeaturesTask', null])
+      expect(allArgs).toContain(['test.send', 'pageview'])
+    })
   })
 })