Dockerfile update to include knexfile (#32)

* VR-104: Dockerfile update - copy over knexfile along with the migrations folder.

* VR-104: temporary remove from the workflows.

* VR-104: missing dependency.

* VR-104 some timeout for database to actually get up.

* VR-104: also for test workflow 🤦.

---------

Co-authored-by: Paulius Michelevicius <[email protected]>

.github/workflows/release.yml

@@ -72,6 +72,16 @@ jobs:
             ${{ runner.OS }}-
       - name: Install Packages
         run: npm ci
+      - name: Setup dependencies
+        run: docker compose up -d
+      - name: Sleep
+        uses: kibertoad/[email protected]
+        with:
+          time: '30s'
+      - name: Run database migrations
+        run: npm run db:migrate
+        env:
+          NODE_ENV: test
       - name: Run tests
         run: npm run test
 

.github/workflows/test.yml

@@ -66,6 +66,16 @@ jobs:
             ${{ runner.OS }}-
       - name: Install Packages
         run: npm ci
+      - name: Setup dependencies
+        run: docker compose up -d
+      - name: Sleep
+        uses: kibertoad/[email protected]
+        with:
+          time: '30s'
+      - name: Run database migrations
+        run: npm run db:migrate
+        env:
+          NODE_ENV: test
       - name: Run tests
         run: npm run test
 

Dockerfile

@@ -25,6 +25,8 @@ COPY package*.json ./
 RUN npm ci --omit-dev
 
 COPY public ./public
+COPY migrations ./migrations
+COPY knexfile.js ./
 COPY --from=builder /veritable-ui/build ./build
 
 EXPOSE 80

package.json

@@ -1,6 +1,6 @@
 {
   "name": "veritable-ui",
-  "version": "0.3.6",
+  "version": "0.3.7",
   "description": "UI for Veritable",
   "main": "src/index.ts",
   "type": "commonjs",