fix: add support for the React Native environment.

e2e/BKTClient.spec.ts

@@ -3,7 +3,7 @@ import {
   destroyBKTClient,
   getBKTClient,
   initializeBKTClient,
-} from '../src/main.browser'
+} from './module'
 import { BKTConfig, defineBKTConfig } from '../src/BKTConfig'
 import { BKTUser, defineBKTUser } from '../src/BKTUser'
 import { FEATURE_ID_STRING, USER_ID } from './constants'
@@ -12,14 +12,14 @@ import { BKTClient, BKTClientImpl } from '../src/BKTClient'
 import { DefaultComponent } from '../src/internal/di/Component'
 import { EvaluationStorageImpl } from '../src/internal/evaluation/EvaluationStorage'
 import { evaluation1 } from '../test/mocks/evaluations'
+import { fetchLike } from './environment'
 
 suite('e2e/BKTClientTest', () => {
   let config: BKTConfig
   let user: BKTUser
 
   afterEach(() => {
     destroyBKTClient()
-    localStorage.clear()
   })
 
   suite('get string variation using user attribute when initializing', () => {
@@ -29,7 +29,7 @@ suite('e2e/BKTClientTest', () => {
         apiKey: import.meta.env.VITE_BKT_API_KEY,
         featureTag: 'javascript',
         appVersion: '1.2.3',
-        fetch: window.fetch,
+        fetch: fetchLike,
       })
 
       user = defineBKTUser({
@@ -54,7 +54,7 @@ suite('e2e/BKTClientTest', () => {
         apiKey: import.meta.env.VITE_BKT_API_KEY,
         featureTag: 'javascript',
         appVersion: '1.2.3',
-        fetch: window.fetch,
+        fetch: fetchLike,
       })
 
       user = defineBKTUser({
@@ -99,7 +99,7 @@ suite('e2e/BKTClientTest', () => {
         apiKey: import.meta.env.VITE_BKT_API_KEY,
         featureTag: 'javascript',
         appVersion: '1.2.3',
-        fetch: window.fetch,
+        fetch: fetchLike,
       })
 
       user = defineBKTUser({
@@ -201,7 +201,7 @@ suite('e2e/BKTClientTest', () => {
         apiEndpoint: import.meta.env.VITE_BKT_API_ENDPOINT,
         apiKey: import.meta.env.VITE_BKT_API_KEY,
         appVersion: '1.2.3',
-        fetch: window.fetch,
+        fetch: fetchLike,
       })
 
       user = defineBKTUser({

e2e/environment.ts

@@ -0,0 +1,19 @@
+import { FetchLike } from '../src/internal/remote/fetch'
+
+let fetchLike: FetchLike
+let isNodeEnvironment = false
+
+function setFetchProvider(fetch: FetchLike) {
+  fetchLike = fetch
+}
+
+function setIsNodeEnvironment(isNode: boolean) {
+  isNodeEnvironment = isNode
+}
+
+export {
+  setFetchProvider,
+  setIsNodeEnvironment,
+  fetchLike,
+  isNodeEnvironment,
+}

e2e/evaluations.spec.ts

@@ -3,7 +3,7 @@ import {
   destroyBKTClient,
   getBKTClient,
   initializeBKTClient,
-} from '../src/main.browser'
+} from './module'
 import { BKTConfig, defineBKTConfig } from '../src/BKTConfig'
 import { BKTUser, defineBKTUser } from '../src/BKTUser'
 import {
@@ -15,6 +15,7 @@ import {
   USER_ID,
 } from './constants'
 import './assertions'
+import { fetchLike } from './environment'
 
 suite('e2e/evaluations', () => {
   let config: BKTConfig
@@ -26,7 +27,7 @@ suite('e2e/evaluations', () => {
       apiKey: import.meta.env.VITE_BKT_API_KEY,
       featureTag: 'javascript',
       appVersion: '1.2.3',
-      fetch: window.fetch,
+      fetch: fetchLike,
     })
 
     user = defineBKTUser({
@@ -38,7 +39,6 @@ suite('e2e/evaluations', () => {
 
   afterEach(() => {
     destroyBKTClient()
-    localStorage.clear()
   })
 
   suite('stringVariation', () => {

e2e/events.spec.ts

@@ -2,10 +2,8 @@ import { suite, test, expect, beforeEach, afterEach, assert } from 'vitest'
 import {
   BKTClient,
   BKTClientImpl,
-  destroyBKTClient,
-  getBKTClient,
 } from '../src/BKTClient'
-import { initializeBKTClient } from '../src/main.browser'
+import { initializeBKTClient, getBKTClient, destroyBKTClient } from './module'
 import { BKTConfig, defineBKTConfig } from '../src/BKTConfig'
 import { BKTUser, defineBKTUser } from '../src/BKTUser'
 import { DefaultComponent } from '../src/internal/di/Component'
@@ -23,6 +21,7 @@ import './assertions'
 import { EventType } from '../src/internal/model/Event'
 import { ForbiddenException, TimeoutException } from '../src/BKTExceptions'
 import { ApiId, MetricsEventType } from '../src/internal/model/MetricsEventData'
+import { fetchLike, isNodeEnvironment } from './environment'
 
 function getDefaultComponent(client: BKTClient): DefaultComponent {
   return (client as BKTClientImpl).component as DefaultComponent
@@ -38,7 +37,7 @@ suite('e2e/events', () => {
       apiKey: import.meta.env.VITE_BKT_API_KEY,
       featureTag: 'javascript',
       appVersion: '1.2.3',
-      fetch: window.fetch,
+      fetch: fetchLike,
       // DO NOT remove this line
       // Because the tests are asynchronous and share the same local storage,
       // It might fail randomly, having more or fewer events in the storage when checking the test.
@@ -55,7 +54,6 @@ suite('e2e/events', () => {
 
   afterEach(() => {
     destroyBKTClient()
-    localStorage.clear()
   })
 
   test('goal event', async () => {
@@ -174,14 +172,13 @@ suite('e2e/events', () => {
 
     test('Using a random string in the api key setting should throw Forbidden', async () => {
       destroyBKTClient()
-      localStorage.clear()
 
       config = defineBKTConfig({
         apiEndpoint: import.meta.env.VITE_BKT_API_ENDPOINT,
         apiKey: 'some-random-string',
         featureTag: 'javascript',
         appVersion: '1.2.3',
-        fetch: window.fetch,
+        fetch: fetchLike,
       })
 
       user = defineBKTUser({
@@ -224,7 +221,7 @@ suite('e2e/events', () => {
         apiKey: import.meta.env.VITE_BKT_API_KEY,
         featureTag: 'javascript',
         appVersion: '1.2.3',
-        fetch: window.fetch,
+        fetch: fetchLike,
       })
 
       await initializeBKTClient(config, user)
@@ -234,37 +231,43 @@ suite('e2e/events', () => {
       const component2 = getDefaultComponent(client)
 
       const events3 = component2.dataModule.eventStorage().getAll()
-      // 2 events - latency and response size
-      expect(events3).toHaveLength(2)
-      // ForbiddenError should not exist
-      expect(
-        events.some((e) => {
-          return (
-            e.type === EventType.METRICS &&
-            e.event.event['@type'] === MetricsEventType.ForbiddenError &&
-            e.event.event.apiId === ApiId.GET_EVALUATIONS
-          )
-        }),
-      ).toBe(false)
 
-      await client2.flush()
-
-      const events4 = component2.dataModule.eventStorage().getAll()
-
-      // error from /register_events does not get stored
-      expect(events4).toHaveLength(0)
+      if (isNodeEnvironment) {
+        // on the node environment, no events should be stored  after destroying the client
+        // because it's using in-memory storage
+        expect(events3).toHaveLength(0)
+      } else {
+        // on the browser environment, we should have 2 events - latency and response size
+        expect(events3).toHaveLength(2)
+        // ForbiddenError should not exist
+        expect(
+          events.some((e) => {
+            return (
+              e.type === EventType.METRICS &&
+              e.event.event['@type'] === MetricsEventType.ForbiddenError &&
+              e.event.event.apiId === ApiId.GET_EVALUATIONS
+            )
+          }),
+        ).toBe(false)
+
+        await client2.flush()
+
+        const events4 = component2.dataModule.eventStorage().getAll()
+
+        // error from /register_events does not get stored
+        expect(events4).toHaveLength(0)
+      }
     })
 
     test('Using a random string in the featureTag setting should not affect api request', async () => {
       destroyBKTClient()
-      localStorage.clear()
 
       config = defineBKTConfig({
         apiEndpoint: import.meta.env.VITE_BKT_API_ENDPOINT,
         apiKey: import.meta.env.VITE_BKT_API_KEY,
         featureTag: 'some-random-feature-tag',
         appVersion: '1.2.3',
-        fetch: window.fetch,
+        fetch: fetchLike,
       })
 
       user = defineBKTUser({
@@ -276,16 +279,14 @@ suite('e2e/events', () => {
 
     test('Timeout', async () => {
       // setting a very low value for the timeout
-
       destroyBKTClient()
-      localStorage.clear()
 
       config = defineBKTConfig({
         apiEndpoint: import.meta.env.VITE_BKT_API_ENDPOINT,
         apiKey: import.meta.env.VITE_BKT_API_KEY,
         featureTag: 'javascript',
         appVersion: '1.2.3',
-        fetch: window.fetch,
+        fetch: fetchLike,
       })
 
       user = defineBKTUser({

e2e/module.browser.ts

@@ -0,0 +1,5 @@
+export {
+  destroyBKTClient,
+  getBKTClient,
+  initializeBKTClient,
+} from '../src/main.browser'

e2e/module.node.ts

@@ -0,0 +1,5 @@
+export {
+  destroyBKTClient,
+  getBKTClient,
+  initializeBKTClient,
+} from '../src/main'

e2e/module.ts

@@ -0,0 +1,5 @@
+export {
+  destroyBKTClient,
+  getBKTClient,
+  initializeBKTClient,
+} from '../src/main'

e2e/setup.browser.ts

@@ -0,0 +1,9 @@
+
+import { afterEach } from 'vitest'
+import { setFetchProvider } from './environment'
+
+setFetchProvider(window.fetch)
+
+afterEach(() => {
+  localStorage.clear()
+})

e2e/setup.node.ts

@@ -0,0 +1,5 @@
+import { setFetchProvider, setIsNodeEnvironment } from './environment'
+
+setFetchProvider(fetch)
+
+setIsNodeEnvironment(true)

package.json

@@ -14,13 +14,19 @@
   "module": "./dist/main.mjs",
   "browser": "./dist/main.browser.mjs",
   "types": "./dist/main.d.ts",
+  "react-native": "./dist/main.native.cjs",
   "exports": {
     ".": {
       "node": {
         "import": "./dist/main.mjs",
         "require": "./dist/main.cjs",
         "types": "./dist/main.d.ts"
       },
+      "react-native": {
+        "import": "./dist/main.native.mjs",
+        "require": "./dist/main.native.cjs",
+        "types": "./dist/main.d.ts"
+      },
       "default": {
         "import": "./dist/main.browser.mjs",
         "require": "./dist/main.browser.cjs",
@@ -35,7 +41,9 @@
     "test": "pnpm test:browser --run ; pnpm test:node --run",
     "test:browser": "vitest --config ./vitest-browser.config.ts --dir test",
     "test:node": "vitest --config ./vitest-node.config.ts --dir test",
-    "test:e2e": "vitest --config ./vitest-e2e.config.ts --dir e2e",
+    "test:e2e": "pnpm test:e2e:browser --run ; pnpm test:e2e:node --run",
+    "test:e2e:browser": "cp -f ./e2e/module.browser.ts ./e2e/module.ts ; vitest --config ./vitest-e2e.config.ts --dir e2e",
+    "test:e2e:node": "cp -f ./e2e/module.node.ts ./e2e/module.ts ; vitest --config ./vitest-e2e-node.config.ts --dir e2e",
     "lint": "eslint .",
     "lint:fix": "eslint --fix .",
     "example:serve": "pnpm exec unbuild && pnpm --filter example serve"

src/BKTConfig.ts

@@ -1,5 +1,6 @@
 import { IllegalArgumentException } from './BKTExceptions'
 import { BKTStorage, createBKTStorage } from './BKTStorage'
+import { IdGenerator } from './internal/IdGenerator'
 import { FetchLike } from './internal/remote/fetch'
 import { SDK_VERSION } from './internal/version'
 
@@ -30,6 +31,7 @@ interface RawBKTConfig {
   userAgent?: string
   fetch?: FetchLike
   storageFactory?: <T>(key: string) => BKTStorage<T>
+  idGenerator?: IdGenerator
 }
 
 export interface BKTConfig extends RawBKTConfig {

src/internal/di/PlatformModule.ts

@@ -3,3 +3,17 @@ import { IdGenerator } from '../IdGenerator'
 export interface PlatformModule {
   idGenerator(): IdGenerator
 }
+
+// BasePlatformModule serves as a base implementation of the PlatformModule interface.
+// It relies on the BKTConfig to inject an instance of IdGenerator, which must be set before use.
+export class BasePlatformModule implements PlatformModule {
+  protected _idGenerator: IdGenerator
+
+  constructor(params: { idGenerator: IdGenerator }) {
+    this._idGenerator = params.idGenerator
+  }
+
+  idGenerator() {
+    return this._idGenerator
+  }
+}