Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. WalkthroughThis PR performs a version downgrade from 21.3.0 to 21.2.0 across pubspec.yaml, README, and SDK version headers. It adds upsert() as a terminal action to Channel extensions for documents and rows, introduces containsAny() and containsAll() query builders with enhanced documentation for the contains() method, removes githubImagine and googleImagine enum variants from OAuthProvider, and updates the compatibility note to reference Appwrite 1.8.x. A corresponding test case validates the new upsert() routing behavior. Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
lib/channel.dart (1)
139-144: Missing test coverage forRowChannel.upsert().
upsert()was added to bothDocumentChannelandRowChannel, but the test file only covers the document path. Consider adding a corresponding test for the row channel (e.g.,Channel.tablesdb('db1').table('table1').row('row1').upsert()) to maintain parity with the document test.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@lib/channel.dart` around lines 139 - 144, The test suite is missing coverage for RowChannel.upsert(); add a unit test that mirrors the document-path upsert test but targets the row channel, e.g. invoke Channel.tablesdb('db1').table('table1').row('row1').upsert() and assert the same behaviors/assertions used in the DocumentChannel upsert test (request payload, endpoint, and response handling). Locate the existing DocumentChannel upsert test and duplicate its structure, replacing the document-channel construction with Channel.tablesdb(...).table(...).row(...) and referencing RowChannel.upsert() to verify parity.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@lib/channel.dart`:
- Around line 139-144: The test suite is missing coverage for
RowChannel.upsert(); add a unit test that mirrors the document-path upsert test
but targets the row channel, e.g. invoke
Channel.tablesdb('db1').table('table1').row('row1').upsert() and assert the same
behaviors/assertions used in the DocumentChannel upsert test (request payload,
endpoint, and response handling). Locate the existing DocumentChannel upsert
test and duplicate its structure, replacing the document-channel construction
with Channel.tablesdb(...).table(...).row(...) and referencing
RowChannel.upsert() to verify parity.
This PR contains updates to the Flutter SDK for version 21.4.0 .