cozy-client 34.6.1 → 34.6.2

package/dist/CozyClient.js

@@ -2644,7 +2644,7 @@ var CozyClient = /*#__PURE__*/function () {
 CozyClient.hooks = CozyClient.hooks || {};
 CozyClient.fetchPolicies = _policies.default; //COZY_CLIENT_VERSION_PACKAGE in replaced by babel. See babel config
 
-CozyClient.version = "34.6.1";
+CozyClient.version = "34.6.2";
 
 _microee.default.mixin(CozyClient);
 

package/dist/hooks/useQuery.js

@@ -60,10 +60,15 @@ var useQuery = function useQuery(queryDefinition, options) {
     throw new Error('Bad query');
   }
 
-  var definition = resolveToValue(queryDefinition);
   var as = options.as,
       _options$enabled = options.enabled,
-      enabled = _options$enabled === void 0 ? true : _options$enabled;
+      enabled = _options$enabled === void 0 ? true : _options$enabled; // If the query is not enabled, no need to call the queryDefinition
+  // because sometimes, we can have a getById(null) since we want to
+  // enabled the query only when the specific id is defined. And since
+  // Q() can throw error when some checks are KO we don't call Q() if
+  // enabled is not true
+
+  var definition = enabled ? resolveToValue(queryDefinition) : null;
 
   if (!as) {
     throw new Error('You must specify options.as when using useQuery');

package/dist/hooks/useQuery.spec.jsx

@@ -8,6 +8,14 @@ import { Q } from '../queries/dsl'
 import { setupClient, makeWrapper } from '../testing/utils'
 import simpsonsFixture from '../testing/simpsons.json'
 
+/**
+ * @param {object} Options options
+ * @param {object=} Options.customClient custom Client
+ * @param {object=} Options.queryDefinition custom Querydef
+ * @param {object=} Options.queryOptions custom QueryOpts
+ * @param {object=} Options.storeQueries custome StoreQueries
+ * @returns
+ */
 const setupQuery = ({
   customClient,
   queryDefinition,
@@ -51,7 +59,32 @@ describe('use query', () => {
       })
     })
   })
-
+  it('should respect the enable property', () => {
+    const { client } = setupQuery({
+      queryOptions: {
+        as: 'simpsons',
+        enabled: false
+      },
+      queryDefinition: () => Q('io.cozy.simpsons')
+    })
+    expect(client.query).not.toHaveBeenCalled()
+  })
+  it('should respect not throw error if the qDef contains a getById with null', () => {
+    const {
+      client,
+      hookResult: {
+        result: { current }
+      }
+    } = setupQuery({
+      queryOptions: {
+        as: 'simpsonsTest',
+        enabled: false
+      },
+      queryDefinition: () => Q('io.cozy.simpsons').getById(null)
+    })
+    expect(client.query).not.toThrowError()
+    expect(current.fetchStatus).toEqual('pending')
+  })
   it('should share the same API than ObservableQuery', () => {
     const {
       hookResult: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-client",
-  "version": "34.6.1",
+  "version": "34.6.2",
   "license": "MIT",
   "main": "dist/index.node.js",
   "browser": "dist/index.js",
@@ -73,5 +73,5 @@
     "react-native-ios11-devicecheck": "https://github.com/cozy/react-native-devicecheck#app-attest-v0.1"
   },
   "sideEffects": false,
-  "gitHead": "3860bc98e92f4c4a610739b55b3929ba7537446f"
+  "gitHead": "83651b3c5763b255b796d9dcdd037ed4527fb965"
 }