Run tests for Nest.js project in CI

funidata · Aug 1, 2023 · d1dcea1 · d1dcea1
1 parent 5234506
commit d1dcea1
Show file tree

Hide file tree

Showing 8 changed files with 83 additions and 39 deletions.
diff --git a/.github/workflows/build_image.yaml b/.github/workflows/build_image.yaml
@@ -1,6 +1,6 @@
 name: Build and upload image
 
-on: 
+on:
   workflow_call:
 
 jobs:
@@ -23,3 +23,23 @@ jobs:
         with:
           name: hybridilusmu
           path: /tmp/hybridilusmu.tar
+
+  build-nest:
+    name: Build and upload image (Nest.js)
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+      - name: Set up Docker Buildx
+        uses: docker/setup-buildx-action@v2
+      - name: Build and export
+        uses: docker/build-push-action@v4
+        with:
+          context: app-nest
+          tags: hybridilusmu-nest
+          outputs: type=docker,dest=/tmp/hybridilusmu-nest.tar
+      - name: Upload artifact
+        uses: actions/upload-artifact@v3
+        with:
+          name: hybridilusmu-nest
+          path: /tmp/hybridilusmu-nest.tar
diff --git a/.github/workflows/run_tests.yaml b/.github/workflows/run_tests.yaml
@@ -33,35 +33,54 @@ jobs:
         run: |
           docker load --input /tmp/hybridilusmu.tar
           docker image ls -a
-      - name: Start containers
-        run: docker compose -f compose.yaml -f compose-ci.yaml up -d lusmu
       - name: Run unit tests
-        run: npm test
+        run: docker compose -f compose.yaml -f compose-ci.yaml run lusmu npm test
+      - name: Stop containers
+        if: always()
+        run: npm stop
+
+  unit-tests-nest:
+    name: Unit tests (Nest.js)
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+      - name: Set up Docker Buildx
+        uses: docker/setup-buildx-action@v2
+      - name: Download artifact
+        uses: actions/download-artifact@v3
+        with:
+          name: hybridilusmu-nest
+          path: /tmp
+      - name: Load image
+        run: |
+          docker load --input /tmp/hybridilusmu-nest.tar
+          docker image ls -a
+      - name: Run unit tests
+        run: docker compose -f compose.yaml -f compose-ci.yaml run lusmu-nest npm test
+      - name: Stop containers
+        if: always()
+        run: npm stop
+
+  integration-tests-nest:
+    name: Integration tests (Nest.js)
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+      - name: Set up Docker Buildx
+        uses: docker/setup-buildx-action@v2
+      - name: Download artifact
+        uses: actions/download-artifact@v3
+        with:
+          name: hybridilusmu-nest
+          path: /tmp
+      - name: Load image
+        run: |
+          docker load --input /tmp/hybridilusmu-nest.tar
+          docker image ls -a
+      - name: Run integration tests
+        run: docker compose -f compose.yaml -f compose-ci.yaml run lusmu-nest npm run test:integration
       - name: Stop containers
         if: always()
         run: npm stop
-# FIXME: Enable these once integration tests exit cleanly.
-# integration-tests:
-#   name: Integration tests
-#   runs-on: ubuntu-latest
-#   needs: build
-#   steps:
-#     - name: Checkout
-#       uses: actions/checkout@v3
-#     - name: Set up Docker Buildx
-#       uses: docker/setup-buildx-action@v2
-#     - name: Download artifact
-#       uses: actions/download-artifact@v3
-#       with:
-#         name: hybridilusmu
-#         path: /tmp
-#     - name: Load image
-#       run: |
-#         docker load --input /tmp/hybridilusmu.tar
-#         docker image ls -a
-#     - name: Start containers
-#       run: docker compose -f compose.yaml -f compose-ci.yaml up -d lusmu
-#     - name: Run integration tests
-#       run: npm run test:integration
-#     - name: Stop containers
-#       run: npm stop
diff --git a/app-nest/package.json b/app-nest/package.json
@@ -25,7 +25,7 @@
     "test:watch": "jest --watch",
     "test:cov": "jest --coverage",
     "test:debug": "node --inspect-brk -r tsconfig-paths/register -r ts-node/register node_modules/.bin/jest --runInBand",
-    "test:integration": "jest --config ./test/jest-integration.json"
+    "test:integration": "jest --config ./test/jest-integration.json -w 1"
   },
   "dependencies": {
     "@golevelup/nestjs-discovery": "^4.0.0",

diff --git a/app-nest/test/app.int-spec.ts b/app-nest/test/app.int-spec.ts
@@ -11,13 +11,14 @@ describe("App", () => {
     await app.init();
   });
 
-  afterAll(async () => {
+  afterEach(async () => {
     await app.close();
   });
 
   it("Find all users", async () => {
     const userService = app.get(UserService);
     const users = await userService.findAll();
     expect(users).toHaveLength(1);
+    expect(1).toEqual(1);
   });
 });
diff --git a/app-nest/test/jest-integration.json b/app-nest/test/jest-integration.json
@@ -9,5 +9,6 @@
   "testRegex": ".int-spec.ts$",
   "transform": {
     "^.+\\.(t|j)s$": "ts-jest"
-  }
+  },
+  "workerIdleMemoryLimit": "512MB"
 }
diff --git a/compose-ci.yaml b/compose-ci.yaml
@@ -1,3 +1,6 @@
 services:
   lusmu:
-    image: hybridilusmu-ci
+    image: hybridilusmu
+
+  lusmu-nest:
+    image: hybridilusmu-nest
diff --git a/compose-dev.yaml b/compose-dev.yaml
@@ -11,12 +11,6 @@ services:
     command: npm run watch
 
   lusmu-nest:
-    environment:
-      NODE_ENV: development
-      DATABASE_HOST: db
-      DATABASE_NAME: nest
-      DATABASE_USERNAME: postgres
-      DATABASE_PASSWORD: postgres
     env_file:
       - .env.development
     command: npm run start:dev
diff --git a/compose.yaml b/compose.yaml
@@ -14,6 +14,12 @@ services:
   lusmu-nest:
     container_name: lusmu-nest
     build: app-nest/
+    environment:
+      NODE_ENV: development
+      DATABASE_HOST: db
+      DATABASE_NAME: nest
+      DATABASE_USERNAME: postgres
+      DATABASE_PASSWORD: postgres
     volumes:
       - ./app-nest/src:/usr/src/app/src
       - ./app-nest/test:/usr/src/app/test