feat(facets): Unify __unknown__ and __unknown-facet__ in a constant

packages/x-components/src/components/decorators/injection.consts.ts

@@ -52,3 +52,10 @@ export const SELECTED_VARIANTS_KEY: XInjectKey<ResultVariant[]> = 'selectedVaria
 export const SELECT_RESULT_VARIANT_KEY: XInjectKey<
   (variant: ResultVariant, level?: number) => void
 > = 'selectResultVariant';
+
+/**
+ * It's used to identify filters without facetId.
+ *
+ * @internal
+ */
+export const FACET_KEY: string | number = '__unknown__';

packages/x-components/src/x-modules/facets/entities/filter-entity.factory.ts

@@ -1,6 +1,7 @@
 import { Facet, FilterModelName, Filter, isFacetFilter } from '@empathyco/x-types';
 import { Store } from 'vuex';
 import { RootXStoreState } from '../../../store/store.types';
+import { FACET_KEY } from '../../../components/index';
 import { EditableNumberRangeFilterEntity } from './editable-number-range-filter.entity';
 import { HierarchicalFilterEntity } from './hierarchical-filter.entity';
 import { NumberRangeFilterEntity } from './number-range-filter.entity';
@@ -49,7 +50,7 @@ export class FilterEntityFactory {
    *
    * @internal
    */
-  protected cache: Record<Facet['id'] | '__unknown-facet__', FilterEntity> = {};
+  protected cache: Record<Facet['id'] | typeof FACET_KEY, FilterEntity> = {};
 
   /**
    * Creates a new FilterEntity from a filter.
@@ -61,7 +62,7 @@ export class FilterEntityFactory {
    * @returns The {@link FilterEntity} created by the factory.
    */
   getFilterEntity(store: Store<RootXStoreState>, filter: Filter): FilterEntity {
-    const cacheKey = isFacetFilter(filter) ? filter.facetId : '__unknown-facet__';
+    const cacheKey = isFacetFilter(filter) ? filter.facetId : FACET_KEY;
     return this.cache[cacheKey] ?? (this.cache[cacheKey] = this.createFilterEntity(store, filter));
   }
 

packages/x-components/src/x-modules/facets/store/__tests__/getters.spec.ts

@@ -19,6 +19,7 @@ import { SafeStore } from '../../../../store/__tests__/utils';
 import { arrayToObject } from '../../../../utils/array';
 import { facetsXStoreModule } from '../module';
 import { FacetsActions, FacetsGetters, FacetsMutations, FacetsState } from '../types';
+import { FACET_KEY } from '../../../../components/index';
 import { resetFacetsStateWith } from './utils';
 
 describe('testing facets module getters', () => {
@@ -266,7 +267,7 @@ describe('testing facets module getters', () => {
         createNumberRangeFilter('price', { min: 25, max: 50 }, true),
         createEditableNumberRangeFilter('age', { min: null, max: 5 }),
         createEditableNumberRangeFilter('size', { min: null, max: null }),
-        // Raw filters don't belong to a facet so they will be under `__unknown-facet__` key
+        // Raw filters don't belong to a facet, so they will be under FACET_KEY key
         createRawFilter('size:xl')
       ]);
 
@@ -275,7 +276,7 @@ describe('testing facets module getters', () => {
         category: [store.state.filters['category:Shorts']],
         price: [store.state.filters['price:25-50']],
         age: [store.state.filters['age:*-5']],
-        ['__unknown-facet__']: [store.state.filters['size:xl']]
+        [FACET_KEY]: [store.state.filters['size:xl']]
       });
     });
 
@@ -326,7 +327,7 @@ describe('testing facets module getters', () => {
       const store = createFacetsStore([createRawFilter('size:xl')]);
 
       expect(store.getters.selectedFiltersByFacet).toEqual({
-        ['__unknown-facet__']: [store.state.filters['size:xl']]
+        [FACET_KEY]: [store.state.filters['size:xl']]
       });
     });
   });

packages/x-components/src/x-modules/facets/store/getters/facets.getter.ts

@@ -2,6 +2,7 @@ import { isFacetFilter } from '@empathyco/x-types';
 import { map } from '@empathyco/x-utils';
 import { groupItemsBy } from '../../../../utils/array';
 import { FacetsXStoreModule } from '../types';
+import { FACET_KEY } from '../../../../components/index';
 
 /**
  * Default implementation for the {@link FacetsGetters.facets} getter.
@@ -14,7 +15,7 @@ import { FacetsXStoreModule } from '../types';
  */
 export const facets: FacetsXStoreModule['getters']['facets'] = state => {
   const filtersByFacet = groupItemsBy(Object.values(state.filters), filter =>
-    isFacetFilter(filter) ? filter.facetId : '__unknown-facet__'
+    isFacetFilter(filter) ? filter.facetId : FACET_KEY
   );
   return map(state.facets, (_id, facet) => ({
     ...facet,

packages/x-components/src/x-modules/facets/store/getters/selected-filters-by-facet.getter.ts

@@ -2,6 +2,7 @@ import { isFacetFilter } from '@empathyco/x-types';
 import { map } from '@empathyco/x-utils';
 import { groupItemsBy } from '../../../../utils/array';
 import { FacetsXStoreModule, FiltersByFacet } from '../types';
+import { FACET_KEY } from '../../../../components/index';
 
 /**
  * Default implementation for the {@link FacetsGetters.selectedFiltersByFacet} getter.
@@ -12,8 +13,8 @@ import { FacetsXStoreModule, FiltersByFacet } from '../types';
  * facets' module.
  *
  * @returns A record containing the selected filters indexed by its facet id.
- * @remarks If there are filters without facet Id (RawFilter), they will be grouped under
- * `__unknown-facet__` key.
+ * @remarks If there are filters without facetId (RawFilter), they will be grouped under
+ * {@link FACET_KEY | '__unknown__'} key.
  *
  * @public
  */
@@ -24,7 +25,7 @@ export const selectedFiltersByFacet: FacetsXStoreModule['getters']['selectedFilt
   // The `emptyRecord` is to return an empty array for those facets that haven't selected filters.
   const emptyRecord: FiltersByFacet = map(state.facets, () => []);
   const filtersByFacet = groupItemsBy(getters.selectedFilters, filter =>
-    isFacetFilter(filter) ? filter.facetId : '__unknown-facet__'
+    isFacetFilter(filter) ? filter.facetId : FACET_KEY
   );
   return Object.assign(emptyRecord, filtersByFacet);
 };

packages/x-components/src/x-modules/history-queries/store/__tests__/actions.spec.ts

@@ -427,7 +427,7 @@ describe('testing history queries module actions', () => {
           query: 'gato',
           page: 1,
           filters: {
-            __unknown__: [
+            FACET_KEY: [
               {
                 id: 'categoryIds:66dd06d9f',
                 selected: true,

packages/x-components/src/x-modules/history-queries/store/actions/update-history-queries-with-search-response.action.ts

@@ -1,6 +1,8 @@
 import { Filter } from '@empathyco/x-types';
-import { HistoryQueriesXStoreModule } from '../types';
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
+import { HistoryQueriesActions, HistoryQueriesXStoreModule } from '../types';
 import { InternalSearchResponse } from '../../../search/index';
+import { FACET_KEY } from '../../../../components/index';
 
 /**
  * Default implementation for the
@@ -67,7 +69,7 @@ function getHistoryQueriesFiltersList(
   } else {
     return Object.entries(requestFilters).flatMap(([facetId, facetFilters]) => {
       const matchingFacet =
-        facetId !== '__unknown__' ? responseFacets.find(facet => facet.id === facetId) : null;
+        facetId !== FACET_KEY ? responseFacets.find(facet => facet.id === facetId) : null;
 
       return facetFilters.reduce<Filter[]>((accFilters, requestFilter) => {
         const matchingFilter = matchingFacet

packages/x-components/src/x-modules/search/store/module.ts

@@ -4,6 +4,7 @@ import { setStatus } from '../../../store/utils/status-store.utils';
 import { groupItemsBy } from '../../../utils/array';
 import { mergeConfig, setConfig } from '../../../store/utils/config-store.utils';
 // eslint-disable-next-line max-len
+import { FACET_KEY } from '../../../components/index';
 import {
   cancelFetchAndSaveSearchResponse,
   fetchAndSaveSearchResponse
@@ -61,7 +62,7 @@ export const searchXStoreModule: SearchXStoreModule = {
     },
     setSelectedFilters(state, selectedFilters) {
       state.selectedFilters = groupItemsBy(selectedFilters, filter =>
-        isFacetFilter(filter) ? filter.facetId : '__unknown__'
+        isFacetFilter(filter) ? filter.facetId : FACET_KEY
       );
     },
     setBanners(state, banners) {