Merge pull request #17 from ChinZJ/feature/user-service

test(user-service) Add tests to Models, Services and Utils

Author: verakohh

.github/workflows/codecov.yml

@@ -26,6 +26,7 @@ jobs:
         run: npx jest --coverage
 
       - name: Upload results to Codecov
+        if: always()
         uses: codecov/codecov-action@v5
         with:
           token: ${{ secrets.CODECOV_TOKEN }}

frontend/src/components/authContainer.tsx

@@ -15,6 +15,9 @@ const AuthContainer: React.FC = () => {
   }
   if (user) {
     if (!user.profileComplete && window.location.pathname !== '/complete-profile') {
+      console.log('do i make it here bro');
+      console.log(user);
+      console.log('profile complete is ', user.profileComplete);
       return <Navigate to="/complete-profile" replace />;
     }
     const isVerified = user.verified || user.googleOAuthVerified || user.githubOAuthVerified;

jest.config.ts

@@ -233,7 +233,7 @@ const config: Config = {
             tsconfig: {
               esModuleInterop: true,
               allowSyntheticDefaultImports: true,
-              module: 'commonjs', // Add this line
+              module: 'commonjs',
             },
           },
         ],

package-lock.json

@@ -36,6 +36,7 @@
     "@types/express": "^5.0.3",
     "@types/jest": "^30.0.0",
     "@types/jsonwebtoken": "^9.0.10",
+    "@types/mongodb-memory-server": "^1.8.0",
     "@types/multer": "^2.0.0",
     "@types/passport": "^1.0.17",
     "@types/passport-github2": "^1.2.9",
@@ -76,7 +77,7 @@
     "fast-csv": "^5.0.5",
     "helmet": "^8.1.0",
     "jsonwebtoken": "^9.0.2",
-    "mongoose": "^8.18.2",
+    "mongoose": "^8.19.1",
     "multer": "^2.0.2",
     "passport": "^0.7.0",
     "passport-github2": "^0.1.12",

package.json

@@ -25,7 +25,7 @@
     "express": "^5.1.0",
     "fast-csv": "^5.0.5",
     "jsonwebtoken": "^9.0.2",
-    "mongoose": "^8.18.2",
+    "mongoose": "^8.19.1",
     "multer": "^2.0.2",
     "passport": "^0.7.0",
     "passport-github2": "^0.1.12",
@@ -41,10 +41,12 @@
     "@types/cors": "^2.8.19",
     "@types/express": "^5.0.3",
     "@types/jsonwebtoken": "^9.0.10",
+    "@types/mongodb-memory-server": "^1.8.0",
     "@types/node": "^24.3.1",
     "@types/passport": "^1.0.17",
     "@types/passport-github2": "^1.2.9",
     "@types/passport-google-oauth20": "^2.0.16",
+    "mongodb-memory-server": "^10.2.3",
     "nodemon": "^3.1.10",
     "ts-node": "^10.9.2",
     "ts-node-dev": "^2.0.0",

user-service/package.json

@@ -1,13 +1,34 @@
 import dotenv from 'dotenv';
 import path from 'path';
-import {fileURLToPath} from 'url';
+
+// Force search to project root to bypass CJS and ESNext conflicts.
+const TARGET_DIR = 'cs3219-ay2526s1-project-g03';
+const findProjectRoot = (targetDirName = TARGET_DIR): string => {
+  let currentDir = process.cwd();
+
+  while (true) {
+    const base = path.basename(currentDir);
+    if (base == targetDirName) {
+      return currentDir;
+    }
+
+    const parentDir = path.dirname(currentDir);
+    if (parentDir === currentDir) {
+      throw new Error(`Could not find directory ${targetDirName} in path hierarchy.`);
+    }
+
+    currentDir = parentDir;
+  }
+};
+
+dotenv.config({path: path.join(findProjectRoot(), '.env')});
 
 // Solution adapted from:
 // https://stackoverflow.com/questions/64383909/dirname-is-not-defined-error-in-node-js-14-version
-const filename = fileURLToPath(import.meta.url);
-const dirname = path.dirname(filename);
+// const filename = fileURLToPath(import.meta.url);
+// const dirname = path.dirname(filename);
 
-dotenv.config({path: path.resolve(dirname, '../../../.env')});
+// dotenv.config({path: path.resolve(dirname, '../../../.env')});
 
 /**
  * Processes all environment variables.

user-service/src/constants/env.ts

@@ -1,4 +1,4 @@
-const enum OAuthType {
+enum OAuthType {
   Google = 'google',
   GitHub = 'github',
 }

user-service/src/constants/jwtAudience.ts

@@ -1,4 +1,4 @@
-const enum UserRoleTypes {
+enum UserRoleTypes {
   User = 'user',
   Admin = 'admin',
 }