fix: Optimize the request

No need to get the current_state (of mapping all the jobs) for the
triggers here. So let's make a call to /data/io.cozy.triggers instead

packages/cozy-harvest-lib/src/connections/accounts.js

@@ -237,7 +237,22 @@ export const fetchAccountsWithoutTriggers = async (client, triggers) => {
  * @return {Account} An account without trigger for the given konnector
  */
 export const fetchReusableAccount = async (client, konnector) => {
-  const { data: triggers } = await client.collection('io.cozy.triggers').all()
+  // the where is there because ATM TriggerCollection check if there is
+  // a selector to know if it should call /jobs/triggers or
+  // /data/io.cozy.triggers.
+  // In our case, we only want the triggers without the job, so we want
+  // to use /data/io.cozy.triggers
+  const triggers = await client.queryAll(
+    Q('io.cozy.triggers')
+      .where({
+        _id: { $gt: null }
+      })
+      .partialIndex({
+        worker: { $in: ['konnector', 'client'] }
+      })
+      .indexFields(['_id'])
+  )
+
   const accountsWithoutTrigger = await fetchAccountsWithoutTriggers(
     client,
     triggers

packages/cozy-harvest-lib/src/connections/accounts.spec.js

@@ -417,20 +417,15 @@ describe('Account mutations', () => {
 describe('fetchReusableAccount', () => {
   const setup = ({ accounts, triggers }) => {
     const client = new CozyClient({})
-    client.collection = jest.fn(doctype => {
-      if (doctype === 'io.cozy.triggers') {
-        return {
-          all: jest.fn().mockResolvedValue({ data: triggers })
-        }
-      } else {
-        throw new Error(`client.collection for ${doctype} is not mocked`)
-      }
-    })
     client.query = jest.fn().mockImplementation(() => {
       return {
         data: accounts
       }
     })
+
+    client.queryAll = jest.fn().mockImplementation(() => {
+      return triggers
+    })
     return { client }
   }
   it('should return the right account when possible', async () => {