-
Notifications
You must be signed in to change notification settings - Fork 11
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
BoxesView: Implement Loading/Error Handling #1094
Conversation
vahidbazzaz
commented
Nov 3, 2023
- implement error handling
- add test case 4.8.1
Codecov Report
@@ Coverage Diff @@
## master #1094 +/- ##
==========================================
+ Coverage 71.75% 73.05% +1.29%
==========================================
Files 289 289
Lines 7428 7429 +1
Branches 1560 1561 +1
==========================================
+ Hits 5330 5427 +97
+ Misses 2057 1960 -97
- Partials 41 42 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
// Test case 4.8.1.3 | ||
const row = await screen.findByRole("row", { | ||
// eslint-disable-next-line max-len | ||
name: /toggle all rows selected toggle sortby toggle sortby toggle sortby toggle sortby toggle sortby toggle sortby toggle sortby toggle sortby/i, |
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.
lol @vahidbazzaz what's that?
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.
@HaGuesto I will change it later, but I got this quick selector using the testing library chrome plugin :)