PP-11681 Refactor MockClient url test condition.

alphagov · Mar 8, 2024 · d02a386 · d02a386
1 parent 6009de2
commit d02a386
Showing 1 changed file with 10 additions and 24 deletions.
diff --git a/app/services/clients/ledger.client.test.js b/app/services/clients/ledger.client.test.js
@@ -61,30 +61,16 @@ class MockClient {
 
   async get (url, description) {
     let dataResponse
-    if (url.match(/\.*\/event/)) {
-      dataResponse = ledgerTransactionEventsFixture
-      return Promise.resolve({ data: dataResponse })
-    }
-    if (url.match(/\/transaction\//)) {
-      dataResponse = validCreatedTransactionDetailsResponse
-      return Promise.resolve({ data: dataResponse })
-    }
-    if (url.match(/\/report\/transactions-summary/)) {
-      dataResponse = validTransactionSummaryResponse
-      return Promise.resolve({ data: dataResponse })
-    }
-    if (url.match(/\/agreement/)) {
-      dataResponse = validTransactionSummaryResponse
-      return Promise.resolve({ data: dataResponse })
-    }
-    if (url.match(/\/payout/)) {
-      dataResponse = {}
-      return Promise.resolve({ data: dataResponse })
-    }
-    if (url.match(/\/transaction\?account_id.*limit_total/)) {
-      dataResponse = validTransactionSearchResponse
-      return Promise.resolve({ data: dataResponse })
-    }
+
+    if (url.match(/\.*\/event/)) dataResponse = ledgerTransactionEventsFixture
+    else if (url.match(/\/transaction\//)) dataResponse = validCreatedTransactionDetailsResponse
+    else if (url.match(/\/report\/transactions-summary/)) dataResponse = validTransactionSummaryResponse
+    else if (url.match(/\/agreement/)) dataResponse = validTransactionSummaryResponse
+    else if (url.match(/\/payout/)) dataResponse = {}
+    else if (url.match(/\/transaction\?account_id.*limit_total/)) dataResponse = validTransactionSearchResponse
+    else dataResponse = {}
+
+    return Promise.resolve({ data: dataResponse })
   }
 }