Fix/service agreement prunning migration fix (#3250)

* Fix missing return in getCountOfServiceAgreementsByBlockchainAndContract * Add commnet for removeServiceAgreementsByBlockchainAndContract * Version bump
OriginTrail · Jul 19, 2024 · f12c6cb · f12c6cb
1 parent 15069ae
commit f12c6cb
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
     "name": "origintrail_node",
-    "version": "6.5.1+hotfix.1",
+    "version": "6.5.1+hotfix.2",
     "description": "OTNode V6",
     "main": "index.js",
     "type": "module",

diff --git a/src/modules/repository/implementation/sequelize/repositories/service-agreement-repository.js b/src/modules/repository/implementation/sequelize/repositories/service-agreement-repository.js
@@ -288,7 +288,7 @@ class ServiceAgreementRepository {
     }
 
     async getCountOfServiceAgreementsByBlockchainAndContract(blockchainId, contract) {
-        await this.model.count({
+        return this.model.count({
             where: {
                 blockchainId,
                 assetStorageContractAddress: {
@@ -298,6 +298,7 @@ class ServiceAgreementRepository {
         });
     }
 
+    // Sequelize destroy method doesn't support limit
     async removeServiceAgreementsByBlockchainAndContract(blockchainId, contract) {
         const query = `
             DELETE FROM service_agreement