cozy-harvest-lib 7.2.4 → 7.3.0

package/CHANGELOG.md

@@ -3,6 +3,22 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [7.3.0](https://github.com/cozy/cozy-libs/compare/cozy-harvest-lib@7.2.4...cozy-harvest-lib@7.3.0) (2022-02-18)
+
+
+### Bug Fixes
+
+* **deps:** pin dependencies ([e53d065](https://github.com/cozy/cozy-libs/commit/e53d065090224ea340b2c25c3afd14f223f4d119))
+
+
+### Features
+
+* Do not allow multiple accounts with same BI connection ID ([55b5f23](https://github.com/cozy/cozy-libs/commit/55b5f23f0adbc308c3b70fa287c3938ee1b0a4cc))
+
+
+
+
+
 ## [7.2.4](https://github.com/cozy/cozy-libs/compare/cozy-harvest-lib@7.2.3...cozy-harvest-lib@7.2.4) (2022-02-15)
 
 

package/dist/helpers/konnectors.js

@@ -36,7 +36,8 @@ var TERMS_VERSION_MISMATCH = 'TERMS_VERSION_MISMATCH';
 var UNKNOWN_ERROR = 'UNKNOWN_ERROR';
 var USER_ACTION_NEEDED = 'USER_ACTION_NEEDED';
 var VENDOR_DOWN = 'VENDOR_DOWN';
-var KNOWN_ERRORS = [CHALLENGE_ASKED, DISK_QUOTA_EXCEEDED, LOGIN_FAILED, MAINTENANCE, NOT_EXISTING_DIRECTORY, TERMS_VERSION_MISMATCH, USER_ACTION_NEEDED, VENDOR_DOWN];
+var ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED = 'ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED';
+var KNOWN_ERRORS = [CHALLENGE_ASKED, DISK_QUOTA_EXCEEDED, LOGIN_FAILED, MAINTENANCE, NOT_EXISTING_DIRECTORY, TERMS_VERSION_MISMATCH, USER_ACTION_NEEDED, VENDOR_DOWN, ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED];
 var USER_ERRORS = [CHALLENGE_ASKED, DISK_QUOTA_EXCEEDED, LOGIN_FAILED, NOT_EXISTING_DIRECTORY, USER_ACTION_NEEDED];
 var sanitizeAccountIdentifierRx = /\//g;
 /**

package/dist/locales/en.json

@@ -204,6 +204,10 @@
       "VENDOR_DOWN.LINXO_DOWN": {
         "title": "Unavailable service",
         "description": "It seems that we are experiencing overload with our bank konnectors at the moment. Please rerun the connector later."
+      },
+      "ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED": {
+        "title": "This account already exists",
+        "description": "You already have configured an account with these identifiers."
       }
     }
   },

package/dist/locales/fr.json

@@ -204,6 +204,10 @@
       "VENDOR_DOWN.LINXO_DOWN": {
         "title": "Service non disponible",
         "description": "Il semble que le service [%{name}](%{link}) ne nous ait pas répondu dans les temps. Vous pouvez tenter de le relancer manuellement maintenant ou ultérieurement."
+      },
+      "ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED": {
+        "title": "Ce compte est déjà configuré",
+        "description": "Vous avez déjà configuré un compte avec ces identifiants."
       }
     }
   },

package/dist/services/budget-insight.js

@@ -1,3 +1,4 @@
+import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import _objectWithoutProperties from "@babel/runtime/helpers/objectWithoutProperties";
 import _asyncToGenerator from "@babel/runtime/helpers/asyncToGenerator";
@@ -23,7 +24,9 @@ import get from 'lodash/get';
 import omit from 'lodash/omit';
 import clone from 'lodash/clone';
 import set from 'lodash/set';
+import keyBy from 'lodash/keyBy';
 import defaults from 'lodash/defaults';
+import { Q } from 'cozy-client';
 import { waitForRealtimeEvent } from './jobUtils';
 import { getBIConnection, createBIConnection, updateBIConnection, getBIUserConfig, updateBIUserConfig, setBIConnectionSyncStatus } from './bi-http';
 import assert from '../assert';
@@ -51,7 +54,8 @@ var getBIIdFromContract = function getBIIdFromContract(bankAccount) {
 
 
 var extraBIErrorMap = {
-  config: 'LOGIN_FAILED'
+  config: 'LOGIN_FAILED',
+  ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED: 'ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED'
 };
 /**
  * Converts and chains error
@@ -144,7 +148,7 @@ export var getBIConfig = function getBIConfig(flow) {
 
 export var createOrUpdateBIConnection = /*#__PURE__*/function () {
   var _ref4 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee2(_ref3) {
-    var account, client, konnector, flow, connId, biConfig, tempToken, config, credentials, credsToSend, biConnection, connection;
+    var account, client, konnector, flow, connId, biConfig, tempToken, config, credentials, credsToSend, biConnection, isUpdate, connection, sameAccount, err;
     return _regeneratorRuntime.wrap(function _callee2$(_context2) {
       while (1) {
         switch (_context2.prev = _context2.next) {
@@ -212,25 +216,52 @@ export var createOrUpdateBIConnection = /*#__PURE__*/function () {
 
           case 31:
             logger.info('Creating or updating connection...');
-            _context2.next = 34;
-            return connId ? updateBIConnection(config, connId, credsToSend, tempToken) : createBIConnection(config, credsToSend, tempToken);
+            isUpdate = Boolean(connId);
+            _context2.next = 35;
+            return isUpdate ? updateBIConnection(config, connId, credsToSend, tempToken) : createBIConnection(config, credsToSend, tempToken);
 
-          case 34:
+          case 35:
             connection = _context2.sent;
+
+            if (isUpdate) {
+              _context2.next = 44;
+              break;
+            }
+
+            _context2.next = 39;
+            return findAccountWithBiConnection({
+              client: client,
+              konnector: konnector,
+              connectionId: connection.id
+            });
+
+          case 39:
+            sameAccount = _context2.sent;
+
+            if (!sameAccount) {
+              _context2.next = 44;
+              break;
+            }
+
+            err = new KonnectorJobError('ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED');
+            err.accountId = sameAccount._id;
+            throw err;
+
+          case 44:
             logger.info("Created or updated connection ".concat(connection.id));
             return _context2.abrupt("return", connection);
 
-          case 39:
-            _context2.prev = 39;
+          case 48:
+            _context2.prev = 48;
             _context2.t1 = _context2["catch"](15);
             return _context2.abrupt("return", convertBIErrortoKonnectorJobError(_context2.t1));
 
-          case 42:
+          case 51:
           case "end":
             return _context2.stop();
         }
       }
-    }, _callee2, null, [[15, 39], [17, 27]]);
+    }, _callee2, null, [[15, 48], [17, 27]]);
   }));
 
   return function createOrUpdateBIConnection(_x2) {
@@ -761,7 +792,64 @@ export var fetchExtraOAuthUrlParams = /*#__PURE__*/function () {
 var shouldResumeConnection = function shouldResumeConnection(error) {
   return error === DECOUPLED_ERROR || error === ADDITIONAL_INFORMATION_NEEDED_ERROR;
 };
+/**
+ * Tries to find an existing account, associated to an existing trigger
+ * with the given bi connection id
+ *
+ * @param  {CozyClient} options.client - Cozy client
+ * @param  {Object} options.konnector - Konnector manifest
+ * @param  {Integer} options.connectionId - BI connection id
+ * @return {Account|null} An account with a trigger with the same identifier if any
+ */
+
+
+export var findAccountWithBiConnection = /*#__PURE__*/function () {
+  var _ref22 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11(_ref21) {
+    var client, konnector, connectionId, _yield$Promise$all, _yield$Promise$all2, accountsResult, triggersResult, accountsIndex, trigger;
+
+    return _regeneratorRuntime.wrap(function _callee11$(_context11) {
+      while (1) {
+        switch (_context11.prev = _context11.next) {
+          case 0:
+            client = _ref21.client, konnector = _ref21.konnector, connectionId = _ref21.connectionId;
+            _context11.next = 3;
+            return Promise.all([client.query(Q('io.cozy.accounts').where({
+              data: {
+                auth: {
+                  bi: {
+                    connId: connectionId
+                  }
+                }
+              }
+            }).indexFields(['data.auth.bi.connId'])), client.query(Q('io.cozy.triggers').where({
+              message: {
+                konnector: konnector.slug
+              }
+            }).indexFields(['message.konnector']))]);
+
+          case 3:
+            _yield$Promise$all = _context11.sent;
+            _yield$Promise$all2 = _slicedToArray(_yield$Promise$all, 2);
+            accountsResult = _yield$Promise$all2[0];
+            triggersResult = _yield$Promise$all2[1];
+            accountsIndex = keyBy(accountsResult.data, '_id');
+            trigger = triggersResult.data.find(function (t) {
+              return accountsIndex[get(t, 'message.account')];
+            });
+            return _context11.abrupt("return", trigger ? accountsIndex[trigger.message.account] : null);
 
+          case 10:
+          case "end":
+            return _context11.stop();
+        }
+      }
+    }, _callee11);
+  }));
+
+  return function findAccountWithBiConnection(_x12) {
+    return _ref22.apply(this, arguments);
+  };
+}();
 export var konnectorPolicy = {
   name: 'budget-insight',
   match: isBudgetInsightConnector,
@@ -772,5 +860,6 @@ export var konnectorPolicy = {
   fetchExtraOAuthUrlParams: fetchExtraOAuthUrlParams,
   getAdditionalInformationNeeded: getAdditionalInformationNeeded,
   handleOAuthAccount: handleOAuthAccount,
-  setSync: setSync
+  setSync: setSync,
+  findAccountWithBiConnection: findAccountWithBiConnection
 };

package/dist/services/budget-insight.spec.js

@@ -449,6 +449,22 @@ describe('createOrUpdateBIConnection', function () {
         }
       }, _callee10);
     })));
+    jest.spyOn(client, 'query').mockImplementation( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11() {
+      return _regeneratorRuntime.wrap(function _callee11$(_context11) {
+        while (1) {
+          switch (_context11.prev = _context11.next) {
+            case 0:
+              return _context11.abrupt("return", {
+                data: []
+              });
+
+            case 1:
+            case "end":
+              return _context11.stop();
+          }
+        }
+      }, _callee11);
+    })));
     createBIConnection.mockReset().mockResolvedValue({
       id: 'created-bi-connection-id-789'
     });
@@ -461,15 +477,15 @@ describe('createOrUpdateBIConnection', function () {
     };
   };
 
-  it('should accept a connector without parameter and find bank id in the account', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11() {
+  it('should accept a connector without parameter and find bank id in the account', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12() {
     var _setup7, client, flow, connection;
 
-    return _regeneratorRuntime.wrap(function _callee11$(_context11) {
+    return _regeneratorRuntime.wrap(function _callee12$(_context12) {
       while (1) {
-        switch (_context11.prev = _context11.next) {
+        switch (_context12.prev = _context12.next) {
           case 0:
             _setup7 = setup(), client = _setup7.client, flow = _setup7.flow;
-            _context11.next = 3;
+            _context12.next = 3;
             return createOrUpdateBIConnection({
               client: client,
               account: account,
@@ -478,7 +494,7 @@ describe('createOrUpdateBIConnection', function () {
             });
 
           case 3:
-            connection = _context11.sent;
+            connection = _context12.sent;
             expect(waitForRealtimeEvent).toHaveBeenCalledWith(client, {
               _id: 'job-id-1337'
             }, 'result', 60000);
@@ -494,20 +510,20 @@ describe('createOrUpdateBIConnection', function () {
 
           case 8:
           case "end":
-            return _context11.stop();
+            return _context12.stop();
         }
       }
-    }, _callee11);
+    }, _callee12);
   })));
-  it('should create a BI connection if no connection id in account', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12() {
+  it('should create a BI connection if no connection id in account', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13() {
     var _setup8, client, flow, connection;
 
-    return _regeneratorRuntime.wrap(function _callee12$(_context12) {
+    return _regeneratorRuntime.wrap(function _callee13$(_context13) {
       while (1) {
-        switch (_context12.prev = _context12.next) {
+        switch (_context13.prev = _context13.next) {
           case 0:
             _setup8 = setup(), client = _setup8.client, flow = _setup8.flow;
-            _context12.next = 3;
+            _context13.next = 3;
             return createOrUpdateBIConnection({
               client: client,
               account: account,
@@ -516,7 +532,7 @@ describe('createOrUpdateBIConnection', function () {
             });
 
           case 3:
-            connection = _context12.sent;
+            connection = _context13.sent;
             expect(waitForRealtimeEvent).toHaveBeenCalledWith(client, {
               _id: 'job-id-1337'
             }, 'result', 60000);
@@ -532,17 +548,17 @@ describe('createOrUpdateBIConnection', function () {
 
           case 8:
           case "end":
-            return _context12.stop();
+            return _context13.stop();
         }
       }
-    }, _callee12);
+    }, _callee13);
   })));
-  it('should update the BI connection if connection id in account', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13() {
+  it('should update the BI connection if connection id in account', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14() {
     var _setup9, client, flow, connection;
 
-    return _regeneratorRuntime.wrap(function _callee13$(_context13) {
+    return _regeneratorRuntime.wrap(function _callee14$(_context14) {
       while (1) {
-        switch (_context13.prev = _context13.next) {
+        switch (_context14.prev = _context14.next) {
           case 0:
             _setup9 = setup(), client = _setup9.client, flow = _setup9.flow;
             getBIConnection.mockReset().mockResolvedValue({
@@ -551,7 +567,7 @@ describe('createOrUpdateBIConnection', function () {
             updateBIConnection.mockReset().mockResolvedValue({
               id: 'updated-bi-connection-id-789'
             });
-            _context13.next = 5;
+            _context14.next = 5;
             return createOrUpdateBIConnection({
               client: client,
               flow: flow,
@@ -568,7 +584,7 @@ describe('createOrUpdateBIConnection', function () {
             });
 
           case 5:
-            connection = _context13.sent;
+            connection = _context14.sent;
             expect(waitForRealtimeEvent).toHaveBeenCalledWith(client, {
               _id: 'job-id-1337'
             }, 'result', 60000);
@@ -590,21 +606,21 @@ describe('createOrUpdateBIConnection', function () {
 
           case 11:
           case "end":
-            return _context13.stop();
+            return _context14.stop();
         }
       }
-    }, _callee13);
+    }, _callee14);
   })));
-  it('should create the BI connection if connection id in the account does not exist', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14() {
+  it('should create the BI connection if connection id in the account does not exist', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15() {
     var _setup10, client, flow, connection;
 
-    return _regeneratorRuntime.wrap(function _callee14$(_context14) {
+    return _regeneratorRuntime.wrap(function _callee15$(_context15) {
       while (1) {
-        switch (_context14.prev = _context14.next) {
+        switch (_context15.prev = _context15.next) {
           case 0:
             _setup10 = setup(), client = _setup10.client, flow = _setup10.flow;
             getBIConnection.mockRejectedValueOnce();
-            _context14.next = 4;
+            _context15.next = 4;
             return createOrUpdateBIConnection({
               client: client,
               flow: flow,
@@ -621,7 +637,7 @@ describe('createOrUpdateBIConnection', function () {
             });
 
           case 4:
-            connection = _context14.sent;
+            connection = _context15.sent;
             expect(waitForRealtimeEvent).toHaveBeenCalledWith(client, {
               _id: 'job-id-1337'
             }, 'result', 60000);
@@ -643,24 +659,24 @@ describe('createOrUpdateBIConnection', function () {
 
           case 10:
           case "end":
-            return _context14.stop();
+            return _context15.stop();
         }
       }
-    }, _callee14);
+    }, _callee15);
   })));
-  it('should convert wrongpass correctly', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15() {
+  it('should convert wrongpass correctly', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16() {
     var _setup11, client, flow, err;
 
-    return _regeneratorRuntime.wrap(function _callee15$(_context15) {
+    return _regeneratorRuntime.wrap(function _callee16$(_context16) {
       while (1) {
-        switch (_context15.prev = _context15.next) {
+        switch (_context16.prev = _context16.next) {
           case 0:
             _setup11 = setup(), client = _setup11.client, flow = _setup11.flow;
             err = new Error();
             err.code = 'wrongpass';
             getBIConnection.mockReset().mockResolvedValueOnce({});
             updateBIConnection.mockReset().mockRejectedValue(err);
-            _context15.next = 7;
+            _context16.next = 7;
             return expect(createOrUpdateBIConnection({
               client: client,
               flow: flow,
@@ -678,24 +694,24 @@ describe('createOrUpdateBIConnection', function () {
 
           case 7:
           case "end":
-            return _context15.stop();
+            return _context16.stop();
         }
       }
-    }, _callee15);
+    }, _callee16);
   })));
-  it('should convert config correctly', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16() {
+  it('should convert config correctly', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee17() {
     var _setup12, client, flow, err;
 
-    return _regeneratorRuntime.wrap(function _callee16$(_context16) {
+    return _regeneratorRuntime.wrap(function _callee17$(_context17) {
       while (1) {
-        switch (_context16.prev = _context16.next) {
+        switch (_context17.prev = _context17.next) {
           case 0:
             _setup12 = setup(), client = _setup12.client, flow = _setup12.flow;
             err = new Error();
             err.code = 'config';
             getBIConnection.mockReset().mockResolvedValueOnce({});
             updateBIConnection.mockReset().mockRejectedValue(err);
-            _context16.next = 7;
+            _context17.next = 7;
             return expect(createOrUpdateBIConnection({
               client: client,
               flow: flow,
@@ -713,24 +729,24 @@ describe('createOrUpdateBIConnection', function () {
 
           case 7:
           case "end":
-            return _context16.stop();
+            return _context17.stop();
         }
       }
-    }, _callee16);
+    }, _callee17);
   })));
-  it('should convert SCARequired correctly', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee17() {
+  it('should convert SCARequired correctly', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee18() {
     var _setup13, client, flow, err;
 
-    return _regeneratorRuntime.wrap(function _callee17$(_context17) {
+    return _regeneratorRuntime.wrap(function _callee18$(_context18) {
       while (1) {
-        switch (_context17.prev = _context17.next) {
+        switch (_context18.prev = _context18.next) {
           case 0:
             _setup13 = setup(), client = _setup13.client, flow = _setup13.flow;
             err = new Error();
             err.code = 'SCARequired';
             getBIConnection.mockReset().mockResolvedValueOnce({});
             updateBIConnection.mockReset().mockRejectedValue(err);
-            _context17.next = 7;
+            _context18.next = 7;
             return expect(createOrUpdateBIConnection({
               client: client,
               flow: flow,
@@ -748,17 +764,17 @@ describe('createOrUpdateBIConnection', function () {
 
           case 7:
           case "end":
-            return _context17.stop();
+            return _context18.stop();
         }
       }
-    }, _callee17);
+    }, _callee18);
   })));
-  it('should remove sensible data from account and create bi connection', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee18() {
+  it('should remove sensible data from account and create bi connection', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee19() {
     var _setup14, client, flow, account, konnector, createOrUpdateBIConnection, accountToSave;
 
-    return _regeneratorRuntime.wrap(function _callee18$(_context18) {
+    return _regeneratorRuntime.wrap(function _callee19$(_context19) {
       while (1) {
-        switch (_context18.prev = _context18.next) {
+        switch (_context19.prev = _context19.next) {
           case 0:
             _setup14 = setup(), client = _setup14.client, flow = _setup14.flow;
             account = {
@@ -780,7 +796,7 @@ describe('createOrUpdateBIConnection', function () {
             createOrUpdateBIConnection = jest.fn().mockResolvedValue({
               id: 'created-bi-connection-id'
             });
-            _context18.next = 7;
+            _context19.next = 7;
             return onBIAccountCreation({
               client: client,
               flow: flow,
@@ -790,7 +806,7 @@ describe('createOrUpdateBIConnection', function () {
             });
 
           case 7:
-            accountToSave = _context18.sent;
+            accountToSave = _context19.sent;
             expect(createOrUpdateBIConnection).toHaveBeenCalledWith(expect.objectContaining({
               account: {
                 _id: 'created-account-id',
@@ -825,10 +841,209 @@ describe('createOrUpdateBIConnection', function () {
 
           case 11:
           case "end":
-            return _context18.stop();
+            return _context19.stop();
         }
       }
-    }, _callee18);
+    }, _callee19);
+  })));
+  it('should refuse to create an account with a bi connection id which already exists', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee21() {
+    var _setup15, client, flow, account, konnector;
+
+    return _regeneratorRuntime.wrap(function _callee21$(_context21) {
+      while (1) {
+        switch (_context21.prev = _context21.next) {
+          case 0:
+            _setup15 = setup(), client = _setup15.client, flow = _setup15.flow;
+            account = {
+              auth: {
+                login: '1234',
+                password: '4567',
+                dob: '20/12/1890',
+                bankId: '100000'
+              }
+            };
+            konnector = {
+              slug: 'bankingconnectortest'
+            };
+            jest.spyOn(flow, 'saveAccount').mockImplementation(function (account) {
+              return _objectSpread({
+                _id: 'created-account-id'
+              }, account);
+            });
+            jest.spyOn(client, 'query').mockImplementation( /*#__PURE__*/function () {
+              var _ref22 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee20(_ref21) {
+                var doctype;
+                return _regeneratorRuntime.wrap(function _callee20$(_context20) {
+                  while (1) {
+                    switch (_context20.prev = _context20.next) {
+                      case 0:
+                        doctype = _ref21.doctype;
+
+                        if (!(doctype === 'io.cozy.accounts')) {
+                          _context20.next = 5;
+                          break;
+                        }
+
+                        return _context20.abrupt("return", {
+                          data: [{
+                            _id: 'account_id',
+                            auth: {
+                              bi: {
+                                connId: 12
+                              }
+                            }
+                          }]
+                        });
+
+                      case 5:
+                        if (!(doctype === 'io.cozy.triggers')) {
+                          _context20.next = 9;
+                          break;
+                        }
+
+                        return _context20.abrupt("return", {
+                          data: [{
+                            message: {
+                              account: 'account_id'
+                            }
+                          }]
+                        });
+
+                      case 9:
+                        throw new Error('unexpected doctype ' + doctype);
+
+                      case 10:
+                      case "end":
+                        return _context20.stop();
+                    }
+                  }
+                }, _callee20);
+              }));
+
+              return function (_x2) {
+                return _ref22.apply(this, arguments);
+              };
+            }());
+            _context21.next = 7;
+            return expect(onBIAccountCreation({
+              client: client,
+              flow: flow,
+              account: account,
+              konnector: konnector
+            })).rejects.toEqual(new Error('ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED'));
+
+          case 7:
+          case "end":
+            return _context21.stop();
+        }
+      }
+    }, _callee21);
+  })));
+  it('should create an account if bi connection is not already used', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee23() {
+    var _setup16, client, flow, account, konnector;
+
+    return _regeneratorRuntime.wrap(function _callee23$(_context23) {
+      while (1) {
+        switch (_context23.prev = _context23.next) {
+          case 0:
+            _setup16 = setup(), client = _setup16.client, flow = _setup16.flow;
+            account = {
+              auth: {
+                login: '1234',
+                password: '4567',
+                dob: '20/12/1890',
+                bankId: '100000'
+              }
+            };
+            konnector = {
+              slug: 'bankingconnectortest'
+            };
+            jest.spyOn(flow, 'saveAccount').mockImplementation(function (account) {
+              return _objectSpread({
+                _id: 'created-account-id'
+              }, account);
+            });
+            jest.spyOn(client, 'query').mockImplementation( /*#__PURE__*/function () {
+              var _ref25 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee22(_ref24) {
+                var doctype;
+                return _regeneratorRuntime.wrap(function _callee22$(_context22) {
+                  while (1) {
+                    switch (_context22.prev = _context22.next) {
+                      case 0:
+                        doctype = _ref24.doctype;
+
+                        if (!(doctype === 'io.cozy.accounts')) {
+                          _context22.next = 5;
+                          break;
+                        }
+
+                        return _context22.abrupt("return", {
+                          data: [{
+                            _id: 'account_id',
+                            auth: {
+                              bi: {
+                                connId: 12
+                              }
+                            }
+                          }]
+                        });
+
+                      case 5:
+                        if (!(doctype === 'io.cozy.triggers')) {
+                          _context22.next = 9;
+                          break;
+                        }
+
+                        return _context22.abrupt("return", {
+                          data: [{
+                            message: {
+                              account: 'other_account_id'
+                            }
+                          }]
+                        });
+
+                      case 9:
+                        throw new Error('unexpected doctype ' + doctype);
+
+                      case 10:
+                      case "end":
+                        return _context22.stop();
+                    }
+                  }
+                }, _callee22);
+              }));
+
+              return function (_x3) {
+                return _ref25.apply(this, arguments);
+              };
+            }());
+            _context23.next = 7;
+            return expect(onBIAccountCreation({
+              client: client,
+              flow: flow,
+              account: account,
+              konnector: konnector
+            })).resolves.toEqual({
+              _id: 'created-account-id',
+              auth: {
+                bankId: '100000',
+                login: '1234'
+              },
+              data: {
+                auth: {
+                  bi: {
+                    connId: 'created-bi-connection-id-789'
+                  }
+                }
+              }
+            });
+
+          case 7:
+          case "end":
+            return _context23.stop();
+        }
+      }
+    }, _callee23);
   })));
 });
 describe('fetchExtraOAuthUrlParams', function () {
@@ -843,13 +1058,13 @@ describe('fetchExtraOAuthUrlParams', function () {
         }
       }
     });
-    waitForRealtimeEvent.mockImplementation( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee19() {
-      return _regeneratorRuntime.wrap(function _callee19$(_context19) {
+    waitForRealtimeEvent.mockImplementation( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee24() {
+      return _regeneratorRuntime.wrap(function _callee24$(_context24) {
         while (1) {
-          switch (_context19.prev = _context19.next) {
+          switch (_context24.prev = _context24.next) {
             case 0:
               sleep(2);
-              return _context19.abrupt("return", {
+              return _context24.abrupt("return", {
                 data: {
                   result: {
                     code: 'bi-temporary-access-token-121212',
@@ -862,24 +1077,24 @@ describe('fetchExtraOAuthUrlParams', function () {
 
             case 2:
             case "end":
-              return _context19.stop();
+              return _context24.stop();
           }
         }
-      }, _callee19);
+      }, _callee24);
     })));
     return {
       client: client
     };
   };
 
-  it('should create a temporary token', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee20() {
-    var _setup15, client, konnector, account, _yield$fetchExtraOAut, id_connector, token;
+  it('should create a temporary token', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee25() {
+    var _setup17, client, konnector, account, _yield$fetchExtraOAut, id_connector, token;
 
-    return _regeneratorRuntime.wrap(function _callee20$(_context20) {
+    return _regeneratorRuntime.wrap(function _callee25$(_context25) {
       while (1) {
-        switch (_context20.prev = _context20.next) {
+        switch (_context25.prev = _context25.next) {
           case 0:
-            _setup15 = setup(), client = _setup15.client;
+            _setup17 = setup(), client = _setup17.client;
             konnector = {
               slug: 'revolut',
               parameters: {
@@ -887,7 +1102,7 @@ describe('fetchExtraOAuthUrlParams', function () {
               }
             };
             account = {};
-            _context20.next = 5;
+            _context25.next = 5;
             return fetchExtraOAuthUrlParams({
               client: client,
               konnector: konnector,
@@ -895,7 +1110,7 @@ describe('fetchExtraOAuthUrlParams', function () {
             });
 
           case 5:
-            _yield$fetchExtraOAut = _context20.sent;
+            _yield$fetchExtraOAut = _context25.sent;
             id_connector = _yield$fetchExtraOAut.id_connector;
             token = _yield$fetchExtraOAut.token;
             expect(token).toEqual('bi-temporary-access-token-121212');
@@ -903,18 +1118,18 @@ describe('fetchExtraOAuthUrlParams', function () {
 
           case 10:
           case "end":
-            return _context20.stop();
+            return _context25.stop();
         }
       }
-    }, _callee20);
+    }, _callee25);
   })));
 });
 describe('handleOAuthAccount', function () {
-  it('should handle webauth if any connection is found in the account', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee22() {
+  it('should handle webauth if any connection is found in the account', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee27() {
     var client, flow, account, t;
-    return _regeneratorRuntime.wrap(function _callee22$(_context22) {
+    return _regeneratorRuntime.wrap(function _callee27$(_context27) {
       while (1) {
-        switch (_context22.prev = _context22.next) {
+        switch (_context27.prev = _context27.next) {
           case 0:
             client = new CozyClient({
               uri: 'http://testcozy.mycozy.cloud'
@@ -924,23 +1139,23 @@ describe('handleOAuthAccount', function () {
             flow.handleFormSubmit = jest.fn();
 
             flow.saveAccount = /*#__PURE__*/function () {
-              var _ref22 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee21(account) {
-                return _regeneratorRuntime.wrap(function _callee21$(_context21) {
+              var _ref29 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee26(account) {
+                return _regeneratorRuntime.wrap(function _callee26$(_context26) {
                   while (1) {
-                    switch (_context21.prev = _context21.next) {
+                    switch (_context26.prev = _context26.next) {
                       case 0:
-                        return _context21.abrupt("return", account);
+                        return _context26.abrupt("return", account);
 
                       case 1:
                       case "end":
-                        return _context21.stop();
+                        return _context26.stop();
                     }
                   }
-                }, _callee21);
+                }, _callee26);
               }));
 
-              return function (_x2) {
-                return _ref22.apply(this, arguments);
+              return function (_x4) {
+                return _ref29.apply(this, arguments);
               };
             }();
 
@@ -952,7 +1167,7 @@ describe('handleOAuthAccount', function () {
               }
             };
             t = jest.fn();
-            _context22.next = 9;
+            _context27.next = 9;
             return handleOAuthAccount({
               account: account,
               flow: flow,
@@ -983,18 +1198,18 @@ describe('handleOAuthAccount', function () {
 
           case 10:
           case "end":
-            return _context22.stop();
+            return _context27.stop();
         }
       }
-    }, _callee22);
+    }, _callee27);
   })));
 });
 describe('setSync', function () {
-  it('should set synchronization status for a contract', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee24() {
+  it('should set synchronization status for a contract', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee29() {
     var client, flow, biConnId, tempToken, biContractId, account, contract, createTemporaryToken, setBIConnectionSyncStatus;
-    return _regeneratorRuntime.wrap(function _callee24$(_context24) {
+    return _regeneratorRuntime.wrap(function _callee29$(_context29) {
       while (1) {
-        switch (_context24.prev = _context24.next) {
+        switch (_context29.prev = _context29.next) {
           case 0:
             client = new CozyClient({
               uri: 'http://testcozy.mycozy.cloud'
@@ -1004,23 +1219,23 @@ describe('setSync', function () {
             flow.handleFormSubmit = jest.fn();
 
             flow.saveAccount = /*#__PURE__*/function () {
-              var _ref24 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee23(account) {
-                return _regeneratorRuntime.wrap(function _callee23$(_context23) {
+              var _ref31 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee28(account) {
+                return _regeneratorRuntime.wrap(function _callee28$(_context28) {
                   while (1) {
-                    switch (_context23.prev = _context23.next) {
+                    switch (_context28.prev = _context28.next) {
                       case 0:
-                        return _context23.abrupt("return", account);
+                        return _context28.abrupt("return", account);
 
                       case 1:
                       case "end":
-                        return _context23.stop();
+                        return _context28.stop();
                     }
                   }
-                }, _callee23);
+                }, _callee28);
               }));
 
-              return function (_x3) {
-                return _ref24.apply(this, arguments);
+              return function (_x5) {
+                return _ref31.apply(this, arguments);
               };
             }();
 
@@ -1043,7 +1258,7 @@ describe('setSync', function () {
               code: tempToken
             });
             setBIConnectionSyncStatus = jest.fn();
-            _context24.next = 14;
+            _context29.next = 14;
             return setSync({
               client: client,
               account: account,
@@ -1060,18 +1275,18 @@ describe('setSync', function () {
 
           case 15:
           case "end":
-            return _context24.stop();
+            return _context29.stop();
         }
       }
-    }, _callee24);
+    }, _callee29);
   })));
 });
 describe('updateBIConnectionFromFlow', function () {
-  it('should update a connection with given fields', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee25() {
+  it('should update a connection with given fields', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee30() {
     var client, biConnId, account, flow, connectionData;
-    return _regeneratorRuntime.wrap(function _callee25$(_context25) {
+    return _regeneratorRuntime.wrap(function _callee30$(_context30) {
       while (1) {
-        switch (_context25.prev = _context25.next) {
+        switch (_context30.prev = _context30.next) {
           case 0:
             client = new CozyClient({
               uri: 'http://testcozy.mycozy.cloud'
@@ -1098,7 +1313,7 @@ describe('updateBIConnectionFromFlow', function () {
             connectionData = {
               login: '1234'
             };
-            _context25.next = 10;
+            _context30.next = 10;
             return updateBIConnectionFromFlow(flow, connectionData);
 
           case 10:
@@ -1110,10 +1325,10 @@ describe('updateBIConnectionFromFlow', function () {
 
           case 11:
           case "end":
-            return _context25.stop();
+            return _context30.stop();
         }
       }
-    }, _callee25);
+    }, _callee30);
   })));
 });
 describe('sendTwoFACode', function () {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-harvest-lib",
-  "version": "7.2.4",
+  "version": "7.3.0",
   "description": "Provides logic, modules and components for Cozy's harvest applications.",
   "main": "dist/index.js",
   "author": "Cozy",
@@ -29,8 +29,8 @@
     "@cozy/minilog": "^1.0.0",
     "@sentry/browser": "^6.0.1",
     "cozy-bi-auth": "0.0.24",
-    "cozy-doctypes": "^1.83.5",
-    "cozy-logger": "^1.8.0",
+    "cozy-doctypes": "^1.83.6",
+    "cozy-logger": "^1.8.1",
     "date-fns": "^1.30.1",
     "final-form": "^4.18.5",
     "lodash": "^4.17.19",
@@ -45,37 +45,37 @@
     "@babel/core": "7.16.12",
     "@babel/register": "7.16.9",
     "@cozy/cli-tree": "^0.5.0",
-    "@material-ui/core": "4",
-    "@material-ui/lab": "^4.0.0-alpha.57",
-    "@testing-library/jest-dom": "^4.1.2",
+    "@material-ui/core": "4.11.3",
+    "@material-ui/lab": "4.0.0-alpha.57",
+    "@testing-library/jest-dom": "4.2.4",
     "@testing-library/react": "10.4.9",
     "babel-jest": "26.6.3",
     "babel-plugin-inline-react-svg": "1.1.2",
     "babel-preset-cozy-app": "^2.0.2",
-    "cozy-client": "17.6.1",
+    "cozy-client": "27.14.3",
     "cozy-device-helper": "^1.17.0",
-    "cozy-flags": "^2.8.4",
+    "cozy-flags": "^2.8.5",
     "cozy-keys-lib": "3.8.0",
     "cozy-realtime": "^4.0.5",
-    "cozy-ui": "^57.6.0",
+    "cozy-ui": "57.6.0",
     "enzyme": "3.11.0",
     "enzyme-adapter-react-16": "1.15.6",
     "form-data": "3.0.0",
     "identity-obj-proxy": "3.0.0",
     "jest": "26.6.3",
-    "jest-environment-jsdom-sixteen": "^1.0.3",
-    "jest-resolve-cached": "^1.0.0",
-    "jsdom": "^16.1.0",
-    "leaflet": "^1.7.1",
+    "jest-environment-jsdom-sixteen": "1.0.3",
+    "jest-resolve-cached": "1.0.0",
+    "jsdom": "16.4.0",
+    "leaflet": "1.7.1",
     "prop-types": "15.7.2",
-    "react": "^16.12.0",
-    "react-dom": "^16.12.0",
-    "react-swipeable-views": "^0.13.9"
+    "react": "16.12.0",
+    "react-dom": "16.12.0",
+    "react-swipeable-views": "0.13.9"
   },
   "peerDependencies": {
     "@babel/runtime": ">=7.12.5",
     "@material-ui/core": ">=4",
-    "cozy-client": ">=17.6.1",
+    "cozy-client": ">=27.14.3",
     "cozy-device-helper": ">=1.10.2",
     "cozy-flags": ">=2.3.5",
     "cozy-keys-lib": ">=3.7.0",
@@ -85,5 +85,5 @@
     "react-router-dom": "^5.0.1"
   },
   "sideEffects": false,
-  "gitHead": "5f0a839a3dcc01f2aa0106ccdbd67062c8faf44f"
+  "gitHead": "3bfcea11a7a0d354b5feabc3d4a2655345eee2ee"
 }

package/src/helpers/konnectors.js

@@ -22,6 +22,8 @@ const TERMS_VERSION_MISMATCH = 'TERMS_VERSION_MISMATCH'
 const UNKNOWN_ERROR = 'UNKNOWN_ERROR'
 const USER_ACTION_NEEDED = 'USER_ACTION_NEEDED'
 const VENDOR_DOWN = 'VENDOR_DOWN'
+const ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED =
+  'ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED'
 
 const KNOWN_ERRORS = [
   CHALLENGE_ASKED,
@@ -31,7 +33,8 @@ const KNOWN_ERRORS = [
   NOT_EXISTING_DIRECTORY,
   TERMS_VERSION_MISMATCH,
   USER_ACTION_NEEDED,
-  VENDOR_DOWN
+  VENDOR_DOWN,
+  ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED
 ]
 
 const USER_ERRORS = [

package/src/locales/en.json

@@ -204,6 +204,10 @@
       "VENDOR_DOWN.LINXO_DOWN": {
         "title": "Unavailable service",
         "description": "It seems that we are experiencing overload with our bank konnectors at the moment. Please rerun the connector later."
+      },
+      "ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED": {
+        "title": "This account already exists",
+        "description": "You already have configured an account with these identifiers."
       }
     }
   },

package/src/locales/fr.json

@@ -204,6 +204,10 @@
       "VENDOR_DOWN.LINXO_DOWN": {
         "title": "Service non disponible",
         "description": "Il semble que le service [%{name}](%{link}) ne nous ait pas répondu dans les temps. Vous pouvez tenter de le relancer manuellement maintenant ou ultérieurement."
+      },
+      "ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED": {
+        "title": "Ce compte est déjà configuré",
+        "description": "Vous avez déjà configuré un compte avec ces identifiants."
       }
     }
   },

package/src/services/budget-insight.js

@@ -9,7 +9,9 @@ import get from 'lodash/get'
 import omit from 'lodash/omit'
 import clone from 'lodash/clone'
 import set from 'lodash/set'
+import keyBy from 'lodash/keyBy'
 import defaults from 'lodash/defaults'
+import { Q } from 'cozy-client'
 
 import { waitForRealtimeEvent } from './jobUtils'
 import {
@@ -42,7 +44,9 @@ const getBIIdFromContract = bankAccount => bankAccount.vendorId
  * front-end.
  */
 const extraBIErrorMap = {
-  config: 'LOGIN_FAILED'
+  config: 'LOGIN_FAILED',
+  ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED:
+    'ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED'
 }
 
 /**
@@ -166,10 +170,26 @@ export const createOrUpdateBIConnection = async ({
       }
     }
     logger.info('Creating or updating connection...')
-    const connection = await (connId
+    const isUpdate = Boolean(connId)
+    const connection = await (isUpdate
       ? updateBIConnection(config, connId, credsToSend, tempToken)
       : createBIConnection(config, credsToSend, tempToken))
 
+    if (!isUpdate) {
+      const sameAccount = await findAccountWithBiConnection({
+        client,
+        konnector,
+        connectionId: connection.id
+      })
+      if (sameAccount) {
+        const err = new KonnectorJobError(
+          'ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED'
+        )
+        err.accountId = sameAccount._id
+        throw err
+      }
+    }
+
     logger.info(`Created or updated connection ${connection.id}`)
     return connection
   } catch (e) {
@@ -505,6 +525,44 @@ const shouldResumeConnection = error => {
   )
 }
 
+/**
+ * Tries to find an existing account, associated to an existing trigger
+ * with the given bi connection id
+ *
+ * @param  {CozyClient} options.client - Cozy client
+ * @param  {Object} options.konnector - Konnector manifest
+ * @param  {Integer} options.connectionId - BI connection id
+ * @return {Account|null} An account with a trigger with the same identifier if any
+ */
+export const findAccountWithBiConnection = async ({
+  client,
+  konnector,
+  connectionId
+}) => {
+  const [accountsResult, triggersResult] = await Promise.all([
+    client.query(
+      Q('io.cozy.accounts')
+        .where({ data: { auth: { bi: { connId: connectionId } } } })
+        .indexFields(['data.auth.bi.connId'])
+    ),
+    client.query(
+      Q('io.cozy.triggers')
+        .where({
+          message: {
+            konnector: konnector.slug
+          }
+        })
+        .indexFields(['message.konnector'])
+    )
+  ])
+
+  const accountsIndex = keyBy(accountsResult.data, '_id')
+  const trigger = triggersResult.data.find(
+    t => accountsIndex[get(t, 'message.account')]
+  )
+  return trigger ? accountsIndex[trigger.message.account] : null
+}
+
 export const konnectorPolicy = {
   name: 'budget-insight',
   match: isBudgetInsightConnector,
@@ -515,5 +573,6 @@ export const konnectorPolicy = {
   fetchExtraOAuthUrlParams: fetchExtraOAuthUrlParams,
   getAdditionalInformationNeeded,
   handleOAuthAccount,
-  setSync
+  setSync,
+  findAccountWithBiConnection
 }

package/src/services/budget-insight.spec.js

@@ -297,6 +297,7 @@ describe('createOrUpdateBIConnection', () => {
         }
       }
     })
+    jest.spyOn(client, 'query').mockImplementation(async () => ({ data: [] }))
 
     createBIConnection
       .mockReset()
@@ -604,6 +605,127 @@ describe('createOrUpdateBIConnection', () => {
       }
     })
   })
+
+  it('should refuse to create an account with a bi connection id which already exists', async () => {
+    const { client, flow } = setup()
+
+    const account = {
+      auth: {
+        login: '1234',
+        password: '4567',
+        dob: '20/12/1890',
+        bankId: '100000'
+      }
+    }
+
+    const konnector = {
+      slug: 'bankingconnectortest'
+    }
+
+    jest.spyOn(flow, 'saveAccount').mockImplementation(account => ({
+      _id: 'created-account-id',
+      ...account
+    }))
+
+    jest.spyOn(client, 'query').mockImplementation(async ({ doctype }) => {
+      if (doctype === 'io.cozy.accounts') {
+        return {
+          data: [
+            {
+              _id: 'account_id',
+              auth: { bi: { connId: 12 } }
+            }
+          ]
+        }
+      } else if (doctype === 'io.cozy.triggers') {
+        return {
+          data: [
+            {
+              message: { account: 'account_id' }
+            }
+          ]
+        }
+      } else {
+        throw new Error('unexpected doctype ' + doctype)
+      }
+    })
+
+    await expect(
+      onBIAccountCreation({
+        client,
+        flow,
+        account,
+        konnector
+      })
+    ).rejects.toEqual(new Error('ACCOUNT_WITH_SAME_IDENTIFIER_ALREADY_DEFINED'))
+  })
+
+  it('should create an account if bi connection is not already used', async () => {
+    const { client, flow } = setup()
+
+    const account = {
+      auth: {
+        login: '1234',
+        password: '4567',
+        dob: '20/12/1890',
+        bankId: '100000'
+      }
+    }
+
+    const konnector = {
+      slug: 'bankingconnectortest'
+    }
+
+    jest.spyOn(flow, 'saveAccount').mockImplementation(account => ({
+      _id: 'created-account-id',
+      ...account
+    }))
+
+    jest.spyOn(client, 'query').mockImplementation(async ({ doctype }) => {
+      if (doctype === 'io.cozy.accounts') {
+        return {
+          data: [
+            {
+              _id: 'account_id',
+              auth: { bi: { connId: 12 } }
+            }
+          ]
+        }
+      } else if (doctype === 'io.cozy.triggers') {
+        return {
+          data: [
+            {
+              message: { account: 'other_account_id' }
+            }
+          ]
+        }
+      } else {
+        throw new Error('unexpected doctype ' + doctype)
+      }
+    })
+
+    await expect(
+      onBIAccountCreation({
+        client,
+        flow,
+        account,
+        konnector
+      })
+    ).resolves.toEqual({
+      _id: 'created-account-id',
+      auth: {
+        bankId: '100000',
+        login: '1234'
+      },
+      data: {
+        auth: {
+          bi: {
+            connId: 'created-bi-connection-id-789'
+          }
+        }
+      }
+    })
+  })
 })
 
 describe('fetchExtraOAuthUrlParams', () => {