Fix bad testing semantics for async errors

stellar · Sep 16, 2024 · 89e904b · 89e904b
1 parent 503fd0d
commit 89e904b
Showing 1 changed file with 13 additions and 6 deletions.
diff --git a/test/unit/server/soroban/get_contract_balance_test.js b/test/unit/server/soroban/get_contract_balance_test.js
@@ -135,12 +135,19 @@ describe("Server#getContractBalance", function () {
   });
 
   it("throws on invalid addresses", function (done) {
-    expect(
-      this.server.getSACBalance(Keypair.random().publicKey(), token),
-    ).to.throw(/TypeError/);
+    this.server
+      .getSACBalance(Keypair.random().publicKey(), token)
+      .then(() => done(new Error("Error didn't occur")))
+      .catch((err) => {
+        expect(err).to.match(/TypeError/);
+      });
 
-    expect(
-      this.server.getSACBalance(contract.substring(0, -1), token),
-    ).to.throw(/TypeError/);
+    this.server
+      .getSACBalance(contract.substring(0, -1), token)
+      .then(() => done(new Error("Error didn't occur")))
+      .catch((err) => {
+        expect(err).to.match(/TypeError/);
+        done();
+      });
   });
 });