sadpandajoe opened a new pull request, #40778:
URL: https://github.com/apache/superset/pull/40778

   ### SUMMARY
   
   Migrate the previously-skipped Cypress native-filter scenarios (formerly 
`_skip.nativeFilters.test.ts` and `_skip.nativeFilters.noInitState.test.ts` on 
the `6.1` branch) to Jest + React Testing Library so the dashboard 
native-filter UI keeps component-level coverage after the Cypress base is 
removed.
   
   The new tests sit alongside the existing suites for `FiltersConfigModal`, 
`NativeFiltersModal`, and `FilterBar`. Scope is deliberately limited to modal- 
and filter-bar-local behavior; cross-filter chart interaction and live 
numerical-range widget rendering remain out of scope and are covered by the 
parallel Playwright migration.
   
   New tests:
   - `FiltersConfigModal`: restore a deleted filter, undo a deletion via the 
sidebar "Undo?" link, value-filter info tooltips, numerical-range Range Type 
selector display-mode change, "Filter has default value" toggle reveals / hides 
the Default Value control
   - `NativeFiltersModal`: the confirm-cancel button on the unsaved-changes 
alert proceeds with cancel
   - `FilterBar`: the configured filter name renders in the bar; clicking the 
gear menu's "Add or edit filters and controls" item opens the 
`FiltersConfigModal`
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   
   N/A — test-only.
   
   ### TESTING INSTRUCTIONS
   
   From `superset-frontend/`:
   
   ```
   npx jest src/dashboard/components/nativeFilters
   ```
   
   Expected: all suites green (1 pre-existing unrelated skip).
   
   ### ADDITIONAL INFORMATION
   
   - [ ] Has associated issue:
   - [ ] Required feature flags:
   - [ ] Changes UI
   - [ ] Includes DB Migration
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to