set to stage#1434
Conversation
Greptile SummaryThis PR replaces the production WebSocket realtime endpoint (
Confidence Score: 1/5Not safe to merge — all changes are accidental staging configuration that must be reverted. Every changed line in this PR replaces a production endpoint with a staging URL in test files. This will break CI for other contributors whenever the staging server is unavailable and leaks an internal hostname into the public repo. All four files: tests/languages/android/Tests.kt, tests/languages/apple/Tests.swift, tests/languages/flutter/tests.dart, tests/languages/web/index.html
|
| Filename | Overview |
|---|---|
| tests/languages/android/Tests.kt | Realtime endpoint changed from production to staging URL — should not be merged as-is. |
| tests/languages/apple/Tests.swift | Realtime endpoint changed from production to staging URL — same issue as Android test. |
| tests/languages/flutter/tests.dart | Realtime endpoint changed from production to staging URL — same issue across all test files. |
| tests/languages/web/index.html | Realtime endpoint changed from production to staging URL — same issue across all test files. |
Reviews (1): Last reviewed commit: "set to stage" | Re-trigger Greptile
| // reset configs | ||
| client.setProject("console") | ||
| .setEndpointRealtime("wss://cloud.appwrite.io/v1") | ||
| .setEndpointRealtime("wss://fra.stage.cloud.appwrite.io/v1") |
There was a problem hiding this comment.
Staging URL committed to test file
This replaces the production endpoint with a staging URL (wss://fra.stage.cloud.appwrite.io/v1). Test files in this repo should point to the canonical production endpoint; the staging server may be unavailable in CI, causing non-deterministic test failures for all contributors.
| .setEndpointRealtime("wss://fra.stage.cloud.appwrite.io/v1") | |
| .setEndpointRealtime("wss://cloud.appwrite.io/v1") |
| @@ -34,7 +34,7 @@ class Tests: XCTestCase { | |||
|
|
|||
| // reset configs | |||
| client.setSelfSigned(); | ||
| client.setProject('console'); | ||
| client.setEndPointRealtime("wss://cloud.appwrite.io/v1"); | ||
| client.setEndPointRealtime("wss://fra.stage.cloud.appwrite.io/v1"); |
| // Realtime setup | ||
| client.setProject('console'); | ||
| client.setEndpointRealtime('wss://cloud.appwrite.io/v1'); | ||
| client.setEndpointRealtime('wss://fra.stage.cloud.appwrite.io/v1'); |
What does this PR do?
(Provide a description of what this PR does.)
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)
Related PRs and Issues
(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)
Have you read the Contributing Guidelines on issues?
(Write your answer here.)