@playpilot/tpi 6.4.2 → 6.4.3

package/package.json

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

package/src/lib/enums/SplitTest.ts

@@ -4,9 +4,4 @@ export const SplitTest = {
     numberOfVariants: 2,
     variantNames: ['Separated', 'Inline'] as string[],
   },
-  PostersScrollReveal: {
-    key: 'posters_scroll_reveal',
-    numberOfVariants: 2,
-    variantNames: ['Default', 'With Posters'] as string[],
-  },
 } as const

package/src/routes/+page.svelte

@@ -234,7 +234,6 @@
 
 {#key linkInjections}
   <UserJourney />
-  <PostersScrollReveal />
 {/key}
 
 <Consent onchange={afterConsent} />

package/src/routes/components/PostersScrollReveal.svelte

@@ -1,7 +1,5 @@
 <script lang="ts">
   import { mobileBreakpoint } from '$lib/constants'
-  import { SplitTest } from '$lib/enums/SplitTest'
-  import { isInSplitTestVariant, trackSplitTestAction, trackSplitTestView } from '$lib/splitTest'
   import { onMount } from 'svelte'
 
   const linksWithPosters = Array.from(document.querySelectorAll<HTMLElement>('[data-playpilot-poster-url]'))
@@ -18,27 +16,11 @@
   onMount(() => {
     if (!isMobile) return
 
-    trackSplitTestView(SplitTest.PostersScrollReveal)
-
     return destroyAllPosters
   })
 
-  $effect(() => {
-    if (!isMobile) return
-    if (!linksWithPosters.length) return
-
-    const onclick = () => trackSplitTestAction(SplitTest.PostersScrollReveal, 'click')
-
-    for(const link of linksWithPosters) link.addEventListener('click', onclick)
-
-    return () => {
-      for(const link of linksWithPosters) link.removeEventListener('click', onclick)
-    }
-  })
-
   function onscroll(): void {
     if (!isMobile) return
-    if (!isInSplitTestVariant(SplitTest.PostersScrollReveal)) return
 
     setScrollTimeout()
 

package/src/tests/routes/components/PostersScrollReveal.test.js

@@ -1,14 +1,7 @@
-import { fireEvent, render } from '@testing-library/svelte'
+import { render } from '@testing-library/svelte'
 import { describe, it, expect, vi, beforeEach } from 'vitest'
 import PostersScrollReveal from '../../../routes/components/PostersScrollReveal.svelte'
 import { mobileBreakpoint } from '$lib/constants'
-import { isInSplitTestVariant, trackSplitTestAction } from '$lib/splitTest'
-
-vi.mock('$lib/splitTest', () => ({
-  trackSplitTestView: vi.fn(),
-  trackSplitTestAction: vi.fn(),
-  isInSplitTestVariant: vi.fn(() => true),
-}))
 
 function createPosterLink(top = 0, url = 'poster.jpg') {
   const link = document.createElement('a')
@@ -37,8 +30,6 @@ describe('FloatingPoster', () => {
 
     vi.useFakeTimers()
     vi.resetAllMocks()
-
-    vi.mocked(isInSplitTestVariant).mockReturnValue(true)
   })
 
   it('Should create poster when link is in viewport', async () => {
@@ -147,16 +138,4 @@ describe('FloatingPoster', () => {
     window.dispatchEvent(new Event('scroll'))
     expect(document.querySelector('[data-playpilot-poster]')).toBeTruthy()
   })
-
-  it('Should fire split test tracking on click', async () => {
-    createPosterLink(500)
-
-    render(PostersScrollReveal)
-
-    window.dispatchEvent(new Event('scroll'))
-
-    await fireEvent.click(/** @type {HTMLElement} */ (document.querySelector('[data-playpilot-poster]')))
-
-    expect(trackSplitTestAction).toHaveBeenCalled()
-  })
 })