Merge pull request #317 from proto-kit/feature/lint-integration-tests

Lint integration tests

Author: rpanic

package.json

@@ -9,6 +9,7 @@
     "build:clean": "npx nx reset && npm run build",
     "build-docker": "docker build -t protokit-base -f ./packages/deployment/docker/base/Dockerfile .",
     "lint": "npx lerna run lint",
+    "lint:fix": "npx lerna run lint -- --fix",
     "lint:staged": "eslint",
     "test": "npx lerna run test -- --passWithNoTests",
     "test:ci": "npx lerna run test --concurrency=1 -- --passWithNoTests --forceExit",

packages/persistance/.eslintrc

@@ -1,7 +1,10 @@
 {
   "extends": "../../.eslintrc",
+  "parserOptions": {
+    "project": ["./tsconfig.json", "./test/tsconfig.json", "./test-integration/tsconfig.json"]
+  },
   "rules": {
     "no-underscore-dangle": "off",
     "@typescript-eslint/consistent-type-assertions": "off"
   }
-}
+}

packages/sequencer/.eslintrc

@@ -0,0 +1,6 @@
+{
+  "extends": ["../../.eslintrc"],
+  "parserOptions": {
+    "project": ["./tsconfig.json", "./test/tsconfig.json", "./test-integration/tsconfig.json"]
+  },
+}

packages/sequencer/package.json

@@ -7,7 +7,7 @@
   "scripts": {
     "build": "tsc -p tsconfig.json",
     "dev": "tsc -p tsconfig.json --watch",
-    "lint": "eslint ./src ./test",
+    "lint": "eslint ./src ./test ./test-integration",
     "test:file": "node --experimental-vm-modules --experimental-wasm-modules ../../node_modules/jest/bin/jest.js",
     "test": "npm run test:file -- ./test/**",
     "test:watch": "npm run test:file -- ./test/** --watch",

packages/sequencer/test-integration/ArchiveNode.test.ts

@@ -1,7 +1,6 @@
 import { describe } from "@jest/globals";
 
 import { ArchiveNode } from "../src/settlement/utils/ArchiveNode";
-import { TokenId } from "o1js";
 
 describe("ArchiveNode", () => {
   it("should resolve", async () => {

packages/sequencer/test-integration/Settlement-lightnet.test.ts

@@ -1,8 +1,8 @@
 import { PrivateKey } from "o1js";
 import { log } from "@proto-kit/common";
+import { FungibleToken } from "mina-fungible-token";
 
 import { settlementTestFn } from "../test/settlement/Settlement";
-import { FungibleToken } from "mina-fungible-token";
 
 console.log(PrivateKey.random().toPublicKey().toBase58());
 

packages/sequencer/test-integration/benchmarks/tps.test.ts

@@ -27,7 +27,6 @@ import {
   DatabasePruneModule,
   ManualBlockTrigger,
   Sequencer,
-  AppChain,
 } from "../../src";
 // we import PrivateMempool from dist to satisfy constraints of InMemoryTransactionSender
 import { PrivateMempool } from "../../dist";

packages/sequencer/test-integration/workers/ChildProcessWorker.ts

@@ -21,14 +21,17 @@ export class ChildProcessWorker {
       }
     );
     s.on("error", (err) => {
+      // eslint-disable-next-line no-console
       console.error(err);
     });
     if (forwardLogs) {
       s.stdout.on("data", (data) => {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
         process.stdout.write(data);
       });
     }
     s.stderr.on("data", (data) => {
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
       process.stderr.write(data);
     });
 
@@ -37,6 +40,7 @@ export class ChildProcessWorker {
 
   kill() {
     this.process!.kill("SIGKILL");
+    // eslint-disable-next-line no-console
     console.log("Killed", this.process!.killed);
   }
 }

packages/sequencer/test-integration/workers/modules.ts

@@ -1,7 +1,9 @@
 import { Runtime } from "@proto-kit/module";
 import { Protocol } from "@proto-kit/protocol";
+// eslint-disable-next-line import/no-extraneous-dependencies
 import { VanillaProtocolModules } from "@proto-kit/library";
 import { ModulesConfig } from "@proto-kit/common";
+// eslint-disable-next-line import/no-extraneous-dependencies
 import { BullQueueConfig } from "@proto-kit/deployment";
 
 import { ProvenBalance } from "../../test/integration/mocks/ProvenBalance";

packages/sequencer/test-integration/workers/worker.test.ts

@@ -65,6 +65,8 @@ describe("worker", () => {
         .containerEvents.on("ready", res);
     });
 
+    expect(ready).toBe(true);
+
     console.log("Ready received!");
 
     await sleep(10000000);