tests differ for server-side and client-side failures

BlockIo · Nov 20, 2020 · 8c165be · 8c165be
1 parent 9f77e1f
commit 8c165be
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 4 deletions.
diff --git a/test/helpers/clienttest.js b/test/helpers/clienttest.js
@@ -26,6 +26,10 @@ const ERR_CHECKS = [
   }
 ];
 
+const ERR_DATA_CHECK = (t, args) => {
+  t.ok(typeof args[0].data == 'object', 'must return response data');
+}
+
 function ClientTest(framework, client) {
   this.framework = framework;
   this.client = client;
@@ -60,6 +64,11 @@ ClientTest.prototype.fails = function () {
   return this;
 }
 
+ClientTest.prototype.failsServerSide = function () {
+  this._tests.push(ERR_DATA_CHECK);
+  return this.fails();
+}
+
 ClientTest.prototype.returnsTx = function () {
   TX_CHECKS.forEach(fn => this._tests.push(fn));
   return this;

diff --git a/test/integration/api.js b/test/integration/api.js
@@ -205,7 +205,7 @@ cache.require(['minFee', 'newAddress', 'fromAddress', 'fromLabel'], () => {
 
     CT.create(test, badApiKeyClient).title('Validate API key (invalid key)')
       .method('validate_api_key')
-      .fails()
+      .failsServerSide()
       .execute();
 
     CT.create(test, client).title('Get network fee estimate')

diff --git a/test/integration/dtrust.js b/test/integration/dtrust.js
@@ -73,7 +73,7 @@ CT.create(test, client).title('Get New DTrust Address (too high required sigs)')
     required_signatures: KEYS.length + 1,
     public_keys: KEYS.map(key => key.pub.toString('hex')).join(',')
   })
-  .fails()
+  .failsServerSide()
   .execute();
 
 CT.create(test, client).title('Get New DTrust Address (duplicate signers)')
@@ -83,7 +83,7 @@ CT.create(test, client).title('Get New DTrust Address (duplicate signers)')
     required_signatures: KEYS.length,
     public_keys: KEYS.map(() => KEYS[0].pub.toString('hex')).join(',')
   })
-  .fails()
+  .failsServerSide()
   .execute();
 
 CT.create(test, client).title('Get DTrust Addresses')
@@ -255,7 +255,7 @@ cache.require(['dtrustWithdrawal'], () => {
   CT.create(test, client).title('Sending in an already finished withdrawal')
   .method('sign_and_finalize_withdrawal')
   .payload({ signature_data: JSON.stringify(w.data) })
-  .fails()
+  .failsServerSide()
   .execute();
 
 });