feat: extract skipSponsoredProducts function

Author: Henrique Caúla

react/components/SearchQuery.js

@@ -13,6 +13,7 @@ import {
   detachFiltersByType,
   buildQueryArgsFromSelectedFacets,
 } from '../utils/compatibilityLayer'
+import shouldSkipSponsoredProducts from '../utils/shouldSkipSponsoredProducts'
 import { FACETS_RENDER_THRESHOLD } from '../constants/filterConstants'
 import useRedirect from '../hooks/useRedirect'
 import useSession from '../hooks/useSession'
@@ -169,12 +170,6 @@ const useCorrectSearchStateVariables = (
   return result
 }
 
-const skipSponsoredProducts = ({ sponsoredProductsBehavior, settings }) => {
-  const fetchSponsoredProductsConfig = settings?.fetchSponsoredProductsOnSearch
-
-  return !fetchSponsoredProductsConfig && sponsoredProductsBehavior === 'skip'
-}
-
 const sponsoredProductsResult = (
   sponsoredProductsLoading,
   sponsoredProductsError,
@@ -216,7 +211,7 @@ const useQueries = (
     error: sponsoredProductsError,
   } = useQuery(sponsoredProductsQuery, {
     variables,
-    skip: skipSponsoredProducts({ sponsoredProductsBehavior, settings }),
+    skip: shouldSkipSponsoredProducts(sponsoredProductsBehavior, settings),
   })
 
   const sponsoredProductsReturn = sponsoredProductsResult(

react/utils/shouldSkipSponsoredProducts.ts

@@ -0,0 +1,23 @@
+type SponsoredProductsBehavior = 'skip' | string
+
+type Settings = {
+  fetchSponsoredProductsOnSearch: boolean
+} & Record<string, unknown>
+
+/**
+ * This function checks the store's settings, as well as the passed value of
+ * `sponsoredProductsBehavior` passed in the store-theme, to determine if the
+ * sponsored products request should be skipped or not.
+ * Some accounts may not have configured the store's settings, so we need to check if the
+ * `sponsoredProductsBehavior` parameter for compatibility.
+ */
+const shouldSkipSponsoredProducts = (
+  sponsoredProductsBehavior: SponsoredProductsBehavior,
+  settings: Settings
+) => {
+  const fetchSponsoredProductsConfig = settings?.fetchSponsoredProductsOnSearch
+
+  return !fetchSponsoredProductsConfig && sponsoredProductsBehavior === 'skip'
+}
+
+export default shouldSkipSponsoredProducts