Buck9195/doesitarm
1
0
Fork 0
mirror of https://github.com/ThatGuySam/doesitarm.git synced 2026-05-15 06:35:20 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Add support for updating existing filters

Browse source
This commit is contained in:
Sam Carlton 2022-06-04 19:49:36 -05:00
parent 3d8bd0d61d
commit 62401d1f76
2 changed files with 14 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
helpers/stork/browser.js
View file

@ -284,7 +284,7 @@ export class StorkFilters {
filterValue = filterValue || fromString.value
// If the filter is already set, remove it
if ( this.hasFilter( filterName ) ) {
if ( this.hasFilter( filterNameOrQueryValue ) ) {
this.removeFilter( filterName )
return

15
test/prebuild/filters.js
View file

@ -44,7 +44,18 @@ test('Can handle query values with multiple underscores', async t => {
t.assert( filters.hasFilter( 'test_value_with_multiple_underscores' ) , 'Has correct filters for query' )
})
// Can handle query values with multiple underscores
test( 'Can update existing filter', async t => {
const filters = new StorkFilters({
initialFilters: {
'test': 'works_no'
}
})
filters.toggleFilter('test_works_yes')
t.deepEqual( filters.filtersForQuery, 'test_works_yes', 'Has updated filter')
})
// Can update existing filter