Skip to content

Conversation

@michael-s-molina
Copy link
Member

SUMMARY

Adds tests to the OptionControls component.

TEST PLAN

1 - Execute OptionControls tests
2 - All tests should pass

@rusackas @junlincc

ADDITIONAL INFORMATION

  • Has associated issue:
  • Changes UI
  • Requires DB Migration.
  • Confirm DB Migration upgrade and downgrade tested.
  • Introduces new feature or API
  • Removes existing feature or API

@codecov
Copy link

codecov bot commented Mar 22, 2021

Codecov Report

Merging #13729 (efd2ad8) into master (6c3bfe8) will decrease coverage by 2.31%.
The diff coverage is 100.00%.

❗ Current head efd2ad8 differs from pull request most recent head fe3e92f. Consider uploading reports for the commit fe3e92f to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master   #13729      +/-   ##
==========================================
- Coverage   75.54%   73.22%   -2.32%     
==========================================
  Files         928      615     -313     
  Lines       47023    21874   -25149     
  Branches     5722     5812      +90     
==========================================
- Hits        35523    16017   -19506     
+ Misses      11315     5713    -5602     
+ Partials      185      144      -41     
Flag Coverage Δ
cypress 56.45% <ø> (+12.77%) ⬆️
hive ?
javascript 63.24% <100.00%> (+0.10%) ⬆️
mysql ?
postgres ?
presto ?
python ?
sqlite ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...controls/DndColumnSelectControl/DndSelectLabel.tsx 35.00% <ø> (ø)
...trols/DndColumnSelectControl/components/Option.tsx 50.00% <ø> (ø)
...ents/controls/FilterControl/AdhocFilterControl.jsx 60.00% <ø> (ø)
...nents/controls/FilterControl/AdhocFilterOption.jsx 100.00% <ø> (ø)
...nents/controls/MetricControl/AdhocMetricOption.jsx 100.00% <ø> (+27.27%) ⬆️
...mponents/controls/MetricControl/MetricsControl.jsx 90.05% <ø> (+5.26%) ⬆️
...plore/components/controls/OptionControls/index.tsx 90.42% <ø> (ø)
...ndColumnSelectControl/components/OptionWrapper.tsx 15.15% <100.00%> (ø)
superset/sql_validators/__init__.py
superset/reports/notifications/slack.py
... and 421 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6c3bfe8...fe3e92f. Read the comment docs.

Copy link
Member

@geido geido left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Member

@rusackas rusackas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🎉

@rusackas rusackas merged commit 6fd62e3 into apache:master Apr 1, 2021
amitmiran137 pushed a commit that referenced this pull request Apr 2, 2021
* 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)
  ...
lyndsiWilliams pushed a commit to preset-io/superset that referenced this pull request Apr 7, 2021
allanco91 pushed a commit to allanco91/superset that referenced this pull request May 21, 2021
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.2.0 First shipped in 1.2.0 labels Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/L test:component 🚢 1.2.0 First shipped in 1.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants