fix: the reports test name keeps its apostrophe inside double quotes, and the merge API test sends no unmonitor (M502 #5525)
release / govulncheck (push) Successful in 16s
release / web (push) Successful in 1m19s
release / go (push) Successful in 1m35s
release / integration (push) Successful in 5m12s
release / android (push) Successful in 5m18s
release / Build signed APK (releases and dev) (push) Successful in 5m34s
release / Attach APK to the Release (tag releases only) (push) Skipped
release / Build + push container image (push) Successful in 25s
release / Verify release artifacts (tag releases only) (push) Skipped

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
2026-10-11 13:55:55 -04:00
co-authored by Claude Opus 5.5
parent 3a16732883
commit 12725e71b0
2 changed files with 3 additions and 4 deletions
+2 -3
View File
@@ -55,10 +55,9 @@ describe('admin duplicates API', () => {
it('mergeDuplicateGroup POSTs the chosen survivor', async () => {
(api.post as unknown as ReturnType<typeof vi.fn>).mockResolvedValueOnce({ removed_paths: [] });
await mergeDuplicateGroup('g-1', { survivor_track_id: 't-1', unmonitor: true });
await mergeDuplicateGroup('g-1', { survivor_track_id: 't-1' });
expect(api.post).toHaveBeenCalledWith('/api/admin/library/duplicates/g-1/merge', {
survivor_track_id: 't-1',
unmonitor: true
survivor_track_id: 't-1'
});
});
});
+1 -1
View File
@@ -97,7 +97,7 @@ describe('/admin/reports', () => {
expect(resolvePlaybackError).not.toHaveBeenCalled();
});
test('on a phone the row's other actions are in its menu', async () => {
test("on a phone the row's other actions are in its menu", async () => {
setup([flagged], [failure('e2', 't2', 'Intro', 2)]);
const roygbiv = screen.getAllByTestId('report-row').find((r) => r.textContent?.includes('Roygbiv'))!;
await fireEvent.click(within(roygbiv).getByRole('button', { name: /more actions/i }));