fix(explore): filter popover opening after removing a filter

[kgabryje]

SUMMARY

Fixes adhoc filter popover opening when user has 2 or more filters added and removes 1 of them.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before: see linked issue
After:
https://user-images.githubusercontent.com/15073128/127877801-05e69c2c-8769-4dea-8d05-3e6f573962da.mov

TESTING INSTRUCTIONS

0. Enable drag and drop feature flag
1. Add 2 or more adhoc filters
2. Remove 1 adhoc filter
3. Verify that the popover doesn't open unnecessarily

ADDITIONAL INFORMATION

• Has associated issue: closes [drag&drop] Removing filter option will open popover for the next  #16019
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

CC @junlincc @jinghua-qa

[codecov]

Codecov Report

Merging #16024 (a5ec877) into master (6095978) will decrease coverage by 0.10%.
The diff coverage is 56.13%.

❗ Current head a5ec877 differs from pull request most recent head 6832532. Consider uploading reports for the commit 6832532 to get more accurate results

@@            Coverage Diff             @@
##           master   #16024      +/-   ##
==========================================
- Coverage   76.79%   76.69%   -0.11%     
==========================================
  Files         990      995       +5     
  Lines       52517    52777     +260     
  Branches     6642     6691      +49     
==========================================
+ Hits        40330    40475     +145     
- Misses      11963    12077     +114     
- Partials      224      225       +1     

Flag	Coverage Δ
javascript	71.33% <56.13%> (-0.16%)	⬇️

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

Impacted Files	Coverage Δ
...ntend/src/dashboard/containers/DashboardHeader.jsx	100.00% <ø> (ø)
...ntend/src/explore/components/ExploreChartPanel.jsx	15.00% <ø> (ø)
...nd/src/explore/components/ExploreViewContainer.jsx	2.11% <0.00%> (ø)
superset-frontend/src/explore/reducers/index.js	0.00% <ø> (ø)
superset/reports/api.py	87.78% <ø> (ø)
superset-frontend/src/reports/actions/reports.js	21.73% <21.73%> (ø)
.../ReportModal/HeaderReportActionsDropdown/index.tsx	25.00% <25.00%> (ø)
superset-frontend/src/reports/reducers/reports.js	25.00% <25.00%> (ø)
...tend/src/explore/components/ExploreChartHeader.jsx	60.29% <54.83%> (-2.21%)	⬇️
...frontend/src/dashboard/components/Header/index.jsx	65.47% <63.33%> (-0.96%)	⬇️
... and 11 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 6095978...6832532. Read the comment docs.

[junlincc]

/testenv up ENABLE_EXPLORE_DRAG_AND_DROP=true

[github-actions]

@junlincc Ephemeral environment spinning up at http://54.190.43.140:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[github-actions]

Ephemeral environment shutdown and build artifacts deleted.