@playpilot/tpi 3.7.2 → 3.8.0

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

@@ -369,7 +369,7 @@ describe('linkInjection.js', () => {
     })
 
     it('Should return corresponding messages for failed injections', () => {
-      const sentence = '<p>This is <a href="/">a sentence</a> with an injection.</p>'
+      const sentence = 'This is <a href="/">a sentence</a> with an injection.'
       document.body.innerHTML = `<p>${sentence}</p>`
 
       const elements = Array.from(document.body.querySelectorAll('p'))
@@ -385,6 +385,23 @@ describe('linkInjection.js', () => {
       expect(results[1].failed_message).toBe('Given sentence was not found in the article.')
     })
 
+    it('Should disregard spaces and special characters in failed message', () => {
+      const sentence = 'This is a &amp; sentence with an injection.'
+      document.body.innerHTML = `<p>${sentence}</p>`
+
+      const elements = Array.from(document.body.querySelectorAll('p'))
+
+      const linkInjections = [
+        generateInjection('This is a&sentence with aninjection.', 'an injection'),
+        generateInjection('This is a&sentence with aninjection.', 'nothing'),
+      ]
+
+      const results = injectLinksInDocument(elements, { aiInjections: [], manualInjections: linkInjections })
+
+      expect(results[0].failed_message).not.toBeTruthy()
+      expect(results[1].failed_message).toBe('The link failed to inject for unknown reasons.')
+    })
+
     it('Should mount popover component when element other than popover or link is hovered', async () => {
       document.body.innerHTML = '<p>This is a sentence with an injection.</p>'
 
@@ -449,6 +466,29 @@ describe('linkInjection.js', () => {
       expect(mount).not.toHaveBeenCalled()
     })
 
+    it('Should not mount popover on mouseover again if link is already being hovered', async () => {
+      document.body.innerHTML = '<p>This is a sentence with an injection.</p>'
+
+      const elements = Array.from(document.body.querySelectorAll('p'))
+      const injections = [
+        generateInjection('This is a sentence with an injection.', 'a sentence'),
+        generateInjection('This is a sentence with an injection.', 'an injection'),
+      ]
+
+      injectLinksInDocument(elements, { aiInjections: injections, manualInjections: [] })
+
+      const [link1, link2] = document.querySelectorAll('[data-playpilot-injection-key]')
+
+      vi.useFakeTimers()
+
+      await fireEvent.mouseEnter(link1)
+      vi.advanceTimersByTime(200)
+      await fireEvent.mouseEnter(link2)
+      vi.advanceTimersByTime(500)
+
+      expect(mount).toHaveBeenCalledTimes(1)
+    })
+
     it('Should inject links of the same phrase when multiple are present', () => {
       document.body.innerHTML = '<p>This is a sentence with an injection and another injection</p>'
 
@@ -734,10 +774,10 @@ describe('linkInjection.js', () => {
     it('Should return elements in the same order they were given', () => {
       document.body.innerHTML = `<section>
         <div>
-          <p><span><strong>Some first text</strong></span></p>
+          <div><span><strong>Some first text</strong></span></div>
           <div>
             <em>Some empasized</em>
-            <main><p><em><i>Some deeply nested text</i></em></p></main>
+            <main><span><em><i>Some deeply nested text</i></em></span></main>
             <div>Some text</div>
           </div>
         </div>
@@ -752,6 +792,33 @@ describe('linkInjection.js', () => {
       expect(elements[2].nodeName).toBe('I')
       expect(elements[3].nodeName).toBe('DIV')
     })
+
+    it('Should exclude elements that match or are in the given excludeElementsSelector attribute', () => {
+      document.body.innerHTML = `<section>
+        <p>I am a regular element</p>
+        <div data-exclude>I am to be excluded. <span>And so am I!</span></div>
+      </section>
+      `
+      const parent = /** @type {HTMLElement} */ (document.querySelector('section'))
+
+      const result = getLinkInjectionElements(parent, '[data-exclude]')
+      expect(result).toHaveLength(1)
+      expect(result[0].innerText).toBe('I am a regular element')
+    });
+
+    it('Should return paragraphs fully even if they contain no direct text nodes, skipping empty paragraphs', () => {
+      document.body.innerHTML = `<section>
+        <p>Some text</p>
+        <p><span>Some text</span> <span>broken up</span></p>
+        <p><span>Some text</span> <span>broken up</span> but also contains direct text</p>
+        <p></p>
+      </section>`
+
+      const parent = /** @type {HTMLElement} */ (document.querySelector('section'))
+      const elements = getLinkInjectionElements(parent)
+
+      expect(elements).toHaveLength(3)
+    })
   })
 
   describe('getLinkInjectionsParentElement', () => {

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

@@ -418,7 +418,7 @@ describe('$routes/+page.svelte', () => {
     })
 
     describe('custom_style', () => {
-      it('Should insert custom html tag with returned value for given selector from config', async () => {
+      it('Should insert custom html tag with returned value from config', async () => {
         vi.mocked(fetchConfig).mockResolvedValueOnce({ custom_style: 'some: style' })
 
         // @ts-ignore
@@ -427,17 +427,7 @@ describe('$routes/+page.svelte', () => {
         render(page)
 
         await waitFor(() => {
-          expect(document.querySelector('#playpilot-custom-style')?.textContent).toBe('.some-element, .modal, .popover { some: style }')
-        })
-      })
-
-      it('Should insert custom html tag with returned value for body when no selector is given', async () => {
-        vi.mocked(fetchConfig).mockResolvedValueOnce({ custom_style: 'some: style' })
-
-        render(page)
-
-        await waitFor(() => {
-          expect(document.querySelector('#playpilot-custom-style')?.textContent).toBe('body, .modal, .popover { some: style }')
+          expect(document.querySelector('#playpilot-custom-style')?.textContent).toBe('[data-playpilot-link-injections] { some: style }')
         })
       })