fix: using MongoMemoryServer in test only

Author: JuliaHa0902

src/test/db/mongo-helper.js

@@ -9,64 +9,64 @@ let mongoServer;
 
 // Simplified MongoDB connection for CI environments
 module.exports.dbConnect = async () => {
-  try {
-    console.log('=== Starting MongoDB Connection Process ===');
-
-    // Disconnect any existing connections
-    if (mongoose.connection.readyState !== 0) {
-      console.log('Disconnecting existing MongoDB connection...');
-      await mongoose.disconnect();
-    }
-
-    // Try to use a real MongoDB connection if available, otherwise use a simple in-memory approach
-    const mongoUri = process.env.MONGODB_LOCAL_TEST_URI || 'mongodb://localhost:27017/test';
-
-    console.log('Using MongoDB URI:', mongoUri);
-
-    // Simple connection options
-    const mongooseOpts = {
-      useNewUrlParser: true,
-      useUnifiedTopology: true,
-      serverSelectionTimeoutMS: 30000, // 30 seconds
-      socketTimeoutMS: 30000, // 30 seconds
-      connectTimeoutMS: 30000, // 30 seconds
-      maxPoolSize: 1,
-      minPoolSize: 0,
-    };
+  // try {
+  console.log('=== Starting MongoDB Connection Process ===');
 
-    console.log('Connecting to MongoDB...');
-    await mongoose.connect(mongoUri, mongooseOpts);
-
-    console.log('MongoDB connection established successfully');
-    console.log('=== MongoDB Connection Process Complete ===');
-  } catch (error) {
-    console.error('MongoDB connection failed:', error);
-    console.error('Connection state:', mongoose.connection.readyState);
-
-    // If connection fails, try to create a minimal test environment
-    console.log('Attempting to create minimal test environment...');
+  // Disconnect any existing connections
+  if (mongoose.connection.readyState !== 0) {
+    console.log('Disconnecting existing MongoDB connection...');
     await mongoose.disconnect();
-
-    mongoServer = await MongoMemoryServer.create();
-
-    const uri = mongoServer.getUri();
-
-    const mongooseOpts = {
-      useNewUrlParser: true,
-      useUnifiedTopology: true,
-      serverSelectionTimeoutMS: 30000, // 30 seconds
-      socketTimeoutMS: 30000, // 30 seconds
-      connectTimeoutMS: 30000, // 30 seconds
-      maxPoolSize: 1,
-      minPoolSize: 0,
-    };
-
-    await mongoose.connect(uri, mongooseOpts, (err) => {
-      if (err) {
-        console.error(err);
-      }
-    });
   }
+
+  // Try to use a real MongoDB connection if available, otherwise use a simple in-memory approach
+  //   const mongoUri = process.env.MONGODB_LOCAL_TEST_URI || 'mongodb://localhost:27017/test';
+
+  //   console.log('Using MongoDB URI:', mongoUri);
+
+  //   // Simple connection options
+  //   const mongooseOpts = {
+  //     useNewUrlParser: true,
+  //     useUnifiedTopology: true,
+  //     serverSelectionTimeoutMS: 30000, // 30 seconds
+  //     socketTimeoutMS: 30000, // 30 seconds
+  //     connectTimeoutMS: 30000, // 30 seconds
+  //     maxPoolSize: 1,
+  //     minPoolSize: 0,
+  //   };
+
+  //   console.log('Connecting to MongoDB...');
+  //   await mongoose.connect(mongoUri, mongooseOpts);
+
+  //   console.log('MongoDB connection established successfully');
+  //   console.log('=== MongoDB Connection Process Complete ===');
+  // } catch (error) {
+  //   console.error('MongoDB connection failed:', error);
+  //   console.error('Connection state:', mongoose.connection.readyState);
+
+  //   // If connection fails, try to create a minimal test environment
+  //   console.log('Attempting to create minimal test environment...');
+  //   await mongoose.disconnect();
+
+  mongoServer = await MongoMemoryServer.create();
+
+  const uri = mongoServer.getUri();
+
+  const mongooseOpts = {
+    useNewUrlParser: true,
+    useUnifiedTopology: true,
+    serverSelectionTimeoutMS: 30000, // 30 seconds
+    socketTimeoutMS: 30000, // 30 seconds
+    connectTimeoutMS: 30000, // 30 seconds
+    maxPoolSize: 1,
+    minPoolSize: 0,
+  };
+
+  await mongoose.connect(uri, mongooseOpts, (err) => {
+    if (err) {
+      console.error(err);
+    }
+  });
+  // }
 };
 
 module.exports.dbDisconnect = async () => {