@playpilot/tpi 3.3.3 → 3.4.0-beta.1

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

@@ -3,7 +3,7 @@ import { describe, expect, it, vi, beforeEach, afterEach } from 'vitest'
 
 import { injectLinksInDocument, clearLinkInjections, clearLinkInjection, getLinkInjectionElements, insertAfterArticlePlaylinks, getLinkInjectionsParentElement, isAvailableAsManualInjection, filterRemovedAndInactiveInjections, isEquivalentInjection, filterInvalidInTextInjections, filterInvalidAfterArticleInjections, isValidInjection, isValidPlaylinkType } from '$lib/linkInjection'
 import { mount, unmount } from 'svelte'
-import { fakeFetch, generateInjection } from '../helpers'
+import { generateInjection } from '../helpers'
 
 vi.mock('svelte', () => ({
   mount: vi.fn(),
@@ -23,11 +23,7 @@ function mockMatchMedia(matches = false) {
 describe('linkInjection.js', () => {
   beforeEach(() => {
     vi.resetAllMocks()
-    clearLinkInjections()
-
     mockMatchMedia()
-    fakeFetch()
-    vi.mocked(mount).mockReturnValueOnce({})
 
     // @ts-ignore
     window.PlayPilotLinkInjections = {}
@@ -46,7 +42,7 @@ describe('linkInjection.js', () => {
 
       const elements = Array.from(document.querySelectorAll('p'))
 
-      injectLinksInDocument(elements, { aiInjections: linkInjections, manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: linkInjections, manualInjections: [] })
 
       const link = /** @type {HTMLAnchorElement} */ (document.querySelector('a'))
 
@@ -65,7 +61,7 @@ describe('linkInjection.js', () => {
 
       const elements = Array.from(document.body.querySelectorAll('p'))
 
-      injectLinksInDocument(elements, { aiInjections: linkInjections, manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: linkInjections, manualInjections: [] })
 
       const links = /** @type {HTMLAnchorElement[]} */ (Array.from(document.querySelectorAll('a')))
 
@@ -89,7 +85,7 @@ describe('linkInjection.js', () => {
 
       const elements = Array.from(document.body.querySelectorAll('p'))
 
-      injectLinksInDocument(elements, { aiInjections: linkInjections, manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: linkInjections, manualInjections: [] })
 
       expect(document.body.innerHTML).toBe(`<p>${sentence}</p>`)
     })
@@ -103,7 +99,7 @@ describe('linkInjection.js', () => {
 
       const elements = Array.from(document.body.querySelectorAll('p'))
 
-      injectLinksInDocument(elements, { aiInjections: linkInjections, manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: linkInjections, manualInjections: [] })
 
       expect(document.body.innerHTML).toBe(`<p>${sentence}</p>`)
     })
@@ -121,7 +117,7 @@ describe('linkInjection.js', () => {
         manualInjections: [{ ...injection, manual: true, removed: true }],
       }
 
-      injectLinksInDocument(elements, injections)
+      injectLinksInDocument(elements, () => null, injections)
 
       expect(document.body.innerHTML).toBe(`<p>${sentence}</p>`)
     })
@@ -139,7 +135,7 @@ describe('linkInjection.js', () => {
         manualInjections: [{ ...injection, manual: true, inactive: true }],
       }
 
-      injectLinksInDocument(elements, injections)
+      injectLinksInDocument(elements, () => null, injections)
 
       expect(document.body.innerHTML).toBe(`<p>${sentence}</p>`)
     })
@@ -152,7 +148,7 @@ describe('linkInjection.js', () => {
 
       const elements = Array.from(document.body.querySelectorAll('p'))
 
-      injectLinksInDocument(elements, { aiInjections: linkInjections, manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: linkInjections, manualInjections: [] })
 
       const button = /** @type {HTMLButtonElement} */ (document.querySelector('button'))
 
@@ -165,7 +161,7 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('I am a title', 'a title')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: [injection], manualInjections: [] })
 
       expect(document.body.innerHTML.includes(injection.sentence)).toBeTruthy()
     })
@@ -176,7 +172,7 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: [injection], manualInjections: [] })
 
       expect(document.body.innerHTML.includes('an injection')).not.toBeTruthy()
       expect(document.body.innerHTML.includes('an')).toBeTruthy()
@@ -188,7 +184,7 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a sentence &amp; an injection.', 'an injection')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: [injection], manualInjections: [] })
 
       expect(document.querySelectorAll('a')).toHaveLength(1)
     })
@@ -199,7 +195,7 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a sentence with an injection.', 'a sentence')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: [injection], manualInjections: [] })
 
       expect(document.querySelector('[data-playpilot-injection-key]')).not.toBeTruthy()
     })
@@ -210,7 +206,7 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a sentence with an injection.', 'some word')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: [injection], manualInjections: [] })
 
       expect(document.querySelector('[data-playpilot-injection-key]')).not.toBeTruthy()
     })
@@ -221,7 +217,7 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a word with a word.', 'a word')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: [injection], manualInjections: [] })
 
       expect(document.querySelectorAll('a')).toHaveLength(2)
       expect(Array.from(document.querySelectorAll('a')).filter(a => a.innerText === 'a word')).toHaveLength(2)
@@ -233,7 +229,7 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a word with a word.', 'a word')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: [injection], manualInjections: [] })
 
       expect(document.querySelectorAll('a')).toHaveLength(2)
       expect(Array.from(document.querySelectorAll('a')).filter(a => a.innerText === 'a word')).toHaveLength(2)
@@ -245,7 +241,7 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('this is a word with a word.', 'a word')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: [injection], manualInjections: [] })
 
       expect(document.querySelectorAll('a')).toHaveLength(2)
       expect(Array.from(document.querySelectorAll('a')).filter(a => a.innerText === 'a word')).toHaveLength(2)
@@ -258,7 +254,7 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('But this sentence is a match.', 'a match')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: [injection], manualInjections: [] })
 
       expect(document.body.innerHTML).toContain('This is a match')
       expect(document.querySelectorAll('[data-playpilot-injection-key]')).toHaveLength(1)
@@ -270,7 +266,7 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: [injection], manualInjections: [] })
 
       expect(document.querySelector('[data-playpilot-injection-key]')).toBeTruthy()
     })
@@ -283,7 +279,7 @@ describe('linkInjection.js', () => {
       /** @type {import('$lib/types/injection').LinkInjection[]} */
       const linkInjections = []
 
-      injectLinksInDocument(elements, { aiInjections: linkInjections, manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: linkInjections, manualInjections: [] })
 
       expect(document.body.innerHTML.includes('This is a sentence with an injection.')).toBeTruthy()
     })
@@ -295,24 +291,25 @@ describe('linkInjection.js', () => {
       /** @type {import('$lib/types/injection').LinkInjection[]} */
       const linkInjections = []
 
-      expect(() => injectLinksInDocument(elements, { aiInjections: linkInjections, manualInjections: [] })).not.toThrow()
+      expect(() => injectLinksInDocument(elements, () => null, { aiInjections: linkInjections, manualInjections: [] })).not.toThrow()
     })
 
-    it('Should mount modal when clicked', async () => {
+    it('Should fire given onclick function when clicked', async () => {
       document.body.innerHTML = '<p>This is a sentence with an injection.</p>'
 
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      const mock = vi.fn()
+      injectLinksInDocument(elements, mock, { aiInjections: [injection], manualInjections: [] })
 
       const link = /** @type {HTMLAnchorElement} */ (document.querySelector('a'))
       await fireEvent.click(link)
 
-      expect(mount).toHaveBeenCalled()
+      expect(mock).toHaveBeenCalledWith(injection)
     })
 
-    it('Should not mount modal multiple times if modal is already open', async () => {
+    it('Should fire given onclick function when clicked for each link in a sentence if more than 1 injection is present', async () => {
       const sentence = 'This is a sentence with an injection.'
       document.body.innerHTML = `<p>${sentence}</p>`
 
@@ -323,13 +320,14 @@ describe('linkInjection.js', () => {
         generateInjection(sentence, 'an injection'),
       ]
 
-      injectLinksInDocument(elements, { aiInjections: linkInjections, manualInjections: [] })
+      const mock = vi.fn()
+      injectLinksInDocument(elements, mock, { aiInjections: linkInjections, manualInjections: [] })
 
       const links = document.querySelectorAll('a')
       await fireEvent.click(links[0])
       await fireEvent.click(links[1])
 
-      expect(mount).toHaveBeenCalledTimes(1)
+      expect(mock).toHaveBeenCalledTimes(2)
     })
 
     it('Should not fire given onclick function when clicked with modifier keys or not left click', async () => {
@@ -339,7 +337,7 @@ describe('linkInjection.js', () => {
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')
 
       const mock = vi.fn()
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      injectLinksInDocument(elements, mock, { aiInjections: [injection], manualInjections: [] })
 
       const link = /** @type {HTMLAnchorElement} */ (document.querySelector('a'))
 
@@ -362,7 +360,7 @@ describe('linkInjection.js', () => {
         generateInjection(sentence, 'not an injection'),
       ]
 
-      const results = injectLinksInDocument(elements, { aiInjections: linkInjections, manualInjections: [] })
+      const results = injectLinksInDocument(elements, () => null, { aiInjections: linkInjections, manualInjections: [] })
 
       expect(results[0].failed).toBe(false)
       expect(results[1].failed).toBe(true)
@@ -379,7 +377,7 @@ describe('linkInjection.js', () => {
         generateInjection('Some unknown sentence', 'nothing'),
       ]
 
-      const results = injectLinksInDocument(elements, { aiInjections: [], manualInjections: linkInjections })
+      const results = injectLinksInDocument(elements, () => null, { aiInjections: [], manualInjections: linkInjections })
 
       expect(results[0].failed_message).toBe('Given text is already inside of a link.')
       expect(results[1].failed_message).toBe('Given sentence was not found in the article.')
@@ -391,7 +389,8 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      const mock = vi.fn()
+      injectLinksInDocument(elements, mock, { aiInjections: [injection], manualInjections: [] })
 
       const link = /** @type {HTMLAnchorElement} */ (document.querySelector('[data-playpilot-injection-key]'))
 
@@ -411,7 +410,8 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      const mock = vi.fn()
+      injectLinksInDocument(elements, mock, { aiInjections: [injection], manualInjections: [] })
 
       const link = /** @type {HTMLAnchorElement} */ (document.querySelector('[data-playpilot-injection-key]'))
 
@@ -432,7 +432,8 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      const mock = vi.fn()
+      injectLinksInDocument(elements, mock, { aiInjections: [injection], manualInjections: [] })
 
       const link = /** @type {HTMLAnchorElement} */ (document.querySelector('[data-playpilot-injection-key]'))
 
@@ -452,7 +453,8 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      const mock = vi.fn()
+      injectLinksInDocument(elements, mock, { aiInjections: [injection], manualInjections: [] })
 
       const link = /** @type {HTMLAnchorElement} */ (document.querySelector('[data-playpilot-injection-key]'))
 
@@ -479,7 +481,7 @@ describe('linkInjection.js', () => {
         generateInjection(sentence, 'injection'),
       ]
 
-      injectLinksInDocument(elements, { aiInjections: linkInjections, manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: linkInjections, manualInjections: [] })
 
       const links = /** @type {HTMLAnchorElement[]} */ (Array.from(document.querySelectorAll('[data-playpilot-injection-key]')))
 
@@ -494,7 +496,7 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.querySelectorAll('p'))
       const injection = { ...generateInjection('This is a sentence with an injection.', 'an injection'), in_text: false }
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      injectLinksInDocument(elements, () => null, { aiInjections: [injection], manualInjections: [] })
 
       expect(document.querySelector('a')).not.toBeTruthy()
     })
@@ -507,7 +509,8 @@ describe('linkInjection.js', () => {
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')
 
-      injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+      const mock = vi.fn()
+      injectLinksInDocument(elements, mock, { aiInjections: [injection], manualInjections: [] })
 
       const link = /** @type {HTMLAnchorElement} */ (document.querySelector('a'))
 
@@ -526,7 +529,8 @@ describe('linkInjection.js', () => {
       const injection = generateInjection('Some sentence', 'an injection')
       const linkInjections = [{ ...injection, after_article: true, inactive: true }, { ...injection, after_article: true, removed: true }]
 
-      injectLinksInDocument(elements, { aiInjections: [], manualInjections: linkInjections })
+      const mock = vi.fn()
+      injectLinksInDocument(elements, mock, { aiInjections: [], manualInjections: linkInjections })
 
       expect(mount).not.toHaveBeenCalled()
       expect(document.querySelector('[data-playpilot-after-article-playlinks]')).not.toBeTruthy()
@@ -549,7 +553,8 @@ describe('linkInjection.js', () => {
         generateInjection('Some fourthtext', 'Some'),
       ]
 
-      injectLinksInDocument(elements, { aiInjections: [], manualInjections: injections })
+      const mock = vi.fn()
+      injectLinksInDocument(elements, mock, { aiInjections: [], manualInjections: injections })
 
       expect(document.querySelectorAll('a')).toHaveLength(4)
     })
@@ -569,7 +574,8 @@ describe('linkInjection.js', () => {
         generateInjection('Somethird text', 'Some'),
       ]
 
-      injectLinksInDocument(elements, { aiInjections: [], manualInjections: injections })
+      const mock = vi.fn()
+      injectLinksInDocument(elements, mock, { aiInjections: [], manualInjections: injections })
 
       expect(document.querySelectorAll('a')).toHaveLength(3)
     })
@@ -593,7 +599,7 @@ describe('linkInjection.js', () => {
         generateInjection('First Title Some other title', 'Fourth'),
       ]
 
-      injectLinksInDocument(elements, { aiInjections: [], manualInjections: injections })
+      injectLinksInDocument(elements, () => null, { aiInjections: [], manualInjections: injections })
 
       expect(document.querySelectorAll('a')).toHaveLength(4)
     })
@@ -696,7 +702,7 @@ describe('linkInjection.js', () => {
       expect(getLinkInjectionElements(parent)[1].tagName).toBe('OL')
     })
 
-    it('Should ignore links, buttons, script tags, style tags, iframes, headers, and more', () => {
+    it('Should ignore links, buttons, script tags, style tags, iframes, and headers', () => {
       document.body.innerHTML = `<section>
         <h1>Some header</h1>
         <h4>Some smaller header</h4>
@@ -708,7 +714,6 @@ describe('linkInjection.js', () => {
         <style>I am styling</style>
         <iframe>I am an iframe</iframe>
         <noscript>I am noscript</noscript>
-        <figcaption>I am a figcaption</figcaption>
 
         <div>
           <a>I am another link</a>
@@ -817,7 +822,7 @@ describe('linkInjection.js', () => {
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')
 
       const elements = getLinkInjectionElements(document.body)
-      insertAfterArticlePlaylinks(elements, [injection])
+      insertAfterArticlePlaylinks(elements, [injection], () => null)
 
       expect(mount).toHaveBeenCalled()
       expect(document.querySelector('p:last-of-type + [data-playpilot-after-article-playlinks]')).toBeTruthy()
@@ -831,7 +836,7 @@ describe('linkInjection.js', () => {
       `
 
       const elements = getLinkInjectionElements(document.body)
-      insertAfterArticlePlaylinks(elements, [])
+      insertAfterArticlePlaylinks(elements, [], () => null)
 
       expect(mount).not.toHaveBeenCalled()
       expect(document.querySelector('[data-playpilot-after-article-playlinks]')).not.toBeTruthy()
@@ -849,7 +854,7 @@ describe('linkInjection.js', () => {
 
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')
       const elements = getLinkInjectionElements(document.body)
-      insertAfterArticlePlaylinks(elements, [injection])
+      insertAfterArticlePlaylinks(elements, [injection], () => null)
 
       expect(document.querySelector('hr + [data-playpilot-after-article-playlinks]')).toBeTruthy()
     })
@@ -866,7 +871,7 @@ describe('linkInjection.js', () => {
 
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')
       const elements = getLinkInjectionElements(document.body)
-      insertAfterArticlePlaylinks(elements, [injection])
+      insertAfterArticlePlaylinks(elements, [injection], () => null)
 
       expect(document.querySelector('hr + [data-playpilot-after-article-playlinks]')).toBeTruthy()
     })
@@ -884,7 +889,7 @@ describe('linkInjection.js', () => {
 
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')
       const elements = getLinkInjectionElements(document.body)
-      insertAfterArticlePlaylinks(elements, [injection])
+      insertAfterArticlePlaylinks(elements, [injection], () => null)
 
       expect(document.querySelector('[data-playpilot-after-article-playlinks] + hr')).toBeTruthy()
     })

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

@@ -53,13 +53,6 @@ describe('localization.js', () => {
       expect(getLanguage()).toBe('sv-SE')
     })
 
-    it('Should return the language given as document language even when using shorthand', () => {
-      const html = /** @type {HTMLElement} */ (document.querySelector('html'))
-      html.setAttribute('lang', 'sv')
-
-      expect(getLanguage()).toBe('sv-SE')
-    })
-
     it('Should return the default language when document language is invalid', () => {
       const html = /** @type {HTMLElement} */ (document.querySelector('html'))
       html.setAttribute('lang', 'no')

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

@@ -0,0 +1,95 @@
+import { describe, it, expect, afterEach, vi } from 'vitest'
+import { fetchAsSession, getSessionId, isAllowedToEdit, sessionKey, setSessionId } from '$lib/session'
+import { fetchLinkInjections } from '$lib/api'
+import { waitFor } from '@testing-library/svelte'
+
+vi.mock('$lib/api', () => ({
+  fetchLinkInjections: vi.fn(() => {}),
+}))
+
+describe('session.ts', () => {
+  afterEach(() => {
+    sessionStorage.clear()
+    vi.resetAllMocks()
+  })
+
+  describe('fetchAsSession', () => {
+    it('Should fetch and fetch again when user is allowed to edit', async() => {
+      // @ts-ignore
+      vi.mocked(fetchLinkInjections).mockResolvedValue({ session_id: 'a', session_last_ping: null })
+
+      fetchAsSession('abc')
+
+      await waitFor(() => {
+        expect(fetchLinkInjections).toHaveBeenCalledTimes(2)
+      })
+    })
+
+    it('Should fetch only once if user is not allowed to edit', async() => {
+      // @ts-ignore
+      vi.mocked(fetchLinkInjections).mockResolvedValue({ session_id: 'a', session_last_ping: Date.now().toString() })
+
+      fetchAsSession('abc')
+
+      await new Promise(res => setTimeout(res, 100)) // Await potential fetch
+
+      expect(fetchLinkInjections).toHaveBeenCalledOnce()
+    })
+  })
+
+  describe('isAllowedToEdit', () => {
+    it('Should return true if either argument is null or empty', () => {
+      expect(isAllowedToEdit('a', null)).toBe(true)
+      expect(isAllowedToEdit('a', '')).toBe(true)
+
+      expect(isAllowedToEdit(null, Date.now().toString())).toBe(true)
+      expect(isAllowedToEdit('', Date.now().toString())).toBe(true)
+    })
+
+    it('Should return true if session id is given and last ping is more than 10 seconds ago', () => {
+      expect(isAllowedToEdit('a', '2020-10-10')).toBe(true)
+      expect(isAllowedToEdit('a', '0')).toBe(true)
+      expect(isAllowedToEdit('a', new Date(Date.now() - 11000).toString())).toBe(true)
+    })
+
+    it('Should return false if session id equals current session id regardless of last ping', () => {
+      sessionStorage.setItem(sessionKey, 'me')
+
+      expect(isAllowedToEdit('me', '2020-10-10')).toBe(true)
+      expect(isAllowedToEdit('me', Date.now().toString())).toBe(true)
+    })
+  })
+
+  describe('getSessionId', () => {
+    it('Should get the item stored in session storage', () => {
+      sessionStorage.setItem(sessionKey, 'abc')
+      expect(getSessionId()).toBe('abc')
+    })
+
+    it('Should set a new item if no value is stored in session storage', () => {
+      sessionStorage.setItem(sessionKey, 'abc')
+
+      const result = getSessionId()
+
+      expect(result).toEqual(expect.any(String))
+      expect(sessionStorage.getItem(sessionKey)).toBe(result)
+    })
+  })
+
+  describe('setSessionId', () => {
+    it('Should set a new item in session storage when no previous value is set', () => {
+      const result = setSessionId()
+
+      expect(result).toEqual(expect.any(String))
+      expect(sessionStorage.getItem(sessionKey)).toBe(result)
+    })
+
+    it('Should return existing item in session storage when set', () => {
+      sessionStorage.setItem(sessionKey, 'abc')
+      const result = setSessionId()
+
+      expect(result).toBe('abc')
+      expect(sessionStorage.getItem(sessionKey)).toBe('abc')
+    })
+  })
+})

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

@@ -4,7 +4,6 @@ import { setTrackingSids, track } from '$lib/tracking'
 import { title } from '$lib/fakeData'
 import { get } from 'svelte/store'
 import { currentDomainSid, currentOrganizationSid } from '$lib/stores/organization'
-import { getFullUrlPath } from '$lib/url'
 
 global.fetch = vi.fn()
 
@@ -41,21 +40,6 @@ describe('$lib/tracking', () => {
       )
     })
 
-    it('Should strip url to full path only', () => {
-      window.location.href = 'https://some-url.com/page?key=value'
-
-      track('Some event')
-
-      expect(global.fetch).toHaveBeenCalledWith(
-        expect.any(String),
-        expect.objectContaining({
-          body: expect.stringMatching(
-            new RegExp(`"url":"${getFullUrlPath()}"`),
-          ),
-        }),
-      )
-    })
-
     it('Should include domain sid in request', () => {
       currentDomainSid.set('some-domain')
 

package/src/tests/routes/+page.test.js

@@ -11,10 +11,20 @@ import { injectLinksInDocument } from '$lib/linkInjection'
 import { isCrawler } from '$lib/crawler'
 
 vi.mock('$lib/api', () => ({
+  fetchLinkInjections: vi.fn(() => {}),
   pollLinkInjections: vi.fn(() => {}),
   fetchConfig: vi.fn(),
 }))
 
+// @ts-ignore
+vi.mock(import('$lib/session'), async (importOriginal) => {
+  const actual = await importOriginal()
+  return {
+    ...actual,
+    fetchAsSession: vi.fn(() => {}),
+  }
+})
+
 vi.mock(import('$lib/linkInjection'), async (importOriginal) => {
   const actual = await importOriginal()
   return {
@@ -80,8 +90,8 @@ describe('$routes/+page.svelte', () => {
     render(page)
 
     await waitFor(() => {
-      expect(pollLinkInjections).toHaveBeenCalledWith(expect.any(String), expect.any(String), { maxTries: 1 })
-      expect(pollLinkInjections).toHaveBeenCalledWith(expect.any(String), expect.any(String), { requireCompletedResult: true, onpoll: expect.any(Function) })
+      expect(pollLinkInjections).toHaveBeenCalledWith(expect.any(String), { maxTries: 1 })
+      expect(pollLinkInjections).toHaveBeenCalledWith(expect.any(String), { requireCompletedResult: true, onpoll: expect.any(Function) })
     })
   })
 
@@ -104,7 +114,7 @@ describe('$routes/+page.svelte', () => {
     })
 
     await waitFor(() => {
-      expect(pollLinkInjections).toHaveBeenCalledWith(expect.any(String), expect.any(String), { requireCompletedResult: true, onpoll: expect.any(Function) })
+      expect(pollLinkInjections).toHaveBeenCalledWith(expect.any(String), { requireCompletedResult: true, onpoll: expect.any(Function) })
     })
 
     expect(pollLinkInjections).toHaveBeenCalledTimes(2)
@@ -167,7 +177,7 @@ describe('$routes/+page.svelte', () => {
     render(page)
 
     await waitFor(() => {
-      expect(pollLinkInjections).toHaveBeenCalledWith(expect.any(String), '<p>Here</p>', { maxTries: 1 })
+      expect(pollLinkInjections).toHaveBeenCalledWith('<p>Here</p>', { maxTries: 1 })
     })
   })
 

package/src/tests/routes/components/Editorial/Editor.test.js

@@ -1,21 +1,37 @@
 import { fireEvent, render } from '@testing-library/svelte'
-import { describe, expect, it, vi } from 'vitest'
+import { beforeAll, describe, expect, it, vi } from 'vitest'
 
 import Editor from '../../../../routes/components/Editorial/Editor.svelte'
 import { title } from '$lib/fakeData'
 import { generateInjection } from '../../../helpers'
 import { track } from '$lib/tracking'
 import { TrackingEvent } from '$lib/enums/TrackingEvent'
+import { fetchLinkInjections } from '$lib/api'
 
 vi.mock('$lib/api', () => ({
   saveLinkInjections: vi.fn(),
+  fetchLinkInjections: vi.fn(() => {}),
 }))
 
+// @ts-ignore
+vi.mock(import('$lib/session'), async (importOriginal) => {
+  const actual = await importOriginal()
+  return {
+    ...actual,
+    isAllowedToEdit: vi.fn(() => true),
+  }
+})
+
 vi.mock('$lib/tracking', () => ({
   track: vi.fn(),
 }))
 
 describe('Editor.svelte', () => {
+  beforeAll(() => {
+    // @ts-ignore
+    vi.mocked(fetchLinkInjections).mockResolvedValue({})
+  })
+
   /** @type {import('$lib/types/injection').LinkInjection[]} */
   const linkInjections = [
     generateInjection('This is a sentence with an injection.', 'an injection'),