Create ci unit tests

.github/workflows/pr-momoka-node.yml

@@ -0,0 +1,32 @@
+name: Build, Lint and Unit Tests for Momoka NODE
+
+on:
+  pull_request:
+    types: [labeled, opened, synchronize, unlabeled]
+    # Run only when some modifications are perform in the momoka-node directory
+    paths:
+      - 'momoka-node/**'
+
+jobs:
+  build-lint-test:
+    name: Build-Lint-Test
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - uses: pnpm/action-setup@v2
+        with:
+          version: 8
+
+      - name: Install dependencies
+        run: cd momoka-node && pnpm install
+
+      - name: Lint the code
+        run: cd momoka-node && pnpm run lint
+
+      - name: Build the packages
+        run: cd momoka-node && pnpm run build
+
+      - name: Run the unit tests
+        run: cd momoka-node && pnpm run test:ci

.github/workflows/pr-momoka-rs.yml

@@ -0,0 +1,22 @@
+name: Build, Lint and Unit Tests for Momoka RUST
+
+on:
+  pull_request:
+    types: [labeled, opened, synchronize, unlabeled]
+    # Run only when some modifications are perform in the momoka-rs directory
+    paths:
+      - 'momoka-rs/**'
+
+jobs:
+  build-lint-test:
+    name: Build-Lint-Test
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Build 
+        run: cd momoka-rs && cargo build
+
+      - name: Run the unit tests
+        run: cd momoka-rs && cargo test

momoka-node/.env.ci

@@ -0,0 +1,3 @@
+ETHEREUM_NETWORK=MUMBAI
+NODE_URL="https://polygon-mumbai.g.alchemy.com/v2/s1Hq6Tyswh1mDXOlgawIqSCndvsNfYb6"
+CONCURRENCY=3

momoka-node/jest.config.js

@@ -1,9 +1,11 @@
 /** @type {import('ts-jest').JestConfigWithTsJest} */
+const { resolve } = require('path');
+
 module.exports = {
   preset: 'ts-jest',
   testEnvironment: 'node',
-  collectCoverage: true,
   coverageDirectory: 'coverage',
+  testRegex: [resolve(__dirname, '/src/__TESTS__/*/.*\\.test\\.ts$')],
   setupFiles: ['<rootDir>/src/__TESTS__/config/jest.setup.js'],
   coveragePathIgnorePatterns: [
     'node_modules',

momoka-node/package.json

@@ -16,15 +16,16 @@
     "README.md"
   ],
   "scripts": {
-    "build": "tsc",
+    "build": "rm -rf lib/ && tsc",
     "eslint:fix": "eslint \"src/**/*.ts\" --quiet --fix",
     "eslint": "eslint \"src/**/*.ts\" --quiet",
     "start:fork": "REQ_TIMEOUT=100000 anvil --fork-url NODE_URL --silent",
     "start": "env-cmd -f .env node lib/runnable/da-verifier-node.runnable.js",
     "start:failed": "env-cmd -f .env ts-node src/failed-submissons.runnable.ts",
-    "debug:playground": "npm run build && env-cmd -f .env node lib/__PLAYGROUND__/index.js",
+    "debug:playground": "pnpm run build && env-cmd -f .env node lib/__PLAYGROUND__/index.js",
     "generate": "graphql-codegen",
-    "test": "npm run build && env-cmd -f .env.test jest",
+    "test": "env-cmd -f .env.test jest --forceExit --coverage",
+    "test:ci": "env-cmd -f .env.ci jest --forceExit",
     "lint": "pnpm run prettier && pnpm run tsc",
     "lint:fix": "pnpm run prettier:fix && pnpm run eslint",
     "prettier:fix": "prettier --write .",

momoka-node/src/__TESTS__/comment.e2e.test.ts

@@ -1,5 +1,4 @@
 // apply mocks!
-jest.setTimeout(3000);
 jest.mock('../input-output/db');
 jest.mock('../input-output/bundlr/get-bundlr-by-id.api');
 jest.mock('../submitters');
@@ -16,15 +15,15 @@ describe('comment', () => {
   describe('with delegate', () => {
     let baseMock = commentCreatedDelegateArweaveResponse;
 
-    beforeEach(() => {
+    beforeAll(() => {
       baseMock = commentCreatedDelegateArweaveResponse;
       sharedMocks.mockGetDAPublicationByIdAPI.mockImplementation(async () =>
         deepClone(commentCreatedDelegateArweaveResponse)
       );
     });
 
     describe('should return success when', () => {
-      test('signed by delegate is true', async () => {
+      test('signed by delegate is true', () => {
         expect(baseMock.chainProofs.thisPublication.signedByDelegate).toBe(true);
       });
 
@@ -43,13 +42,11 @@ describe('comment', () => {
     describe('should return failure when', () => {
       test('NO_SIGNATURE_SUBMITTER', async () => {
         sharedMocks.mockImpl__NO_SIGNATURE_SUBMITTER(baseMock);
-
         await sharedMocks.checkAndValidateDAProof(MomokaValidatorError.NO_SIGNATURE_SUBMITTER);
       });
 
       xtest('INVALID_SIGNATURE_SUBMITTER', async () => {
         sharedMocks.mockIsValidSubmitter.mockImplementationOnce(() => false);
-
         await sharedMocks.checkAndValidateDAProof(MomokaValidatorError.INVALID_SIGNATURE_SUBMITTER);
       });
 
@@ -116,11 +113,12 @@ describe('comment', () => {
         await sharedMocks.checkAndValidateDAProof(MomokaValidatorError.PUBLICATION_NONCE_INVALID);
       });
 
-      xtest('PUBLICATION_SIGNER_NOT_ALLOWED', async () => {});
+      xtest('PUBLICATION_SIGNER_NOT_ALLOWED', () => {
+        // TODO: implement
+      });
 
       test('INVALID_FORMATTED_TYPED_DATA', async () => {
         sharedMocks.mockImpl__INVALID_FORMATTED_TYPED_DATA(baseMock);
-
         await sharedMocks.checkAndValidateDAProof(
           MomokaValidatorError.INVALID_FORMATTED_TYPED_DATA
         );
@@ -252,29 +250,33 @@ describe('comment', () => {
         await sharedMocks.checkAndValidateDAProof(MomokaValidatorError.EVENT_MISMATCH);
       });
 
-      xtest('SIMULATION_NODE_COULD_NOT_RUN', async () => {});
+      xtest('SIMULATION_NODE_COULD_NOT_RUN', () => {
+        // TODO: implement
+      });
 
-      xtest('UNKNOWN', () => {});
+      xtest('UNKNOWN', () => {
+        // TODO: implement
+      });
     });
   });
 
   describe('without delegate', () => {
     let baseMock = commentCreatedWithoutDelegateArweaveResponse;
 
-    beforeEach(() => {
+    beforeAll(() => {
       baseMock = commentCreatedWithoutDelegateArweaveResponse;
       sharedMocks.mockGetDAPublicationByIdAPI.mockImplementation(async () =>
         deepClone(commentCreatedWithoutDelegateArweaveResponse)
       );
     });
 
     describe('should return success when', () => {
-      test('signed by delegate is false', async () => {
+      test('signed by delegate is false', () => {
         expect(baseMock.chainProofs.thisPublication.signedByDelegate).toBe(false);
       });
 
       test('txExists in the db already', async () => {
-        sharedMocks.mockGetTxDb.mockImplementationOnce(async () => mockTxValidationResult);
+        sharedMocks.mockGetTxDb.mockImplementationOnce(async () => await mockTxValidationResult);
         const result = await sharedMocks.callCheckDAProof();
         expect(result.isSuccess()).toBe(true);
       });
@@ -361,11 +363,12 @@ describe('comment', () => {
         await sharedMocks.checkAndValidateDAProof(MomokaValidatorError.PUBLICATION_NONCE_INVALID);
       });
 
-      xtest('PUBLICATION_SIGNER_NOT_ALLOWED', async () => {});
+      xtest('PUBLICATION_SIGNER_NOT_ALLOWED', async () => {
+        // TODO: implement
+      });
 
       test('INVALID_FORMATTED_TYPED_DATA', async () => {
         sharedMocks.mockImpl__INVALID_FORMATTED_TYPED_DATA(baseMock);
-
         await sharedMocks.checkAndValidateDAProof(
           MomokaValidatorError.INVALID_FORMATTED_TYPED_DATA
         );
@@ -381,7 +384,6 @@ describe('comment', () => {
             },
           };
         });
-
         await sharedMocks.checkAndValidateDAProof(MomokaValidatorError.EVENT_MISMATCH);
       });
 
@@ -395,7 +397,6 @@ describe('comment', () => {
             },
           };
         });
-
         await sharedMocks.checkAndValidateDAProof(MomokaValidatorError.EVENT_MISMATCH);
       });
 
@@ -497,9 +498,13 @@ describe('comment', () => {
         await sharedMocks.checkAndValidateDAProof(MomokaValidatorError.EVENT_MISMATCH);
       });
 
-      xtest('SIMULATION_NODE_COULD_NOT_RUN', async () => {});
+      xtest('SIMULATION_NODE_COULD_NOT_RUN', () => {
+        // TODO: implement
+      });
 
-      xtest('UNKNOWN', () => {});
+      xtest('UNKNOWN', () => {
+        // TODO: implement
+      });
     });
   });
 });

momoka-node/src/__TESTS__/config/jest.setup.js

@@ -7,4 +7,4 @@ Object.defineProperty(globalThis, 'crypto', {
 });
 
 // eslint-disable-next-line no-undef
-jest.setTimeout(15000);
+jest.setTimeout(30000);