@playpilot/tpi 8.23.1 → 8.23.2

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@playpilot/tpi",
-	"version": "8.23.1",
+	"version": "8.23.2",
 	"type": "module",
 	"scripts": {
 		"dev": "vite dev",

package/src/lib/splitTest.ts

@@ -1,4 +1,3 @@
-import { hasConsentedTo } from './consent'
 import { TrackingEvent } from './enums/TrackingEvent'
 import { track } from './tracking'
 
@@ -15,8 +14,6 @@ export const sessionStoragePrefix = 'playpilot_split_test_'
  * The identifier is saved on the window object so that it is consistent across this session.
  */
 export function getSplitTestIdentifier({ key }: SplitTest): number {
-  if (!hasConsentedTo('tracking')) return 0
-
   const windowIdentifier = window.PlayPilotLinkInjections?.split_test_identifiers?.[key]
   if (windowIdentifier !== undefined) return windowIdentifier
 
@@ -25,7 +22,7 @@ export function getSplitTestIdentifier({ key }: SplitTest): number {
   let storedIdentifier = sessionStorage.getItem(sessionStoragePrefix + key)
 
   if (!storedIdentifier) {
-    const randomIdentifier = Math.random().toString()
+    const randomIdentifier = (Math.floor(Math.random() * 100) / 100).toString()
     sessionStorage.setItem(sessionStoragePrefix + key, randomIdentifier)
     storedIdentifier = randomIdentifier
   }
@@ -53,8 +50,6 @@ export function getSplitTestVariantName(test: SplitTest): string {
 }
 
 export function isInSplitTestVariant(test: SplitTest, variant = 1): boolean {
-  if (!hasConsentedTo('tracking')) return false
-
   return getSplitTestVariantIndex(test) === variant
 }
 

package/src/tests/lib/splitTest.test.js

@@ -2,11 +2,6 @@ import { describe, it, expect, vi, afterEach } from 'vitest'
 import { getSplitTestIdentifier, getSplitTestVariantIndex, getSplitTestVariantName, isInSplitTestVariant, sessionStoragePrefix, trackSplitTestAction, trackSplitTestView } from '$lib/splitTest'
 import { track } from '$lib/tracking'
 import { TrackingEvent } from '$lib/enums/TrackingEvent'
-import { hasConsentedTo } from '$lib/consent'
-
-vi.mock('$lib/consent', () => ({
-  hasConsentedTo: vi.fn(() => true),
-}))
 
 vi.mock('$lib/tracking', () => ({
   track: vi.fn(),
@@ -23,8 +18,6 @@ describe('$lib/splitTest', () => {
     window.PlayPilotLinkInjections = {}
 
     vi.resetAllMocks()
-
-    vi.mocked(hasConsentedTo).mockImplementation(() => true)
   })
 
   describe('getSplitTestIdentifier', () => {
@@ -60,12 +53,6 @@ describe('$lib/splitTest', () => {
       })
     })
 
-    it('Should return 0 if user did not consent', () => {
-      vi.mocked(hasConsentedTo).mockImplementation(() => false)
-
-      expect(getSplitTestIdentifier(splitTest)).toBe(0)
-    })
-
     it('Should store value in sessionStorage', () => {
       getSplitTestIdentifier(splitTest)
 
@@ -118,16 +105,6 @@ describe('$lib/splitTest', () => {
       expect(isInSplitTestVariant(multiple, 1)).toBe(false)
       expect(isInSplitTestVariant(multiple, 2)).toBe(true)
     })
-
-    it('Should return false if user did not consent', () => {
-      vi.mocked(hasConsentedTo).mockImplementation(() => false)
-
-      // @ts-ignore
-      window.PlayPilotLinkInjections.split_test_identifiers = {}
-      window.PlayPilotLinkInjections.split_test_identifiers[splitTest.key] = 0.75
-
-      expect(isInSplitTestVariant(splitTest)).toBe(false)
-    })
   })
 
   describe('trackSplitTestView', () => {