Closed maximilianoertel closed 5 days ago
Status | Category | Percentage | Covered / Total |
---|---|---|---|
šµ | Lines | 5.1% | 390 / 7637 |
šµ | Statements | 4.99% | 421 / 8423 |
šµ | Functions | 6.13% | 115 / 1876 |
šµ | Branches | 2.41% | 110 / 4563 |
Visit the preview URL for this PR (updated for commit 1522135):
https://roar-staging--pr867-ref-318-query-compos-t79uux7m.web.app
(expires Thu, 17 Oct 2024 22:02:25 GMT)
š„ via Firebase Hosting GitHub Action š
Sign: 2631e9c58fd0104ecbfddd72a62245ddac467460
Passed #8171
•
1522135dc8: Component Tests for PR 867 "Fix error handling in `fetchDocById` helper" from co...
Project |
roar-dashboard-e2e
|
Run status |
Passed #8171
|
Run duration | 02m 01s |
Commit |
1522135dc8: Component Tests for PR 867 "Fix error handling in `fetchDocById` helper" from co...
|
Committer | Maximilian Oertel |
View all properties for this run āļø |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
17
|
Proposed changes
This PR removes the
swallowErrors
parameter on thefetchByDocId
helper to, well, prevent errors from being swallowed. Doing so and allowing the helper to throw any errors, enables any TanStack query using this helper function to properly handle error cases and trigger automatic retries.As this parameter was only used in the
validateCode
method, this PR also slightly refactors that function to handle thrown errors.Types of changes
Checklist
Justification of missing checklist items
n/a
Further comments
n/a
Ref https://github.com/yeatmanlab/roar/issues/318