test: fix api tests

matter-labs · Jan 10, 2024 · 6ca42f1 · 6ca42f1
1 parent e8d4697
commit 6ca42f1
Show file tree

Hide file tree

Showing 4 changed files with 234 additions and 32 deletions.
diff --git a/packages/integration-tests/tests/api/batches.test.ts b/packages/integration-tests/tests/api/batches.test.ts
@@ -30,8 +30,8 @@ describe("/batches", () => {
   });
 
   //@id1514
-  xit("Verify the response via /batches/{batchNumber}", async () => {
-    await setTimeout(localConfig.standardPause); //works unstable without timeout
+  it("Verify the response via /batches/{batchNumber}", async () => {
+    await setTimeout(localConfig.extendedPause); //works unstable without timeout
 
     const batches = await request(environment.blockExplorerAPI).get("/batches");
 
@@ -56,6 +56,6 @@ describe("/batches", () => {
       .expect((res) => expect(typeof res.body.l1GasPrice).toStrictEqual("string"))
       .expect((res) => expect(typeof res.body.l2FairGasPrice).toStrictEqual("string"))
       .expect((res) => expect(typeof res.body.size).toStrictEqual("number"))
-      .expect((res) => expect(res.body.status).toStrictEqual("string"));
+      .expect((res) => expect(typeof res.body.status).toStrictEqual("string"));
   });
 });
diff --git a/packages/integration-tests/tests/api/contracts.test.ts b/packages/integration-tests/tests/api/contracts.test.ts
@@ -5,6 +5,7 @@ import { localConfig } from "../../src/config";
 import { Buffer, Wallets } from "../../src/entities";
 import { Helper } from "../../src/helper";
 import { Playbook } from "../../src/playbook/playbook";
+import { setTimeout } from 'timers/promises';
 
 describe("Contracts API", () => {
   jest.setTimeout(localConfig.standardTimeout);
@@ -24,12 +25,14 @@ describe("Contracts API", () => {
     });
 
     //@id1696
-    xit("Verify the response via /api?module=contract&action=getcontractcreation&contractaddresses={address1},{address2}", async () => {
+    it("Verify the response via /api?module=contract&action=getcontractcreation&contractaddresses={address1},{address2}", async () => {
+      await setTimeout(localConfig.standardPause);
       paymasterContract = await helper.getStringFromFile(bufferFile + Buffer.paymaster);
       paymasterTx = await helper.getStringFromFile(bufferFile + Buffer.paymasterDeployTx);
       multicallCallerContract = await helper.getStringFromFile(bufferFile + Buffer.addressMultiCallCaller);
       multicallCallerTx = await helper.getStringFromFile(bufferFile + Buffer.txMultiCallCaller);
       const apiRoute = `/api?module=contract&action=getcontractcreation&contractaddresses=${paymasterContract},${multicallCallerContract}`;
+      console.log("apiRoute: " + apiRoute);
       return request(environment.blockExplorerAPI)
         .get(apiRoute)
         .expect(200)

diff --git a/packages/integration-tests/tests/api/logs.test.ts b/packages/integration-tests/tests/api/logs.test.ts
@@ -5,8 +5,9 @@ import { localConfig } from "../../src/config";
 import { Buffer } from "../../src/entities";
 import { Helper } from "../../src/helper";
 import { Playbook } from "../../src/playbook/playbook";
+import { setTimeout } from 'timers/promises';
 
-xdescribe("Logs API", () => {
+describe("Logs API", () => {
   jest.setTimeout(localConfig.standardTimeout); //works unstable without timeout
   const helper = new Helper();
   const bufferFile = "src/playbook/";
@@ -22,6 +23,7 @@ xdescribe("Logs API", () => {
 
     //@id1808
     it("Verify the response via /api?module=logs&action=getLogs&page={page}&offset={offset}0&toBlock={toBlock}&fromBlock={fromBlock}&address={address}", async () => {
+      await setTimeout(localConfig.standardPause);
       contractAddress = await helper.getStringFromFile(bufferFile + Buffer.greeterL2);
       txHash = await helper.getStringFromFile(bufferFile + Buffer.executeGreeterTx);