Set correctly the filters in History Query when selecting a Query Preview with filters

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

@@ -16,6 +16,9 @@ import {
   HistoryQueriesMutations,
   HistoryQueriesState
 } from '../types';
+import { InternalSearchResponse } from '../../../search/index';
+// eslint-disable-next-line max-len
+import { createHistoryQueriesFiltersList } from '../actions/update-history-queries-with-search-response.action';
 import { resetHistoryQueriesStateWith } from './utils';
 
 describe('testing history queries module actions', () => {
@@ -279,12 +282,31 @@ describe('testing history queries module actions', () => {
     const requestFilters: Record<string, Filter[]> = {
       categoryPaths: [
         {
-          id: 'categoryIds:c018019b6',
+          id: 'categoryIds:66dd06d9f',
           selected: true,
           modelName: 'HierarchicalFilter'
         }
       ]
     };
+    const responseFacets: InternalSearchResponse['facets'] = [
+      {
+        filters: [
+          {
+            // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+            // @ts-ignore
+            facetId: 'categoryPaths',
+            id: 'categoryIds:66dd06d9f',
+            label: 'suede',
+            modelName: 'HierarchicalFilter',
+            selected: true,
+            totalResults: 2
+          }
+        ],
+        id: 'categoryPaths',
+        label: 'categoryPaths',
+        modelName: 'HierarchicalFacet'
+      }
+    ];
     let gato: HistoryQuery, perro: HistoryQuery;
 
     beforeEach(() => {
@@ -369,9 +391,9 @@ describe('testing history queries module actions', () => {
     });
 
     // eslint-disable-next-line max-len
-    it('updates a history query if search response change because a filter is selected', async () => {
+    it('updates a history query when the search response changes because a filter is selected', async () => {
       gato.totalResults = 50;
-      const selectedFilters = Object.values(requestFilters)[0];
+      const selectedFilters = createHistoryQueriesFiltersList(responseFacets);
       resetStateWith({ historyQueries: [gato, perro] });
       await store.dispatch('updateHistoryQueriesWithSearchResponse', {
         request: {
@@ -380,6 +402,7 @@ describe('testing history queries module actions', () => {
           filters: requestFilters
         },
         status: 'success',
+        facets: responseFacets,
         results,
         totalResults
       });

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

@@ -31,7 +31,7 @@ export const updateHistoryQueriesWithSearchResponse: HistoryQueriesXStoreModule[
         const historyQuery = state.historyQueries[indexOfHistoryQuery];
         const isCurrentSessionHistoryQuery = historyQuery.timestamp > state.sessionTimeStampInMs;
         if (!isCurrentSessionHistoryQuery || historyQuery.totalResults == null || searchResponse) {
-          const filters = createHistoryQueriesFiltersList(searchResponse.request.filters);
+          const filters = createHistoryQueriesFiltersList(searchResponse.facets);
 
           const newHistoryQueries = state.historyQueries.slice();
           newHistoryQueries[indexOfHistoryQuery] = {
@@ -46,19 +46,21 @@ export const updateHistoryQueriesWithSearchResponse: HistoryQueriesXStoreModule[
   };
 
 /**
- * Take filters from the request and push them into a list.
+ * Take facets from the response and push the selected filters into a list.
  *
- * @param requestFilters - Filters from the request.
+ * @param responseFacets - Facets from the response.
  *
  * @returns A list of selected filters in the history query.
  *
  */
-function createHistoryQueriesFiltersList(
-  requestFilters: InternalSearchResponse['request']['filters']
+export function createHistoryQueriesFiltersList(
+  responseFacets: InternalSearchResponse['facets']
 ): Filter[] {
-  return requestFilters
-    ? Object.values(requestFilters).reduce((accFilters, filters) => {
-        accFilters.push(...filters);
+  return responseFacets
+    ? Object.values(responseFacets).reduce((accFilters: Filter[], facet) => {
+        facet.filters.forEach(filter => {
+          return filter.selected ? accFilters.push(filter) : [];
+        });
         return accFilters;
       }, [])
     : [];