-
Notifications
You must be signed in to change notification settings - Fork 14.3k
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
test: RTL overhaul - hackathon #16626
Conversation
Codecov Report
@@ Coverage Diff @@
## master #16626 +/- ##
==========================================
+ Coverage 76.71% 76.86% +0.14%
==========================================
Files 1002 1004 +2
Lines 53801 53970 +169
Branches 6858 7374 +516
==========================================
+ Hits 41276 41482 +206
+ Misses 12288 12248 -40
- Partials 237 240 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
lgtm, thanks for doing the jsx => tsx conversion here too! I really appreciate the improved test quality here
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.
This is great, thank you!
* CachedLabel_spec fully converted to RTL * ColumnTypeLabel_spec fully converted to RTL * MissingChart_spec fully converted to RTL * RefreshIntervalModal_spec mostly converted to RTL * HoverMenu_spec mostly converted to RTL * ResizableContainer_spec fully converted to RTL * ResizableHandle_spec fully converted to RTL * AggregateOption_spec fully converted to RTL * CheckboxControl_spec fully converted to RTL * ColorPickerControl_spec to RTL * Finished converting ColorPickerControl_spec to RTL/TS, also converted RefreshIntervalModal_spec to TS * Added unknown type to ColumnTypeLabelProps * Fixed ColumnTypeLabel_spec
* CachedLabel_spec fully converted to RTL * ColumnTypeLabel_spec fully converted to RTL * MissingChart_spec fully converted to RTL * RefreshIntervalModal_spec mostly converted to RTL * HoverMenu_spec mostly converted to RTL * ResizableContainer_spec fully converted to RTL * ResizableHandle_spec fully converted to RTL * AggregateOption_spec fully converted to RTL * CheckboxControl_spec fully converted to RTL * ColorPickerControl_spec to RTL * Finished converting ColorPickerControl_spec to RTL/TS, also converted RefreshIntervalModal_spec to TS * Added unknown type to ColumnTypeLabelProps * Fixed ColumnTypeLabel_spec
SUMMARY
Migrate Enzyme test files to React Testing library/Typescript and improve test case quality. See the project description doc for more details.
TESTING INSTRUCTIONS
Migrate to the following files and observe the tests converted to RTL/TS:
superset-frontend/spec/javascripts/components/CachedLabel_spec.tsx
superset-frontend/spec/javascripts/components/ColumnTypeLabel_spec.tsx
superset-frontend/spec/javascripts/dashboard/components/MissingChart_spec.tsx
superset-frontend/spec/javascripts/dashboard/components/RefreshIntervalModal_spec.tsx
superset-frontend/spec/javascripts/dashboard/components/menu/HoverMenu_spec.tsx
superset-frontend/spec/javascripts/dashboard/components/resizable/ResizableContainer_spec.tsx
superset-frontend/spec/javascripts/dashboard/components/resizable/ResizableHandle_spec.tsx
superset-frontend/spec/javascripts/explore/components/AggregateOption_spec.tsx
superset-frontend/spec/javascripts/explore/components/CheckboxControl_spec.tsx
superset-frontend/spec/javascripts/explore/components/ColorPickerControl_spec.tsx
ADDITIONAL INFORMATION