@playpilot/tpi 1.4.3 → 2.0.0

package/src/routes/components/Editorial/PlaylinkTypeSelect.svelte

@@ -13,7 +13,7 @@
   <Switch
     fullwidth
     active={linkInjection.in_text ?? true}
-    onclick={(active) => linkInjection.in_text = active}
+    onclick={(active) => { linkInjection.in_text = active; linkInjection.manual = true }}
     label="In-text Playlink">
     <IconAlign align="center" />
     In-text Playlink
@@ -22,7 +22,7 @@
   <Switch
     fullwidth
     active={linkInjection.after_article ?? false}
-    onclick={(active) => linkInjection.after_article = active}
+    onclick={(active) => { linkInjection.after_article = active; linkInjection.manual = true }}
     label="Bottom Playlink">
     <IconAlign align="bottom" />
     Bottom Playlink

package/src/routes/components/Icons/IconAi.svelte

@@ -0,0 +1 @@
+<svg width="24px" height="24px" viewBox="0 0 64 64" stroke-width="3" stroke="currentColor" fill="none"><circle cx="34.52" cy="11.43" r="5.82"/><circle cx="53.63" cy="31.6" r="5.82"/><circle cx="34.52" cy="50.57" r="5.82"/><circle cx="15.16" cy="42.03" r="5.82"/><circle cx="15.16" cy="19.27" r="5.82"/><circle cx="34.51" cy="29.27" r="4.7"/><line x1="20.17" y1="16.3" x2="28.9" y2="12.93"/><line x1="38.6" y1="15.59" x2="49.48" y2="27.52"/><line x1="50.07" y1="36.2" x2="38.67" y2="46.49"/><line x1="18.36" y1="24.13" x2="30.91" y2="46.01"/><line x1="20.31" y1="44.74" x2="28.7" y2="48.63"/><line x1="17.34" y1="36.63" x2="31.37" y2="16.32"/><line x1="20.52" y1="21.55" x2="30.34" y2="27.1"/><line x1="39.22" y1="29.8" x2="47.81" y2="30.45"/><line x1="34.51" y1="33.98" x2="34.52" y2="44.74"/></svg>

package/src/tests/helpers.js

@@ -1,4 +1,5 @@
 import { vi } from 'vitest'
+import { title as title_details } from '$lib/fakeData'
 
 /**
  * @param {Object} [options]
@@ -16,3 +17,19 @@ export function fakeFetch({ response = '', status = 200, ok = true } = {}) {
     }),
   )
 }
+
+/**
+ * @param {string} sentence
+ * @param {string} title
+ * @returns {LinkInjection}
+ */
+export function generateInjection(sentence, title) {
+  return {
+    sid: Math.random().toString(),
+    title,
+    sentence,
+    playpilot_url: 'https://some-link.com/',
+    key: 'some-key-' + Math.random().toString(),
+    title_details,
+  }
+}

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

@@ -2,6 +2,7 @@ import { describe, it, expect, vi, afterEach, beforeEach } from 'vitest'
 import { fetchLinkInjections, pollLinkInjections } from '$lib/api'
 import { fakeFetch } from '../helpers'
 import { authorize, isEditorialModeEnabled } from '$lib/auth'
+import { linkInjections } from '$lib/fakeData'
 
 vi.mock('$lib/auth', async importActual => {
   const actual = await importActual()
@@ -104,10 +105,10 @@ describe('$lib/api', () => {
   })
 
   describe('pollLinkInjections', () => {
-    it('Should poll endpoint while results are not yet ready', async () => {
-      fakeFetch({ response: { injections_ready: false } })
+    it('Should poll endpoint while results are not yet ready when requireCompletedResults is true', async () => {
+      fakeFetch({ response: { automation_enabled: true, ai_running: true } })
 
-      pollLinkInjections('https://some-url', 'some-html', 500)
+      pollLinkInjections('https://some-url', 'some-html', { requireCompletedResult: true, pollInterval: 500 })
 
       expect(global.fetch).toHaveBeenCalled()
 
@@ -119,21 +120,22 @@ describe('$lib/api', () => {
     })
 
     it('Should stop polling if replacements are ready', async () => {
-      fakeFetch({ response: { injections_ready: true, link_injections: [{ title: 'value' }] } })
+      const response = { automation_enabled: true, ai_running: false, link_injections: [{ title: 'value' }] }
+      fakeFetch({ response })
 
-      const result = await pollLinkInjections('https://some-url', 'some-html', 500)
+      const result = await pollLinkInjections('https://some-url', 'some-html', { requireCompletedResult: true, pollInterval: 500 })
 
       expect(global.fetch).toHaveBeenCalled()
-      expect(result).toEqual([{ title: 'value', key: expect.any(String) }])
+      expect(result).toEqual(response)
 
       await new Promise(res => setTimeout(res, 1000)) // Wait for a potential poll
       expect(global.fetch).toHaveBeenCalledTimes(1)
     })
 
     it('Should stop polling if max limit was exceeded', async () => {
-      fakeFetch({ response: { injections_ready: false } })
+      fakeFetch({ response: { automation_enabled: true, ai_running: true } })
 
-      await pollLinkInjections('https://some-url', 'some-html', 200, 2)
+      await pollLinkInjections('https://some-url', 'some-html', { requireCompletedResult: true, pollInterval: 200, maxTries: 2 })
 
       await new Promise(res => setTimeout(res, 1000)) // Wait for a potential poll
       expect(global.fetch).toHaveBeenCalledTimes(2)
@@ -142,7 +144,7 @@ describe('$lib/api', () => {
     it('Should continue polling if endpoint throws error', async () => {
       fakeFetch({ ok: false })
 
-      pollLinkInjections('https://some-url', 'some-html', 500)
+      pollLinkInjections('https://some-url', 'some-html', { pollInterval: 500 })
 
       expect(global.fetch).toHaveBeenCalled()
 
@@ -151,25 +153,10 @@ describe('$lib/api', () => {
     })
 
     it('Should return empty array if link_injections are null', async () => {
-      fakeFetch({ response: { injections_ready: true, link_injections: null } })
+      fakeFetch({ response: { link_injections: null, ai_injections: null } })
 
       const result = await pollLinkInjections('https://some-url', 'some-html')
-      expect(result).toEqual([])
-    })
-
-    it('Should return empty array despite actual results if injections_enabled is false', async () => {
-      fakeFetch({ response: { injections_enabled: false, link_injections: [{ title: 'value' }] } })
-
-      const result = await pollLinkInjections('https://some-url', 'some-html')
-      expect(result).toEqual([])
-    })
-
-    it('Should actual results if injections_enabled is false but editiorial mode is true', async () => {
-      vi.mocked(isEditorialModeEnabled).mockResolvedValueOnce(true)
-      fakeFetch({ response: { injections_enabled: false, link_injections: [{ title: 'value' }] } })
-
-      const result = await pollLinkInjections('https://some-url', 'some-html')
-      expect(result).toEqual([])
+      expect(result).toEqual(expect.objectContaining({ link_injections: [], ai_injections: [] }))
     })
   })
 })