-
Notifications
You must be signed in to change notification settings - Fork 109
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CLIENT-2476 - Disallow use of add-ops in foreground scans/queries #552
base: dev
Are you sure you want to change the base?
Conversation
CLIENT-2476 Disallowed use of operations with foreground queries (foreach and results) Added sufficent testing for each error condition
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## stage #552 +/- ##
==========================================
+ Coverage 81.25% 81.29% +0.04%
==========================================
Files 99 99
Lines 14897 14913 +16
==========================================
+ Hits 12105 12124 +19
+ Misses 2792 2789 -3 ☔ View full report in Codecov by Sentry. |
Disallowed use of ops with scan.results and scan.foreach Replaced test cases with the corect test cases. Fixed lint issuess
You should also specify in the documentation for query.add_ops() that it should not be used for foreground queries (i.e when calling |
Co-authored-by: juliannguyen4 <109386615+juliannguyen4@users.noreply.github.com>
Co-authored-by: juliannguyen4 <109386615+juliannguyen4@users.noreply.github.com>
Co-authored-by: juliannguyen4 <109386615+juliannguyen4@users.noreply.github.com>
I will add documentation to explain this and let you know when it has been finished. |
Co-authored-by: juliannguyen4 <109386615+juliannguyen4@users.noreply.github.com>
CLIENT-2476
Disallowed use of operations with foreground queries (foreach and results) Added sufficent testing for each error condition