Skip to content

Commit

Permalink
Fix up the failing test for favicon
Browse files Browse the repository at this point in the history
  • Loading branch information
gauntface authored and renovate[bot] committed Dec 23, 2023
1 parent 5df1afe commit 1f00268
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions test/browser-tests.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,16 @@ test.beforeEach(async (t) => {

// Ensure we get 200 responses from the server
t.context.page.on('response', (response) => {
const url = response.url();
if (url.endsWith("/favicon.ico")) {
// We don't care about favicons in tests
return;
}

if (response) {
if (response.status() !== 200) {
console.error(`Non-200 response: (${response.status()}) ${url}`);
}
t.deepEqual(response.status(), 200);
}
});
Expand Down

1 comment on commit 1f00268

@vercel
Copy link

@vercel vercel bot commented on 1f00268 Dec 23, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.