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

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

@@ -26,7 +26,7 @@ describe('EditorItem.svelte', () => {
   it('Should highlight and unhighlight the matching link when component is hovered', async () => {
     document.body.innerHTML = '<p>This is a sentence with an injection.</p>'
 
-    injectLinksInDocument(Array.from(document.querySelectorAll('p')), () => null, linkInjections)
+    injectLinksInDocument(Array.from(document.querySelectorAll('p')), linkInjections)
 
     const { getAllByText, container } = render(EditorItem, { linkInjection })
 
@@ -41,7 +41,7 @@ describe('EditorItem.svelte', () => {
     document.body.innerHTML = '<p>This is a sentence with an injection.</p>'
 
     const linkInjectionWithAfterArticle = { ...linkInjection, in_text: true, after_article: true }
-    injectLinksInDocument(Array.from(document.querySelectorAll('p')), () => null, { aiInjections: [linkInjectionWithAfterArticle], manualInjections: [] })
+    injectLinksInDocument(Array.from(document.querySelectorAll('p')), { aiInjections: [linkInjectionWithAfterArticle], manualInjections: [] })
 
     const { container } = render(EditorItem, { linkInjection })
 
@@ -54,7 +54,7 @@ describe('EditorItem.svelte', () => {
 
     const failedInjection = generateInjection('This is a sentence', 'fail')
 
-    injectLinksInDocument(Array.from(document.querySelectorAll('p')), () => null, { aiInjections: [], manualInjections: [failedInjection] })
+    injectLinksInDocument(Array.from(document.querySelectorAll('p')), { aiInjections: [], manualInjections: [failedInjection] })
 
     const { container } = render(EditorItem, { linkInjection })
 
@@ -65,7 +65,7 @@ describe('EditorItem.svelte', () => {
   it('Should scroll matching link into view when component is clicked', async () => {
     document.body.innerHTML = '<p>This is a sentence with an injection.</p>'
 
-    injectLinksInDocument(Array.from(document.querySelectorAll('p')), () => null, linkInjections)
+    injectLinksInDocument(Array.from(document.querySelectorAll('p')), linkInjections)
 
     const { container } = render(EditorItem, { linkInjection })
 
@@ -76,7 +76,7 @@ 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 = '<main><p>This has no matching injections.</p></main>'
 
-    injectLinksInDocument(Array.from(document.querySelectorAll('main p')), () => null, linkInjections)
+    injectLinksInDocument(Array.from(document.querySelectorAll('main p')), linkInjections)
 
     const { container } = render(EditorItem, { linkInjection })
 
@@ -87,7 +87,7 @@ describe('EditorItem.svelte', () => {
   it('Should not scroll matching link into view when component is clicked on button or input', async () => {
     document.body.innerHTML = '<p>This is a sentence with an injection.</p>'
 
-    injectLinksInDocument(Array.from(document.querySelectorAll('p')), () => null, linkInjections)
+    injectLinksInDocument(Array.from(document.querySelectorAll('p')), linkInjections)
 
     const { container } = render(EditorItem, { linkInjection })
 
@@ -101,7 +101,7 @@ describe('EditorItem.svelte', () => {
   it('Should highlight element in editor when hovering element on page', async () => {
     document.body.innerHTML = '<p>This is a sentence with an injection.</p>'
 
-    injectLinksInDocument(Array.from(document.querySelectorAll('p')), () => null, linkInjections)
+    injectLinksInDocument(Array.from(document.querySelectorAll('p')), linkInjections)
 
     await new Promise(res => setTimeout(res))