@playpilot/tpi 8.11.0-beta.1 → 8.11.0

package/src/tests/lib/api/externalPages.test.js

@@ -36,7 +36,7 @@ describe('$lib/api/externalPages', () => {
     it('Should call api with GET method with given url', async () => {
       vi.mocked(api).mockResolvedValueOnce('Some response')
 
-      const response = await fetchLinkInjections('Some text', { url: 'https://some-url', hash: 'some-hash' })
+      const response = await fetchLinkInjections({ url: 'https://some-url' })
 
       expect(response).toBe('Some response')
       expect(api).toHaveBeenCalledWith(
@@ -50,7 +50,7 @@ describe('$lib/api/externalPages', () => {
     it('Should call api with POST method and full details when fetching with method set to POST', async () => {
       vi.mocked(api).mockResolvedValueOnce('Some response')
 
-      const response = await fetchLinkInjections('Some text', { url: 'https://some-url', hash: 'some-hash', params: { key: 'value' }, method: 'POST' })
+      const response = await fetchLinkInjections({ url: 'https://some-url', params: { key: 'value' }, method: 'POST' })
 
       expect(response).toBe('Some response')
       expect(api).toHaveBeenCalledWith(
@@ -69,14 +69,14 @@ describe('$lib/api/externalPages', () => {
     it('Should call api with full details when fetching with run_ai as true and method is set to POST', async () => {
       vi.mocked(api).mockResolvedValueOnce('Some response')
 
-      const response = await fetchLinkInjections('Some text', { url: 'https://some-url', hash: 'some-hash', params: { run_ai: true }, method: 'POST' })
+      const response = await fetchLinkInjections({ url: 'https://some-url', params: { run_ai: true }, method: 'POST' })
 
       expect(response).toBe('Some response')
       expect(api).toHaveBeenCalledWith(
         expect.stringContaining('api-token'),
         expect.objectContaining({
           body: {
-            hash: 'some-hash',
+            hash: expect.any(String),
             url: 'https://some-url',
             run_ai: true,
             metadata: {
@@ -84,7 +84,7 @@ describe('$lib/api/externalPages', () => {
               content_modified_time: null,
               content_published_time: null,
             },
-            page_text: 'Some text',
+            page_text: expect.any(String),
           },
           method: 'POST',
         }),
@@ -95,13 +95,13 @@ describe('$lib/api/externalPages', () => {
       // @ts-ignore
       window.PlayPilotLinkInjections = null
 
-      await expect(async () => await fetchLinkInjections('Some text', { hash: 'some-hash' })).rejects.toThrowError('No token was provided')
+      await expect(async () => await fetchLinkInjections()).rejects.toThrowError('No token was provided')
     })
 
     it('Should throw when response was incorrect', async () => {
       vi.mocked(api).mockRejectedValueOnce({ ok: false })
 
-      await expect(async () => await fetchLinkInjections('Some text', { hash: 'some-hash' })).rejects.toThrowError()
+      await expect(async () => await fetchLinkInjections()).rejects.toThrowError()
     })
 
     it('Should use given api token', async () => {
@@ -110,7 +110,7 @@ describe('$lib/api/externalPages', () => {
       // @ts-ignore
       window.PlayPilotLinkInjections = { token: 'token' }
 
-      await fetchLinkInjections('Some text', { hash: 'some-hash' })
+      await fetchLinkInjections()
 
       expect(api).toHaveBeenCalledWith(
         expect.stringContaining('api-token=token'),
@@ -124,7 +124,7 @@ describe('$lib/api/externalPages', () => {
       // @ts-ignore
       window.PlayPilotLinkInjections = null
 
-      await expect(async () => await fetchLinkInjections('Some text', { hash: 'some-hash' })).rejects.toThrowError()
+      await expect(async () => await fetchLinkInjections()).rejects.toThrowError()
       expect(api).not.toHaveBeenCalled()
     })
 
@@ -134,7 +134,7 @@ describe('$lib/api/externalPages', () => {
       vi.mocked(authorize).mockResolvedValueOnce(true)
       vi.mocked(isEditorialModeEnabled).mockResolvedValueOnce(true)
 
-      const response = await fetchLinkInjections('Some text', { hash: 'some-hash' })
+      const response = await fetchLinkInjections()
 
       expect(response).toBe('Some response')
       expect(api).toHaveBeenCalledWith(
@@ -146,7 +146,7 @@ describe('$lib/api/externalPages', () => {
     it('Should call api with given language', async () => {
       vi.mocked(api).mockResolvedValueOnce('Some response')
 
-      await fetchLinkInjections('Some text', { hash: 'some-hash' })
+      await fetchLinkInjections()
 
       expect(api).toHaveBeenCalledWith(
         expect.stringContaining(`&language=${Language.English}`),
@@ -156,7 +156,7 @@ describe('$lib/api/externalPages', () => {
       // @ts-ignore
       window.PlayPilotLinkInjections = { token: 'token', language: Language.Swedish }
 
-      await fetchLinkInjections('Some text', { hash: 'some-hash' })
+      await fetchLinkInjections()
 
       expect(api).toHaveBeenCalledWith(
         expect.stringContaining(`&language=${Language.Swedish}`),
@@ -169,7 +169,7 @@ describe('$lib/api/externalPages', () => {
     it('Should poll endpoint while results are not yet ready when requireCompletedResults is true', async () => {
       vi.mocked(api).mockResolvedValueOnce({ ai_enabled: true, ai_running: true })
 
-      pollLinkInjections('Some text', { requireCompletedResult: true, pollInterval: 500 })
+      pollLinkInjections({ requireCompletedResult: true, pollInterval: 500 })
 
       expect(api).toHaveBeenCalled()
 
@@ -184,7 +184,7 @@ describe('$lib/api/externalPages', () => {
       vi.mocked(api).mockResolvedValue({ ai_enabled: true, ai_running: true })
 
       const onpoll = vi.fn()
-      pollLinkInjections('Some text', { requireCompletedResult: true, pollInterval: 500, onpoll })
+      pollLinkInjections({ requireCompletedResult: true, pollInterval: 500, onpoll })
 
       await waitFor(() => {
         expect(onpoll).toHaveBeenCalledTimes(1)
@@ -201,7 +201,7 @@ describe('$lib/api/externalPages', () => {
       const response = { ai_enabled: true, ai_running: false, manual_injections: [{ title: 'value' }] }
       vi.mocked(api).mockResolvedValueOnce(response)
 
-      const result = await pollLinkInjections('Some text', { requireCompletedResult: true, pollInterval: 500 })
+      const result = await pollLinkInjections({ requireCompletedResult: true, pollInterval: 500 })
 
       expect(api).toHaveBeenCalled()
       expect(result).toEqual(response)
@@ -213,7 +213,7 @@ describe('$lib/api/externalPages', () => {
     it('Should stop polling if max limit was exceeded', async () => {
       vi.mocked(api).mockResolvedValueOnce({ ai_enabled: true, ai_running: true })
 
-      await pollLinkInjections('Some text', { requireCompletedResult: true, pollInterval: 200, maxTries: 2 })
+      await pollLinkInjections({ requireCompletedResult: true, pollInterval: 200, maxTries: 2 })
 
       await new Promise(res => setTimeout(res, 1000)) // Wait for a potential poll
       expect(api).toHaveBeenCalledTimes(2)
@@ -222,7 +222,7 @@ describe('$lib/api/externalPages', () => {
     it('Should continue polling if endpoint throws error', async () => {
       vi.mocked(api).mockRejectedValueOnce({ ok: false })
 
-      pollLinkInjections('Some text', { pollInterval: 500 })
+      pollLinkInjections({ pollInterval: 500 })
 
       expect(api).toHaveBeenCalled()
 
@@ -233,16 +233,16 @@ describe('$lib/api/externalPages', () => {
     it('Should return empty array if manual_injections are null', async () => {
       vi.mocked(api).mockResolvedValueOnce({ ai_enabled: null, ai_running: null })
 
-      const result = await pollLinkInjections('Some text')
+      const result = await pollLinkInjections()
       expect(result).toEqual(expect.objectContaining({ manual_injections: [], ai_injections: [] }))
     })
 
     it('Should call api twice if runAiWhenRelevant is true and results expects new run', async () => {
-      document.body.innerHTML = '<meta content="2025-04-20T00:00:00Z" property="article:modified_time">'
+      document.body.innerHTML = '<main>Some text</main> <meta content="2025-04-20T00:00:00Z" property="article:modified_time">'
 
       vi.mocked(api).mockResolvedValueOnce({ ai_enabled: true })
 
-      await pollLinkInjections('Some text', { runAiWhenRelevant: true, maxTries: 1 })
+      await pollLinkInjections({ runAiWhenRelevant: true, maxTries: 1 })
 
       expect(api).toHaveBeenCalledTimes(2)
     })
@@ -250,7 +250,7 @@ describe('$lib/api/externalPages', () => {
     it('Should call api twice if initial response returns 404', async () => {
       vi.mocked(api).mockRejectedValueOnce({ status: 404 })
 
-      await pollLinkInjections('Some text', { maxTries: 1 })
+      await pollLinkInjections({ maxTries: 1 })
 
       expect(api).toHaveBeenCalledTimes(2)
       expect(api).toHaveBeenCalledWith(
@@ -266,7 +266,7 @@ describe('$lib/api/externalPages', () => {
     it('Should call api three times if initial response returns 404 and second response returns ai_enabled set to true', async () => {
       vi.mocked(api).mockRejectedValueOnce({ status: 404 }).mockResolvedValueOnce({ ai_enabled: true })
 
-      await pollLinkInjections('Some text', { maxTries: 1 })
+      await pollLinkInjections({ maxTries: 1 })
 
       expect(api).toHaveBeenCalledTimes(3)
       expect(api).toHaveBeenCalledWith(
@@ -287,6 +287,28 @@ describe('$lib/api/externalPages', () => {
         }),
       )
     })
+
+    it('Should use the given selector when present', async () => {
+      vi.mocked(api).mockRejectedValueOnce({ status: 404 }).mockResolvedValueOnce({ ai_enabled: true })
+
+      document.body.innerHTML = '<div class="some-element"><p>Here</p></div> <p>Not here</p>'
+
+      // @ts-ignore
+      window.PlayPilotLinkInjections = { token: 'token', selector: '.some-element' }
+
+      await pollLinkInjections({ runAiWhenRelevant: true, maxTries: 1 })
+
+      await waitFor(() => {
+        expect(api).toHaveBeenCalledWith(
+          expect.any(String),
+          expect.objectContaining({
+            body: expect.objectContaining({
+              page_text: 'Here',
+            }),
+          }),
+        )
+      })
+    })
   })
 
   describe('runAiBasedOnResponse', () => {

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

@@ -18,7 +18,7 @@ describe('$lib/api/session', () => {
       // @ts-ignore
       vi.mocked(fetchLinkInjections).mockResolvedValue({ session_id: 'a', session_last_ping: null })
 
-      fetchAsSession('abc')
+      fetchAsSession()
 
       await waitFor(() => {
         expect(fetchLinkInjections).toHaveBeenCalledTimes(2)
@@ -29,7 +29,7 @@ describe('$lib/api/session', () => {
       // @ts-ignore
       vi.mocked(fetchLinkInjections).mockResolvedValue({ session_id: 'a', session_last_ping: Date.now().toString() })
 
-      fetchAsSession('abc')
+      fetchAsSession()
 
       await new Promise(res => setTimeout(res, 100)) // Await potential fetch
 

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

@@ -240,9 +240,9 @@ describe('meta.js', () => {
     })
 
     it('Should return schema relevant object', () => {
-      document.body.innerHTML = '<script type="application/ld+json">{ "key": "value" }</script>'
+      document.body.innerHTML = '<script type="application/ld+json">{ "key": "value", "dateModified": "a" }</script>'
 
-      expect(getSchemaJson()).toEqual({ key: 'value' })
+      expect(getSchemaJson()).toEqual({ key: 'value', dateModified: 'a' })
     })
 
     it('Should return empty object when json object is invalid', () => {
@@ -261,5 +261,14 @@ describe('meta.js', () => {
       document.body.innerHTML = '<script type="application/ld+json">{ "key": "Some "unescaped" value" }</script>'
       expect(getSchemaJson()).toEqual({})
     })
+
+    it('Should return script with dateModified if multiple scripts are present', () => {
+      document.body.innerHTML = `
+        <script type="application/ld+json"></script>
+        <script type="application/ld+json">{ "dateModified": "a" }</script>
+      `
+
+      expect(getSchemaJson()).toEqual({ dateModified: 'a' })
+    })
   })
 })

package/src/tests/main.test.js

@@ -51,7 +51,6 @@ describe('main.ts', () => {
   describe('initialize', () => {
     it('Should not call setConsent when only a token is passed', () => {
       window.PlayPilotLinkInjections.initialize({ token: 'a' })
-      console.log(window.PlayPilotLinkInjections)
 
       expect(setConsent).not.toHaveBeenCalled()
 

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

@@ -134,8 +134,8 @@ describe('$routes/+page.svelte', () => {
     render(page)
 
     await waitFor(() => {
-      expect(pollLinkInjections).toHaveBeenCalledWith(expect.any(String), { maxTries: 1, runAiWhenRelevant: true })
-      expect(pollLinkInjections).toHaveBeenCalledWith(expect.any(String), { requireCompletedResult: true, onpoll: expect.any(Function) })
+      expect(pollLinkInjections).toHaveBeenCalledWith({ maxTries: 1, runAiWhenRelevant: true })
+      expect(pollLinkInjections).toHaveBeenCalledWith({ requireCompletedResult: true, onpoll: expect.any(Function) })
     })
   })
 
@@ -158,7 +158,7 @@ describe('$routes/+page.svelte', () => {
     })
 
     await waitFor(() => {
-      expect(pollLinkInjections).toHaveBeenCalledWith(expect.any(String), { requireCompletedResult: true, onpoll: expect.any(Function) })
+      expect(pollLinkInjections).toHaveBeenCalledWith({ requireCompletedResult: true, onpoll: expect.any(Function) })
     })
 
     expect(pollLinkInjections).toHaveBeenCalledTimes(2)
@@ -221,7 +221,7 @@ describe('$routes/+page.svelte', () => {
     render(page)
 
     await waitFor(() => {
-      expect(pollLinkInjections).toHaveBeenCalledWith('Here', { maxTries: 1, runAiWhenRelevant: true })
+      expect(pollLinkInjections).toHaveBeenCalledWith({ maxTries: 1, runAiWhenRelevant: true })
     })
   })
 

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

@@ -44,7 +44,7 @@ describe('ManualInjection.svelte', () => {
     vi.mocked(getIndexOfSelection).mockReturnValueOnce({ start: 0, end: 0 })
     createSelectionMock()
 
-    const { getByLabelText } = render(ManualInjection, { pageText: document.body.innerText, onsave: () => null })
+    const { getByLabelText } = render(ManualInjection, { onsave: () => null })
 
     await fireEvent.mouseUp(window)
 
@@ -56,7 +56,7 @@ describe('ManualInjection.svelte', () => {
   it('Should disable save button by default', async () => {
     createSelectionMock()
 
-    const { getByText } = render(ManualInjection, { pageText: '', onsave: () => null })
+    const { getByText } = render(ManualInjection, { onsave: () => null })
 
     expect(getByText('Add playlink').hasAttribute('disabled')).toBeTruthy()
   })
@@ -67,7 +67,7 @@ describe('ManualInjection.svelte', () => {
     vi.mocked(searchTitles).mockResolvedValueOnce([title])
     vi.mocked(getIndexOfSelection).mockReturnValueOnce({ start: 0, end: 0 })
 
-    const { getByText } = render(ManualInjection, { pageText: document.body.innerText, onsave: () => null })
+    const { getByText } = render(ManualInjection, { onsave: () => null })
 
     await fireEvent.mouseUp(window)
     await waitFor(() => getByText(title.title))
@@ -83,7 +83,7 @@ describe('ManualInjection.svelte', () => {
     vi.mocked(getIndexOfSelection).mockReturnValueOnce({ start: 5, end: 10 })
 
     const onsave = vi.fn()
-    const { getByText } = render(ManualInjection, { pageText: document.body.innerText, onsave })
+    const { getByText } = render(ManualInjection, { onsave })
 
     await fireEvent.mouseUp(window)
     await fireEvent.click(getByText(title.title))
@@ -125,7 +125,7 @@ describe('ManualInjection.svelte', () => {
     }))
 
     const onsave = vi.fn()
-    const { getByText } = render(ManualInjection, { pageText: document.body.innerText, onsave })
+    const { getByText } = render(ManualInjection, { onsave })
 
     await fireEvent.mouseUp(window)
     await fireEvent.click(getByText(title.title))
@@ -167,7 +167,7 @@ describe('ManualInjection.svelte', () => {
     }))
 
     const onsave = vi.fn()
-    const { getByText } = render(ManualInjection, { pageText: document.body.innerText, onsave })
+    const { getByText } = render(ManualInjection, { onsave })
 
     await fireEvent.mouseUp(window)
     await fireEvent.click(getByText(title.title))
@@ -209,7 +209,7 @@ describe('ManualInjection.svelte', () => {
     }))
 
     const onsave = vi.fn()
-    const { getByText } = render(ManualInjection, { pageText: document.body.innerText, onsave })
+    const { getByText } = render(ManualInjection, { onsave })
 
     await fireEvent.mouseUp(window)
     await fireEvent.click(getByText(title.title))
@@ -237,7 +237,7 @@ describe('ManualInjection.svelte', () => {
     let container = document.querySelector('div')
 
     const onsave = vi.fn()
-    const { unmount } = render(ManualInjection, { pageText: document.body.innerText, onsave })
+    const { unmount } = render(ManualInjection, { onsave })
 
     // @ts-ignore
     window.getSelection = vi.fn(() => ({
@@ -275,7 +275,7 @@ describe('ManualInjection.svelte', () => {
       focusNode: container,
     }))
 
-    ;(render(ManualInjection, { pageText: document.body.innerText, onsave }))
+    ;(render(ManualInjection, { onsave }))
 
     await fireEvent.mouseUp(window)