Upgrade JS-commons. Updated type definitions & tests

Author: EmilianoSanchez

CHANGES.txt

@@ -1,8 +1,10 @@
-0.16.0 (September XX, 2024)
- - Added `factory.destroy()` method, which invokes the `destroy` method on all SDK clients created by the factory.
- - Updated @splitsoftware/splitio-commons package to version 1.18.0 that includes minor updates:
-    - Added support for targeting rules based on large segments for browsers.
-    - Updated some transitive dependencies for vulnerability fixes.
+1.0.0 (October XX, 2024)
+ - Added support for targeting rules based on large segments for browsers.
+ - Added `factory.destroy()` method, which invokes the `destroy` method of all clients created by the factory.
+ - Updated @splitsoftware/splitio-commons package to version 2.0.0 that includes major updates and updated some transitive dependencies for vulnerability fixes.
+ - BREAKING CHANGES:
+    - Removed internal ponyfills for the `Map` and `Set` global objects, dropping support for IE and other outdated browsers. The SDK now requires the runtime environment to support these features natively or provide a polyfill.
+    - Removed the deprecated `GoogleAnalyticsToSplit` and `SplitToGoogleAnalytics` pluggable integration modules, along with the related interfaces in the TypeScript definitions.
 
 0.15.0 (September 13, 2024)
  - Updated @splitsoftware/splitio-commons package to version 1.17.0 that includes minor updates:
@@ -58,7 +60,7 @@
  - Updated some transitive dependencies for vulnerability fixes.
  - Updated @splitsoftware/splitio-commons package to version 1.8.3 that includes:
     - Updated SDK_READY_TIMED_OUT event to be emitted immediately when a connection error occurs using pluggable storage (i.e., when the wrapper `connect` promise is rejected) in consumer and partial consumer modes.
-    - Bugfixing - The `destroy` method has been updated to immediately flag the SDK client as destroyed, to prevent unexpected behaviours when `getTreatment` and `track` methods are called synchronously after `destroy` method is called.
+    - Bugfixing - The `destroy` method has been updated to immediately flag the SDK client as destroyed, to prevent unexpected behaviors when `getTreatment` and `track` methods are called synchronously after `destroy` method is called.
 
 0.9.5 (May 15, 2023)
  - Updated @splitsoftware/splitio-commons package to version 1.8.2 that includes minor improvements.

karma/e2e.gaIntegration.karma.conf.js

@@ -1,6 +1,6 @@
 {
   "name": "@splitsoftware/splitio-browserjs",
-  "version": "0.15.1-rc.0",
+  "version": "1.0.0-rc.0",
   "description": "Split SDK for JavaScript on Browser",
   "main": "cjs/index.js",
   "module": "esm/index.js",
@@ -14,8 +14,7 @@
     "esm",
     "src",
     "types",
-    "full",
-    "scripts/ga-to-split-autorequire.js"
+    "full"
   ],
   "scripts": {
     "check": "npm run check:lint && npm run check:types && npm run check:version",
@@ -27,7 +26,6 @@
     "build:cjs": "rimraf cjs && tsc -outDir cjs -m CommonJS && ./scripts/build_cjs_replace_imports.sh",
     "build:umd-visualizer": "rimraf umd && rollup --config rollup.visualizer.config.js",
     "build:umd": "rimraf umd && rollup --config rollup.ci.config.js --branch=$BUILD_BRANCH",
-    "build:ga-to-split-autorequire": "terser ./node_modules/@splitsoftware/splitio-commons/src/integrations/ga/autoRequire.js --mangle --output ./scripts/ga-to-split-autorequire.js",
     "test": "npm run test:unit && npm run test:e2e",
     "test:unit": "jest",
     "test:e2e": "npm run test:e2e-logger && npm run test:e2e-offline && npm run test:e2e-online && npm run test:e2e-destroy && npm run test:e2e-errorCatching && npm run test:e2e-push && npm run test:e2e-consumer",
@@ -37,7 +35,6 @@
     "test:e2e-destroy": "karma start karma/e2e.destroy.karma.conf.js",
     "test:e2e-errorCatching": "karma start karma/e2e.errorCatching.karma.conf.js",
     "test:e2e-push": "karma start karma/e2e.push.karma.conf.js",
-    "test:e2e-gaIntegration": "karma start karma/e2e.gaIntegration.karma.conf.js",
     "test:e2e-consumer": "karma start karma/e2e.consumer.karma.conf.js",
     "pretest-ts-decls": "npm run build:esm && npm run build:cjs && npm link",
     "test-ts-decls": "./scripts/ts-tests.sh",
@@ -64,8 +61,7 @@
   "bugs": "https://github.com/splitio/javascript-browser-client/issues",
   "homepage": "https://github.com/splitio/javascript-browser-client#readme",
   "dependencies": {
-    "@splitsoftware/splitio-commons": "1.17.1-rc.1",
-    "@types/google.analytics": "0.0.40",
+    "@splitsoftware/splitio-commons": "2.0.0-rc.0",
     "tslib": "^2.3.1",
     "unfetch": "^4.2.0"
   },

package-lock.json

@@ -72,7 +72,7 @@ function mockSubmittersRequests(fetchMock, assert, impressionFeature, eventTypeI
 
 export default function userConsent(fetchMock, t) {
 
-  // Validate trackers, submitters and browser listener behaviour on different consent status transitions
+  // Validate trackers, submitters and browser listener behavior on different consent status transitions
   t.test(async (assert) => {
     const sendBeaconSpy = sinon.spy(window.navigator, 'sendBeacon');
     let expectedTrackedImpressions = 0;
@@ -97,7 +97,7 @@ export default function userConsent(fetchMock, t) {
       ], ['on', 'on', 'on', 'on', 'on', 'on', 'on', 'on'], 'evaluating on SDK ready');
       if (isTracking) expectedTrackedImpressions += 8;
 
-      // Trigger pagehide event to validate browser listener behaviour
+      // Trigger pagehide event to validate browser listener behavior
       // Beacon API is used only if user consent is GRANTED
       triggerPagehideEvent();
       if (factory.UserConsent.getStatus() === factory.UserConsent.Status.GRANTED) {
@@ -137,9 +137,9 @@ export default function userConsent(fetchMock, t) {
     assert.equal(trackedImpressions.length, expectedTrackedImpressions, 'Tracked impressions are the expected');
     sendBeaconSpy.restore();
     assert.end();
-  }, 'Validate trackers, submitters and browser listener behaviour on different consent status transitions');
+  }, 'Validate trackers, submitters and browser listener behavior on different consent status transitions');
 
-  // Validate submitter's behaviour with full queues and with events first push window
+  // Validate submitter's behavior with full queues and with events first push window
   t.test(async (assert) => {
     const config = {
       ...baseConfig,
@@ -182,6 +182,6 @@ export default function userConsent(fetchMock, t) {
     await client.destroy();
 
     assert.end();
-  }, 'Validate submitter\'s behaviour with full queues and with events first push window');
+  }, 'Validate submitter\'s behavior with full queues and with events first push window');
 
 }