[BUGFIX] Fix transaction et updateAt dans le repo organization-for-admin

#9267
1024pix · Jun 14, 2024 · 7a785d7 · 7a785d7
2 parents ab7e034 + 054056c
commit 7a785d7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/.../organizational-entities/infrastructure/repositories/organization-for-admin.repository.js b/.../organizational-entities/infrastructure/repositories/organization-for-admin.repository.js
@@ -67,7 +67,7 @@ const findChildrenByParentOrganizationId = async function (parentOrganizationId)
  * @return {Promise<OrganizationForAdmin|NotFoundError>}
  */
 const get = async function (id, domainTransaction = DomainTransaction.emptyTransaction()) {
-  const knexConn = domainTransaction.transaction ?? knex;
+  const knexConn = domainTransaction.knexTransaction ?? knex;
   const organization = await knexConn(ORGANIZATIONS_TABLE_NAME)
     .select({
       id: 'organizations.id',
@@ -161,7 +161,7 @@ const save = async function (organization) {
  * @return {Promise<void>}
  */
 const update = async function (organization, domainTransaction = DomainTransaction.emptyTransaction()) {
-  const knexConn = domainTransaction.transaction ?? knex;
+  const knexConn = domainTransaction.knexTransaction ?? knex;
   const organizationRawData = _.pick(organization, [
     'credit',
     'documentationUrl',
@@ -185,7 +185,9 @@ const update = async function (organization, domainTransaction = DomainTransacti
   await _addTags(knexConn, organization.tagsToAdd);
   await _removeTags(knexConn, organization.tagsToRemove);
 
-  await knexConn(ORGANIZATIONS_TABLE_NAME).update(organizationRawData).where({ id: organization.id });
+  await knexConn(ORGANIZATIONS_TABLE_NAME)
+    .update({ ...organizationRawData, updatedAt: new Date() })
+    .where({ id: organization.id });
 };
 
 /**

diff --git a/...ntities/integration/infrastructure/repositories/organization-for-admin.repository.test.js b/...ntities/integration/infrastructure/repositories/organization-for-admin.repository.test.js
@@ -527,6 +527,7 @@ describe('Integration | Organizational Entities | Infrastructure | Repository |
       // then
       const updatedOrganization = await knex('organizations').where({ id: childOrganization.id }).first();
       expect(updatedOrganization.parentOrganizationId).to.equal(parentOrganizationId);
+      expect(updatedOrganization.updatedAt).to.deep.equal(new Date('2022-02-02'));
     });
 
     it('should enable feature', async function () {