@playpilot/tpi 3.1.0 → 3.2.0-beta.1

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

@@ -29,10 +29,10 @@ describe('EditorItem.svelte', () => {
     const { getAllByText, container } = render(EditorItem, { linkInjection })
 
     await fireEvent.mouseEnter(/** @type {HTMLElement} */ (container.querySelector('.item')))
-    expect(/** @type {HTMLElement} */ (getAllByText(linkInjection.title)[0].closest('span')).classList).toContain('injection-highlight')
+    expect(/** @type {HTMLElement} */ (getAllByText(linkInjection.title)[0].closest('span')).classList).toContain('playpilot-injection-highlight')
 
     await fireEvent.mouseLeave(/** @type {HTMLElement} */ (container.querySelector('.item')))
-    expect(/** @type {HTMLElement} */ (getAllByText(linkInjection.title)[0].closest('span')).classList).not.toContain('injection-highlight')
+    expect(/** @type {HTMLElement} */ (getAllByText(linkInjection.title)[0].closest('span')).classList).not.toContain('playpilot-injection-highlight')
   })
 
   it('Should highlight multiple elements if multiple are present', async () => {
@@ -44,7 +44,20 @@ describe('EditorItem.svelte', () => {
     const { container } = render(EditorItem, { linkInjection })
 
     await fireEvent.mouseEnter(/** @type {HTMLElement} */ (container.querySelector('.item')))
-    expect(document.querySelectorAll('.injection-highlight')).toHaveLength(2)
+    expect(document.querySelectorAll('.playpilot-injection-highlight')).toHaveLength(2)
+  })
+
+  it('Should highlight the sentence the injection is in if an injection failed but a matching sentence exists.', async () => {
+    document.body.innerHTML = '<p>This is a sentence with an injection.</p>'
+
+    const failedInjection = generateInjection('This is a sentence', 'fail')
+
+    injectLinksInDocument(Array.from(document.querySelectorAll('p')), () => null, { aiInjections: [], manualInjections: [failedInjection] })
+
+    const { container } = render(EditorItem, { linkInjection })
+
+    await fireEvent.mouseEnter(/** @type {HTMLElement} */ (container.querySelector('.item')))
+    expect(/** @type {HTMLElement} */ (document.querySelector('p')).classList).toContain('playpilot-injection-highlight')
   })
 
   it('Should scroll matching link into view when component is clicked', async () => {
@@ -59,9 +72,9 @@ describe('EditorItem.svelte', () => {
   })
 
   it('Should not scroll matching link into view when component is clicked but there is no matching injection', async () => {
-    document.body.innerHTML = '<p>This has no matching injections.</p>'
+    document.body.innerHTML = '<main><p>This has no matching injections.</p></main>'
 
-    injectLinksInDocument(Array.from(document.querySelectorAll('p')), () => null, linkInjections)
+    injectLinksInDocument(Array.from(document.querySelectorAll('main p')), () => null, linkInjections)
 
     const { container } = render(EditorItem, { linkInjection })
 
@@ -108,9 +121,9 @@ describe('EditorItem.svelte', () => {
   })
 
   it('Should show different state when injection failed', () => {
-    const { getByText, queryByLabelText } = render(EditorItem, { linkInjection: { ...linkInjection, failed: true } })
+    const { getByText, queryByLabelText } = render(EditorItem, { linkInjection: { ...linkInjection, failed: true, failed_message: 'Some message' } })
 
-    expect(getByText('the link could not be injected', { exact: false })).toBeTruthy()
+    expect(getByText('Some message')).toBeTruthy()
     expect(queryByLabelText('Expand')).not.toBeTruthy()
     expect(track).toHaveBeenCalled()
   })
@@ -164,4 +177,16 @@ describe('EditorItem.svelte', () => {
     expect(queryByText('Visible')).toBeTruthy()
     expect(container.querySelector('.inactive')).not.toBeTruthy()
   })
+
+  it('Should display an icon when playlink types are invalid', async () => {
+    const { getByLabelText } = render(EditorItem, { linkInjection: { ...linkInjection, in_text: false  } })
+
+    expect(getByLabelText('Invalid playlink settings')).toBeTruthy()
+  })
+
+  it('Should not display an icon when playlink types are valid', async () => {
+    const { queryByLabelText } = render(EditorItem, { linkInjection: { ...linkInjection, in_text: true  } })
+
+    expect(queryByLabelText('Invalid playlink settings')).not.toBeTruthy()
+  })
 })

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

@@ -4,7 +4,7 @@ import { beforeEach, describe, expect, it } from 'vitest'
 import PlaylinkTypeSelect from '../../../../routes/components/Editorial/PlaylinkTypeSelect.svelte'
 
 describe('PlaylinkTypeSelect.svelte', () => {
-  /** @type {LinkInjection} */
+  /** @type {import('$lib/types/injection').LinkInjection} */
   const linkInjection = {
     sid: '1',
     title: 'test',
@@ -60,4 +60,16 @@ describe('PlaylinkTypeSelect.svelte', () => {
     expect(getByText('Modal button').classList).toContain('active')
     expect(/** @type {HTMLElement} */(document.querySelector('.active-marker')).classList).toContain('right')
   })
+
+  it('Should display a message when playlink types are invalid', async () => {
+    const { getByText } = render(PlaylinkTypeSelect, { linkInjection: { ...linkInjection, in_text: false  } })
+
+    expect(getByText('At least one layout option must be selected for the playlink to be visible.')).toBeTruthy()
+  })
+
+  it('Should not display a message when playlink types are valid', async () => {
+    const { queryByText } = render(PlaylinkTypeSelect, { linkInjection: { ...linkInjection, in_text: true  } })
+
+    expect(queryByText('At least one layout option must be selected for the playlink to be visible.')).not.toBeTruthy()
+  })
 })