Skip to content

Fix test expectations after IngressNetworksMap log level change#782

Merged
lucaslorentz merged 1 commit intomasterfrom
fix/ingress-networks-map-log-test
Mar 11, 2026
Merged

Fix test expectations after IngressNetworksMap log level change#782
lucaslorentz merged 1 commit intomasterfrom
fix/ingress-networks-map-log-test

Conversation

@lucaslorentz
Copy link
Owner

Summary

  • PR Move IngressNetworksMap log message level from info to debug. #777 moved IngressNetworksMap log from INFO to DEBUG, but didn't update test expectations
  • Tests assert on INFO-level log output, so the now-DEBUG line no longer appears
  • Removes the ingressNetworksMapLog and otherIngressNetworksMapLog constants and updates all affected tests

Test plan

  • All tests pass (go test ./...)

🤖 Generated with Claude Code

PR #777 moved IngressNetworksMap from INFO to DEBUG, but the tests
still expected it in the INFO-level log output. Remove the now-invisible
log line from test expectations.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@lucaslorentz lucaslorentz merged commit 89dc140 into master Mar 11, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant