Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix(clear-all): apply excludeAttribute correctly with clearsQuery (#2935
) * fix(clear-all): apply excludeAttribute correctly This fix actually contains a rewrite of the underlying functions that will compute the list of attributes to clear. The complete required behaviour was spread both in the connectors and the utility functions. In order to simplify the code and make it easier to test, this refactoring was needed. A next step for this is to use a single connnector for current refined values and clear all widgets. * refactor(connectClearAll-test): move to jest * refactor(connectCurrentRefinedValues-test): move to jest * refactor(utils): use [].forEach
- Loading branch information
Showing
8 changed files
with
579 additions
and
490 deletions.
There are no files selected for viewing
261 changes: 228 additions & 33 deletions
261
src/connectors/clear-all/__tests__/connectClearAll-test.js
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.