Skip to content

Commit 8dac4bf

Browse files
SkArchonclaude
andcommitted
chore: add diagnostic logging for circuit breaker upgrade test CI failure
The test passes locally but fails consistently on CI with "Internal server error". Diagnostics confirmed the circuit breaker works correctly (employeesCalls=4, status changed=2), so the issue is in the WebSocket communication layer after the HTTP 101 upgrade succeeds. Adding WARN/ERROR log capture to identify the exact subscription resolution error on CI. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent dfb90dd commit 8dac4bf

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

router-tests/security/circuit_breaker_test.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -753,6 +753,14 @@ func TestCircuitBreaker(t *testing.T) {
753753
t.Logf("received message: %s", string(message))
754754
t.Logf("circuit breaker status changed count: %d", xEnv.Observer().FilterMessage("Circuit breaker status changed").Len())
755755
t.Logf("circuit breaker open count: %d", xEnv.Observer().FilterMessage("Circuit breaker open, request callback did not execute").Len())
756+
for _, entry := range xEnv.Observer().FilterMessage("Resolving GraphQL subscription").All() {
757+
t.Logf("subscription resolve error: %v", entry.ContextMap())
758+
}
759+
for _, entry := range xEnv.Observer().All() {
760+
if entry.Level >= zapcore.WarnLevel {
761+
t.Logf("WARN/ERROR log: %s | %v", entry.Message, entry.ContextMap())
762+
}
763+
}
756764

757765
require.JSONEq(t, timestampMessage, string(message))
758766

0 commit comments

Comments
 (0)