@playpilot/tpi 8.36.1 → 8.36.2

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@playpilot/tpi",
-	"version": "8.36.1",
+	"version": "8.36.2",
 	"type": "module",
 	"scripts": {
 		"dev": "vite dev",

package/src/routes/components/Participants/Participant.svelte

@@ -77,7 +77,7 @@
     <div class="jobs">
       {#each ['all', ...participant.jobs] as job}
         <Button variant="border" size="large" active={currentJob === job} onclick={() => currentJob = job as 'all' | Job}>
-          {t('Job: ' + job)}
+          {t('Job: ' + job).replace('Job: ', '')}
         </Button>
       {/each}
     </div>
@@ -186,5 +186,9 @@
     flex-wrap: wrap;
     gap: margin(0.5);
     margin-bottom: margin(0.5);
+
+    :global(.button) {
+      text-transform: capitalize;
+    }
   }
 </style>

package/src/tests/routes/components/Participants/Participant.test.js

@@ -90,6 +90,16 @@ describe('Participant.svelte', () => {
     })
   })
 
+  it('Should render jobs without translations without prefix', async () => {
+    const resolved = ({ results: Array(30).fill(title), next: 'next', previous: 'previous' })
+    vi.mocked(fetchTitles).mockResolvedValueOnce(resolved)
+
+    // @ts-ignore
+    const { getByText } = render(Participant, { participant: { ...participants[0], jobs: ['unknown'] } })
+
+    expect(getByText('unknown')).toBeTruthy()
+  })
+
   it('Should include selected sorting specific to participants', async () => {
     const resolved = ({ results: Array(30).fill(title), next: 'next', previous: 'previous' })
     vi.mocked(fetchTitles).mockResolvedValueOnce(resolved)

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

@@ -6,7 +6,7 @@ import { openModal } from '$lib/modal'
 import { participants, title } from '$lib/fakeData'
 import { track } from '$lib/tracking'
 import { TrackingEvent } from '$lib/enums/TrackingEvent'
-import { fetchParticipantsForTitle } from '$lib/api/participants'
+import { fetchParticipantBySid, fetchParticipantsForTitle } from '$lib/api/participants'
 
 vi.mock('$lib/modal', () => ({
   openModal: vi.fn(),
@@ -18,6 +18,7 @@ vi.mock('$lib/tracking', () => ({
 
 vi.mock('$lib/api/participants', () => ({
   fetchParticipantsForTitle: vi.fn(),
+  fetchParticipantBySid: vi.fn(),
 }))
 
 describe('ParticipantsRail.svelte', () => {
@@ -76,8 +77,9 @@ describe('ParticipantsRail.svelte', () => {
     })
   })
 
-  it('Should open modal on click of participant', async () => {
+  it('Should fetch participant and open modal on click of participant', async () => {
     vi.mocked(fetchParticipantsForTitle).mockResolvedValueOnce(participants)
+    vi.mocked(fetchParticipantBySid).mockResolvedValueOnce(participants[0])
 
     const { getAllByTestId } = render(ParticipantsRail, { title })
 
@@ -85,7 +87,10 @@ describe('ParticipantsRail.svelte', () => {
       await fireEvent.click(getAllByTestId('participant')[0])
     })
 
-    expect(openModal).toHaveBeenCalledWith({ event: expect.any(Object), type: 'participant', data: participants[0] })
+    await waitFor(async () => {
+      expect(fetchParticipantBySid).toHaveBeenCalledWith(participants[0].sid)
+      expect(openModal).toHaveBeenCalledWith({ event: expect.any(Object), type: 'participant', data: participants[0] })
+    })
   })
 
   it('Should not open modal on click if if open_tpi_links_in_explore is true and instead set url', async () => {