Merge pull request #14 from hayleyashby7/fix-test-run-on-build

remove problematic addition to workflow

.github/workflows/CI and Deploy.yml

@@ -18,10 +18,18 @@ jobs:
               with:
                   node-version: '20.x'
             - run: npm ci
+            - run: npm install -g json-server
+            - run: json-server --watch db.json --routes routes.json --port 3000 &
             - run: npm run build --if-present
             - run: npm test
               env:
-                  DB_URL: ${{ secrets.DB_URL }}
+                  NODE_ENV: 'production'
+                  CLIENT_TOKEN: ${{ secrets.CLIENT_TOKEN }}
+                  CLIENT_ID: ${{ secrets.CLIENT_ID }}
+                  KANKA_TOKEN: ${{ secrets.KANKA_TOKEN }}
+                  DB_TOKEN: ${{ secrets.DB_TOKEN }}
+                  DB_URL: http://localhost:3000
+
     deploy:
         name: Fly Deploy
         needs: build
@@ -34,3 +42,4 @@ jobs:
             - run: flyctl deploy --remote-only
               env:
                   FLY_API_TOKEN: ${{ secrets.FLY_API_TOKEN }}
+                  DB_URL: ${{ secrets.DB_URL}}

src/config.ts

@@ -1,6 +1,8 @@
 import dotenv from 'dotenv';
 
-dotenv.config();
+if (process.env.NODE_ENV !== 'production') {
+    dotenv.config();
+}
 
 const { NODE_ENV, CLIENT_TOKEN, CLIENT_ID, KANKA_TOKEN, DB_TOKEN, DB_URL } = process.env;
 

src/mocks/db/db.json

@@ -0,0 +1,8 @@
+{
+    "stats": [
+        {
+            "name": "EchoesLeft",
+            "value": 123
+        }
+    ]
+}

src/mocks/db/routes.json

@@ -0,0 +1,3 @@
+{
+    "/stats?name=eq.EchoesLeft": "/stats/0"
+}

src/services/dbClient.ts

@@ -2,7 +2,7 @@ import axios, { AxiosRequestConfig, RawAxiosRequestHeaders } from 'axios';
 import { config } from '../config';
 
 export const dbClient = axios.create({
-    baseURL: config.DB_URL,
+    baseURL: process.env.NODE_ENV === 'test' ? 'http://localhost:3000' : config.DB_URL,
 });
 
 export const dbConfig: AxiosRequestConfig = {