-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
test(query-core): add test case for query #9145
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
base: main
Are you sure you want to change the base?
Conversation
View your CI Pipeline Execution ↗ for commit a0d2c27.
☁️ Nx Cloud last updated this comment at |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9145 +/- ##
===========================================
+ Coverage 44.65% 58.87% +14.22%
===========================================
Files 206 138 -68
Lines 8219 5464 -2755
Branches 1845 1453 -392
===========================================
- Hits 3670 3217 -453
+ Misses 4103 1944 -2159
+ Partials 446 303 -143 🚀 New features to boost your workflow:
|
expect(query.state.data).toBe('persisted data') | ||
}) | ||
|
||
test('should use queryFn from observer if not provided in options', async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we could also assert here that query.options.queryFn
is set to the one of the observer at the end.
I improved test coverage of
query
Before
After