fix(toolbar): sw-3443 productId key refresh (#1566)

* toolbar, productId key refresh component, align with toolbarFields
* viewReducer, simplify focused query reset to hard reset

Author: cdcabrera

src/components/toolbar/toolbar.js

@@ -62,7 +62,7 @@ const Toolbar = ({
   useToolbarFieldClearAll: useAliasToolbarFieldClearAll = useToolbarFieldClearAll,
   useToolbarFields: useAliasToolbarFields = useToolbarFields
 }) => {
-  const { productGroup } = useAliasProduct();
+  const { productId } = useAliasProduct();
   const toolbarFieldQueries = useAliasProductToolbarQuery();
   const { currentCategory, options } = useAliasSelectCategoryOptions();
   const clearField = useAliasToolbarFieldClear();
@@ -115,7 +115,7 @@ const Toolbar = ({
   return (
     <PfToolbar
       id="curiosity-toolbar"
-      key={productGroup}
+      key={productId}
       className="curiosity-toolbar pf-m-toggle-group-container ins-c-primary-toolbar"
       collapseListedFiltersBreakpoint="sm"
       clearAllFilters={onClearAll}

src/redux/reducers/__tests__/__snapshots__/viewReducer.test.js.snap

@@ -4,30 +4,10 @@ exports[`ViewReducer should handle clearing state with defined types: defined ty
 {
   "result": {
     "graphTallyQuery": {},
-    "inventoryGuestsQuery": {
-      "another_test_id": {
-        "offset": 10,
-      },
-    },
-    "inventoryHostsQuery": {
-      "another_test_id": {
-        "dir": "lorem desc direction",
-        "offset": 90,
-        "sort": "lorem sort",
-      },
-    },
-    "inventorySubscriptionsQuery": {
-      "another_test_id": {
-        "dir": "dolor desc direction",
-        "offset": 40,
-        "sort": "dolor sort",
-      },
-    },
-    "query": {
-      "another_test_id": {
-        "lorem": "ipsum",
-      },
-    },
+    "inventoryGuestsQuery": {},
+    "inventoryHostsQuery": {},
+    "inventorySubscriptionsQuery": {},
+    "query": {},
   },
   "type": "SET_PRODUCT_VARIANT_QUERY_RESET_ALL",
 }

src/redux/reducers/viewReducer.js

@@ -35,23 +35,20 @@ const initialState = {
  */
 const viewReducer = (state = initialState, action) => {
   switch (action.type) {
-    // Reset query state/object associated with a variant/productId
+    /*
+     * Note: Hard reset query state/object associated with a variant/productId. Remove this action type if restoring
+     * queries based on product variant is needed, also review toolbar reducer for restoring/removing legend state.
+     */
     case reduxTypes.app.SET_PRODUCT_VARIANT_QUERY_RESET_ALL:
-      const updateVariantResetQueries = (query = {}, id) => {
-        const updatedQuery = query;
-        delete updatedQuery[id];
-        return updatedQuery;
-      };
-
       return reduxHelpers.setStateProp(
         null,
         {
           ...state,
-          query: updateVariantResetQueries(state.query, action.variant),
-          graphTallyQuery: updateVariantResetQueries(state.graphTallyQuery, action.variant),
-          inventoryGuestsQuery: updateVariantResetQueries(state.inventoryGuestsQuery, action.variant),
-          inventoryHostsQuery: updateVariantResetQueries(state.inventoryHostsQuery, action.variant),
-          inventorySubscriptionsQuery: updateVariantResetQueries(state.inventorySubscriptionsQuery, action.variant)
+          query: {},
+          graphTallyQuery: {},
+          inventoryGuestsQuery: {},
+          inventoryHostsQuery: {},
+          inventorySubscriptionsQuery: {}
         },
         {
           state,