@playpilot/tpi 8.14.0 → 8.15.1

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

@@ -6,6 +6,7 @@ import { mount, unmount } from 'svelte'
 import { fakeFetch, generateInjection } from '../helpers'
 import { openModalForInjectedLink } from '$lib/modal'
 import { getLinkInjectionElements } from '$lib/injectionElements'
+import { titleUrl } from '$lib/routes'
 
 vi.mock('svelte', () => ({
   mount: vi.fn(),
@@ -64,8 +65,9 @@ describe('injection.ts', () => {
 
       const link = /** @type {HTMLAnchorElement} */ (document.querySelector('a'))
 
+      // @ts-ignore
+      expect(link.href).toBe(titleUrl(injection.title_details))
       expect(link.innerText).toBe(injection.title)
-      expect(link.href).toBe(injection.playpilot_url)
     })
 
     it('Should replace given words as expected when more than 1 injection per sentence is present', () => {
@@ -83,11 +85,13 @@ describe('injection.ts', () => {
 
       const links = /** @type {HTMLAnchorElement[]} */ (Array.from(document.querySelectorAll('a')))
 
+      // @ts-ignore
+      expect(links[0].href).toBe(titleUrl(linkInjections[0].title_details))
       expect(links[0].innerText).toBe(linkInjections[0].title)
-      expect(links[0].href).toBe(linkInjections[0].playpilot_url)
 
+      // @ts-ignore
+      expect(links[1].href).toBe(titleUrl(linkInjections[1].title_details))
       expect(links[1].innerText).toBe(linkInjections[1].title)
-      expect(links[1].href).toBe(linkInjections[1].playpilot_url)
     })
 
     it('Should ignore injections that are marked as inactive', () => {
@@ -946,6 +950,43 @@ describe('injection.ts', () => {
 
       expect(document.querySelector('a')?.closest('[data-playpilot-injection-key]')).toBeTruthy()
     })
+
+    describe('config.open_tpi_links_in_explore', () => {
+      beforeEach(() => {
+        window.PlayPilotLinkInjections.config = {
+          open_tpi_links_in_explore: true,
+          explore_navigation_path: 'https://some-path.com/explore',
+        }
+      })
+
+      it('Should use href with explore links if open_tpi_links_in_explore is true', () => {
+        const injection = generateInjection('This is a sentence with an injection.', 'an injection')
+
+        document.body.innerHTML = `<p>${injection.sentence}</p>`
+
+        const elements = Array.from(document.querySelectorAll('p'))
+
+        injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+
+        const link = /** @type {HTMLAnchorElement} */ (document.querySelector('a'))
+        expect(link.href).toBe(window.PlayPilotLinkInjections.config.explore_navigation_path + `?route=modal&sid=${injection.title_details?.sid}`)
+        expect(link.target).not.toBeTruthy()
+      })
+
+      it('Should not open modal when link is clicked when open_tpi_links_in_explore is true', 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')
+
+        injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
+
+        const link = /** @type {HTMLAnchorElement} */ (document.querySelector('a'))
+        await fireEvent.click(link)
+
+        expect(openModalForInjectedLink).not.toHaveBeenCalled()
+      })
+    })
   })
 
   describe('clearLinkInjections', () => {

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

@@ -1,9 +1,10 @@
 import { describe, it, expect } from 'vitest'
-import { titleUrl } from '$lib/routes'
+import { exploreTitleUrl, titleUrl } from '$lib/routes'
 import { playPilotBaseUrl } from '$lib/constants'
+import { title } from '$lib/fakeData'
 
 describe('$lib/routes', () => {
-  describe('mergePlaylinks', () => {
+  describe('titleUrl', () => {
     it('Should return url for given title', () => {
       // @ts-ignore
       expect(titleUrl({ type: 'series', slug: 'some-slug' })).toBe(`${playPilotBaseUrl}/series/some-slug/`)
@@ -12,4 +13,15 @@ describe('$lib/routes', () => {
       expect(titleUrl({ type: 'movie', slug: 'some-other-slug' })).toBe(`${playPilotBaseUrl}/movie/some-other-slug/`)
     })
   })
+
+  describe('exploreTitleUrl', () => {
+    it('Should return url for given title', () => {
+      window.PlayPilotLinkInjections.config = {
+        open_tpi_links_in_explore: true,
+        explore_navigation_path: 'https://some-path.com/explore',
+      }
+
+      expect(exploreTitleUrl(title)).toBe(`https://some-path.com/explore?route=modal&sid=${title.sid}`)
+    })
+  })
 })

package/src/tests/routes/components/Explore/Routes/ExploreTitle.test.js

@@ -0,0 +1,87 @@
+import { render, waitFor, fireEvent } from '@testing-library/svelte'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+
+import TitleDetail from '../../../../../routes/components/Explore/Routes/ExploreTitle.svelte'
+import { fetchTitleBySid } from '$lib/api/titles'
+import { title } from '$lib/fakeData'
+
+vi.mock('$lib/api/titles', () => ({
+  fetchTitleBySid: vi.fn(),
+}))
+
+vi.mock('/src/routes/components/Title.svelte', () => ({
+  default: vi.fn(),
+}))
+
+describe('ExploreTitle.svelte', () => {
+  beforeEach(() => {
+    vi.resetAllMocks()
+    vi.mocked(fetchTitleBySid).mockResolvedValue(title)
+
+    history.pushState({}, '', '/')
+  })
+
+  it('Should render empty state when no sid is in the URL', async () => {
+    const { getByText } = render(TitleDetail)
+
+    await waitFor(() => {
+      expect(getByText('Page not found')).toBeTruthy()
+    })
+  })
+
+  it('Should call fetchTitleBySid with sid from URL', async () => {
+    history.pushState({}, '', '?sid=some-sid')
+
+    render(TitleDetail)
+
+    expect(fetchTitleBySid).toHaveBeenCalledWith('some-sid')
+  })
+
+  it('Should render loading state while fetching', async () => {
+    history.pushState({}, '', '?sid=some-sid')
+    vi.mocked(fetchTitleBySid).mockReturnValue(new Promise(() => {}))
+
+    const { getByText } = render(TitleDetail)
+
+    expect(getByText('Loading...')).toBeTruthy()
+  })
+
+  it('Should render the title when fetchTitleBySid resolves', async () => {
+    history.pushState({}, '', '?sid=some-sid')
+
+    const { getByTestId } = render(TitleDetail)
+
+    await waitFor(() => {
+      expect(getByTestId('title')).toBeTruthy()
+    })
+  })
+
+  it('Should render empty state when fetchTitleBySid rejects', async () => {
+    history.pushState({}, '', '?sid=some-sid')
+    vi.mocked(fetchTitleBySid).mockRejectedValue(new Error('Not found'))
+
+    const { getByText } = render(TitleDetail)
+
+    await waitFor(() => {
+      expect(getByText('Page not found')).toBeTruthy()
+    })
+  })
+
+  it('Should call navigate with "home" when back button is clicked', async () => {
+    history.pushState({}, '', '?sid=some-sid')
+
+    const navigate = vi.fn()
+
+    const { getByText } = render(TitleDetail, { navigate })
+
+    await fireEvent.click(getByText('Home'))
+
+    expect(navigate).toHaveBeenCalledWith('home')
+  })
+
+  it('Should not call fetchTitleBySid when no sid is in the URL', async () => {
+    render(TitleDetail)
+
+    expect(fetchTitleBySid).not.toHaveBeenCalled()
+  })
+})

package/src/tests/routes/components/Rails/TitlesRail.test.js

@@ -213,4 +213,14 @@ describe('TitlesRail.svelte', () => {
 
     expect(container.querySelector('.titles .slider')).toBeTruthy()
   })
+
+  it('Should navigate to explore page rather than open modal when navigateToExplore is true', async () => {
+    const { getAllByRole } = render(TitlesRail, { titles: [title], navigateToExplore: true })
+
+    await fireEvent.click(getAllByRole('link')[0])
+
+    expect(getAllByRole('link')[0].getAttribute('href')).toContain(`?route=modal&sid=${title.sid}`)
+    expect(window.location.href).toContain(`?route=modal&sid=${title.sid}`)
+    expect(openModal).not.toHaveBeenCalledWith()
+  })
 })