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.

đŸ“Ĩ Commits

Reviewing files that changed from the base of the PR and between 5726ec3 and e7407ed.

📒 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.

â¤ī¸ Share

Comment @coderabbitai help to get the list of available commands and usage tips.