Trigger Sentry when tracking called with incorrect event name

app/assets/javascripts/tracking/dispatch_snowplow_event.js

 import * as Sentry from '~/sentry/sentry_browser_wrapper';
 import getStandardContext from './get_standard_context';
+import { validateEvent } from './utils';
 
 export function dispatchSnowplowEvent(
   category = document.body.dataset.page,
@@ -11,6 +12,8 @@ export function dispatchSnowplowEvent(
     throw new Error('Tracking: no category provided for tracking.');
   }
 
+  validateEvent(action);
+
   const { label, property, extra = {} } = data;
   let { value } = data;
 

app/assets/javascripts/tracking/utils.js

 import { omitBy, isUndefined } from 'lodash';
 import { TRACKING_CONTEXT_SCHEMA } from '~/experimentation/constants';
 import { getExperimentData } from '~/experimentation/utils';
+import * as Sentry from '~/sentry/sentry_browser_wrapper';
 import {
   ACTION_ATTR_SELECTOR,
   LOAD_ACTION_ATTR_SELECTOR,
@@ -189,6 +190,12 @@ export const validateAdditionalProperties = (additionalProperties) => {
   });
 };
 
+export const validateEvent = (event) => {
+  if (event && /\s/.test(event)) {
+    Sentry.captureException(new Error(`Event name should not contain whitespace: ${event}`));
+  }
+};
+
 function filterProperties(additionalProperties, predicate) {
   return Object.keys(additionalProperties).reduce((acc, key) => {
     if (predicate(key)) {

spec/frontend/tracking/utils_spec.js

@@ -9,7 +9,9 @@ import {
   validateAdditionalProperties,
   getCustomAdditionalProperties,
   getBaseAdditionalProperties,
+  validateEvent,
 } from '~/tracking/utils';
+import * as Sentry from '~/sentry/sentry_browser_wrapper';
 import { TRACKING_CONTEXT_SCHEMA } from '~/experimentation/constants';
 import { REFERRER_TTL, URLS_CACHE_STORAGE_KEY } from '~/tracking/constants';
 import { TEST_HOST } from 'helpers/test_constants';
@@ -220,6 +222,36 @@ describe('~/tracking/utils', () => {
     });
   });
 
+  describe('validateEvent', () => {
+    let sentrySpy;
+
+    beforeEach(() => {
+      sentrySpy = jest.spyOn(Sentry, 'captureException');
+    });
+
+    afterEach(() => {
+      sentrySpy.mockRestore();
+    });
+
+    it('calls Sentry for event names with whitespace', () => {
+      validateEvent('event name');
+
+      expect(sentrySpy).toHaveBeenCalled();
+    });
+
+    it('does not call Sentry for event names eqaual to nil', () => {
+      validateEvent(null);
+
+      expect(sentrySpy).not.toHaveBeenCalled();
+    });
+
+    it('does not call Sentry for event names without whitespace', () => {
+      validateEvent('event-name');
+
+      expect(sentrySpy).not.toHaveBeenCalled();
+    });
+  });
+
   describe('getCustomAdditionalProperties', () => {
     it('returns only custom properties', () => {
       const additionalProperties = {