Add test for video query params (#490)

Author: Nir Maoz

__TESTS__/unit/analytics/analytics.browser.test.ts

@@ -2,22 +2,64 @@
  * @jest-environment jsdom
  */
 
-
-
 import {createNewImageWithAnalytics} from "./testUtils/createNewImageWithAnalytics";
-
+import {createNewVideoWithAnalytics} from "./testUtils/createNewVideoWithAnalytics";
 
 describe('Add analytics to a URL from the browser', () => {
-  it('Uses default techVersion 0.0.0 when in browser', () => {
+  it('Uses default techVersion 0.0.0 when in browser for image', () => {
     const cldImage = createNewImageWithAnalytics('sample');
+    const url = cldImage.toURL({
+      trackedAnalytics: {
+        sdkSemver: '1.0.0'
+      }
+    });
 
     // ATAAB{NODE_VERSION}0
     // ATAAB{AA}0 -> we expect nodeVersion to be 0.0.0 in browser (Since it's missing)
     // expect ATAABAA0
-    expect(cldImage.toURL({
+    expect(url).toContain('sample?_a=ATAABAA0'); // we shouldn't have a query param at all
+  });
+
+  it('Uses default techVersion 0.0.0 when in browser for image with file extension', () => {
+    const cldImage = createNewImageWithAnalytics('sample.jpg');
+    const url = cldImage.toURL({
       trackedAnalytics: {
         sdkSemver: '1.0.0'
       }
-    })).toContain('ATAABAA0'); // we shouldn't have a query param at all
+    });
+
+    // ATAAB{NODE_VERSION}0
+    // ATAAB{AA}0 -> we expect nodeVersion to be 0.0.0 in browser (Since it's missing)
+    // expect ATAABAA0
+    expect(url).toContain('sample.jpg?_a=ATAABAA0'); // we shouldn't have a query param at all
+  });
+
+  it('Uses default techVersion 0.0.0 when in browser for video', () => {
+    const cldImage = createNewVideoWithAnalytics('sample');
+    const url = cldImage.toURL({
+      trackedAnalytics: {
+        sdkSemver: '1.0.0'
+      }
+    });
+
+    // ATAAB{NODE_VERSION}0
+    // ATAAB{AA}0 -> we expect nodeVersion to be 0.0.0 in browser (Since it's missing)
+    // expect ATAABAA0
+    expect(url).toContain('sample?_a=ATAABAA0'); // we shouldn't have a query param at all
+  });
+
+
+  it('Uses default techVersion 0.0.0 when in browser for video with file extension', () => {
+    const cldImage = createNewVideoWithAnalytics('sample.webm');
+    const url = cldImage.toURL({
+      trackedAnalytics: {
+        sdkSemver: '1.0.0'
+      }
+    });
+
+    // ATAAB{NODE_VERSION}0
+    // ATAAB{AA}0 -> we expect nodeVersion to be 0.0.0 in browser (Since it's missing)
+    // expect ATAABAA0
+    expect(url).toContain('sample.webm?_a=ATAABAA0'); // we shouldn't have a query param at all
   });
 });

__TESTS__/unit/analytics/testUtils/createNewVideoWithAnalytics.ts

@@ -0,0 +1,11 @@
+import ICloudConfig from "../../../../src/config/interfaces/Config/ICloudConfig";
+import IURLConfig from "../../../../src/config/interfaces/Config/IURLConfig";
+import {CloudinaryVideo} from "../../../../src";
+import {createNewVideo} from "../../../TestUtils/createCloudinaryVideo";
+
+/**
+ * Create a new CloudinaryImage with analytics turned on by default
+ */
+export function createNewVideoWithAnalytics(publicID: string, cloudConfig?: ICloudConfig, urlConfig?: IURLConfig): CloudinaryVideo {
+  return createNewVideo(publicID, {cloudName: 'demo'}, {analytics: true, ...urlConfig});
+}