Problem
Most E2E tests in tests/e2e/ accept multiple status codes including 500:
expect([200, 401, 500]).toContain(response.statusCode)
This means tests can pass even when the feature is completely broken, as long as the server doesn't crash. For example, a test expecting user data returns 500 (DB unavailable) and the test still passes.
Fix
Remove 500 from expected status codes. Tests should fail when the server returns an error.
Severity
Medium — false-positive test results.
Problem
Most E2E tests in
tests/e2e/accept multiple status codes including 500:This means tests can pass even when the feature is completely broken, as long as the server doesn't crash. For example, a test expecting user data returns 500 (DB unavailable) and the test still passes.
Fix
Remove 500 from expected status codes. Tests should fail when the server returns an error.
Severity
Medium — false-positive test results.