Skip to content

Commit 5d525d4

Browse files
committed
test: exclude leaderboard API requests from count filters
Add exclusion for '/api/v1/leaderboard-entries/for-current-user' in API request filtering logic within test helpers. This prevents counting requests triggered during header rendering, improving accuracy of API request count assertions in tests.
1 parent 860f90b commit 5d525d4

File tree

2 files changed

+16
-2
lines changed

2 files changed

+16
-2
lines changed

tests/support/api-requests-count.js

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,14 @@ export default function apiRequestsCount(server) {
99
const filteredRequests = server.pretender.handledRequests.filter((request) => {
1010
const pathname = new URL(request.url).pathname;
1111

12-
return pathname !== '/api/v1/analytics-events' && pathname !== '/api/v1/users/current';
12+
return (
13+
// Triggered on every pageview
14+
pathname !== '/api/v1/analytics-events' &&
15+
// Triggered on application boot
16+
pathname !== '/api/v1/users/current' &&
17+
// Triggered when header is rendered
18+
pathname !== '/api/v1/leaderboard-entries/for-current-user'
19+
);
1320
});
1421

1522
// Debugging

tests/support/verify-api-requests.js

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,14 @@ export default function verifyApiRequests(server, expectedRequests) {
55
const filteredRequests = requests.filter((request) => {
66
const pathname = new URL(request.url).pathname;
77

8-
return pathname !== '/api/v1/analytics-events' && pathname !== '/api/v1/users/current';
8+
return (
9+
// Triggered on every pageview
10+
pathname !== '/api/v1/analytics-events' &&
11+
// Triggered on application boot
12+
pathname !== '/api/v1/users/current' &&
13+
// Triggered when header is rendered
14+
pathname !== '/api/v1/leaderboard-entries/for-current-user'
15+
);
916
});
1017

1118
// Debugging

0 commit comments

Comments
 (0)