fix(test): remove orphaned gock mock in TestUpdateRemoteConfig by lightstrike ¡ Pull Request #4913 ¡ supabase/cli
No actionable comments were generated in the recent review. đ
âšī¸ Recent review info
Configuration used: Central YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Pro
Cache: Disabled due to Reviews > Disable Cache setting
Disabled knowledge base sources:
- Linear integration is disabled
You can enable these sources in your CodeRabbit configuration.
đ Files selected for processing (1)
pkg/config/updater_test.go
đ¤ Files with no reviewable changes (1)
- pkg/config/updater_test.go
đ Walkthrough
Summary by CodeRabbit
- Tests
- Updated configuration update test to streamline mock API requests while maintaining validation of remaining flows.
Walkthrough
The test file pkg/config/updater_test.go was modified to remove mock expectations for a network-restrictions API call from the TestUpdateRemoteConfig test. Specifically, the test no longer expects a GET request to /v1/projects/test-project/network-restrictions with a subsequent empty JSON response. The remaining test assertions for postgrest, database, auth, storage, and experimental API flows remain unchanged.
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands and usage tips.