The tab list is pinned by name and order, so adding Missing files failed the assertion. That is the test doing its job: the nav is a deliberate ordering, not an accident, and a new entry should have to be declared rather than slipping in.
This commit is contained in:
@@ -52,7 +52,7 @@ describe('AdminTabs', () => {
|
|||||||
);
|
);
|
||||||
});
|
});
|
||||||
|
|
||||||
test('renders all eight tabs in order', () => {
|
test('renders all nine tabs in order', () => {
|
||||||
state.pageUrl = new URL('http://localhost/admin');
|
state.pageUrl = new URL('http://localhost/admin');
|
||||||
render(AdminTabs);
|
render(AdminTabs);
|
||||||
const links = screen.getAllByRole('link');
|
const links = screen.getAllByRole('link');
|
||||||
@@ -61,6 +61,9 @@ describe('AdminTabs', () => {
|
|||||||
'Integrations',
|
'Integrations',
|
||||||
'Requests',
|
'Requests',
|
||||||
'Quarantine',
|
'Quarantine',
|
||||||
|
// Missing files sits with Quarantine and Playback errors: the three
|
||||||
|
// surfaces that show tracks needing an operator's attention.
|
||||||
|
'Missing files',
|
||||||
'Playback errors',
|
'Playback errors',
|
||||||
'Diagnostics',
|
'Diagnostics',
|
||||||
'Tuning',
|
'Tuning',
|
||||||
|
|||||||
Reference in New Issue
Block a user