-
Notifications
You must be signed in to change notification settings - Fork 14k
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: Adds tests to the CustomFrame component #13675
test: Adds tests to the CustomFrame component #13675
Conversation
Codecov Report
@@ Coverage Diff @@
## master #13675 +/- ##
==========================================
+ Coverage 77.41% 77.53% +0.11%
==========================================
Files 918 933 +15
Lines 46707 47186 +479
Branches 5720 5861 +141
==========================================
+ Hits 36158 36585 +427
- Misses 10413 10461 +48
- Partials 136 140 +4
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! with just a microscopic nit
...t-frontend/src/explore/components/controls/DateFilterControl/components/CustomFrame.test.tsx
Outdated
Show resolved
Hide resolved
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.
Looks awesome, thank you!
* master: (26 commits) chore: bump to new superset-ui version (#13932) fix: do not run containers as root by default in Helm chart (#13917) feat(explore): adhoc column formatting for Table chart (#13758) fix(sqla-query): order by aggregations in Presto and Hive (#13739) feat(alert/report): add ALERTS_ATTACH_REPORTS feature flags + feature (#13894) test: Fixes PropertiesModal_spec (#13548) fix: Pin Prophet dependency after breaking changes (#13852) test: Adds tests to dnd controls (#13650) test: Adds tests to the AnnotationLayer component (#13748) test: Refactor and enhance tests for the Explore DatasourcePanel Component (#13799) Add tests (#13778) test: DisplayQueryButton (#13750) Fixing condition around left margin for dashboard layout. Fixes #13863 (#13905) Revert "fix: select table overlay (#13694)" (#13901) test: Adds tests to the OptionControls component (#13729) test: DatasourceControl (#13605) tests for function handleScroll (#13896) test: Adds tests to the CustomFrame component (#13675) test: Adds tests to the AdvancedFrame component (#13664) test: DataTableControl (#13668) ...
* test: Adds tests to the CustomFrame component * Replaces full path with relative path
* test: Adds tests to the CustomFrame component * Replaces full path with relative path
SUMMARY
Adds tests to the
CustomFrame
component.TEST PLAN
1 - Execute
CustomFrame
tests.2 - All tests should pass.
@rusackas @junlincc
ADDITIONAL INFORMATION