modified the connection string part

Author: deviant101

.github/workflows/devsecops-pipeline.yml

@@ -33,10 +33,9 @@ jobs:
 
       - name: Run unit tests
         run: npm test
+        continue-on-error: false  # Fail pipeline if tests fail
         env:
           MONGO_URI: ${{ secrets.MONGO_URI }}
-          MONGO_USERNAME: ${{ secrets.MONGO_USERNAME }}
-          MONGO_PASSWORD: ${{ secrets.MONGO_PASSWORD }}
 
       - name: Upload test results
         if: always()
@@ -68,8 +67,6 @@ jobs:
         run: npm run coverage
         env:
           MONGO_URI: ${{ secrets.MONGO_URI }}
-          MONGO_USERNAME: ${{ secrets.MONGO_USERNAME }}
-          MONGO_PASSWORD: ${{ secrets.MONGO_PASSWORD }}
 
       - name: Upload coverage reports
         uses: actions/upload-artifact@v4
@@ -296,8 +293,6 @@ jobs:
           docker run -d --name solar-system-app \
             -p 3000:3000 \
             -e MONGO_URI="${{ secrets.MONGO_URI }}" \
-            -e MONGO_USERNAME="${{ secrets.MONGO_USERNAME }}" \
-            -e MONGO_PASSWORD="${{ secrets.MONGO_PASSWORD }}" \
             ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}:${{ github.ref_name }}-${{ github.sha }}
           
           # Wait for application to be ready
@@ -352,8 +347,6 @@ jobs:
   #             -p 3000:3000 \
   #             --restart unless-stopped \
   #             -e MONGO_URI="${{ secrets.MONGO_URI }}" \
-  #             -e MONGO_USERNAME="${{ secrets.MONGO_USERNAME }}" \
-  #             -e MONGO_PASSWORD="${{ secrets.MONGO_PASSWORD }}" \
   #             ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}:${{ github.ref_name }}-${{ github.sha }}
 
   #           # Clean up old images (keep last 3)

.gitignore

@@ -3,4 +3,8 @@ solar-system.png
 .nyc_output
 .talismanrc
 coverage
-test-results.xml
+test-results.xml
+
+# Environment variables (DO NOT COMMIT)
+.env
+.env.local

app-test.js

@@ -8,6 +8,35 @@ let chaiHttp = require("chai-http");
 chai.should();
 chai.use(chaiHttp); 
 
+// Wait for MongoDB connection before running tests
+before(function(done) {
+    this.timeout(15000); // Increase timeout for MongoDB connection
+    
+    // Check if already connected
+    if (mongoose.connection.readyState === 1) {
+        console.log('MongoDB already connected');
+        return done();
+    }
+    
+    // Wait for connection
+    mongoose.connection.on('connected', function() {
+        console.log('MongoDB connection established for tests');
+        done();
+    });
+    
+    mongoose.connection.on('error', function(err) {
+        console.error('MongoDB connection error:', err.message);
+        done(err);
+    });
+    
+    // Timeout fallback
+    setTimeout(function() {
+        if (mongoose.connection.readyState !== 1) {
+            done(new Error('MongoDB connection timeout'));
+        }
+    }, 14000);
+});
+
 describe('Planets API Suite', () => {
 
     describe('Fetching Planet Details', () => {

app.js

@@ -14,15 +14,13 @@ app.use(express.static(path.join(__dirname, '/')));
 app.use(cors())
 
 mongoose.connect(process.env.MONGO_URI, {
-    user: process.env.MONGO_USERNAME,
-    pass: process.env.MONGO_PASSWORD,
     useNewUrlParser: true,
     useUnifiedTopology: true
 }, function(err) {
     if (err) {
         console.log("error!! " + err)
     } else {
-      //  console.log("MongoDB Connection Successful")
+        console.log("MongoDB Connection Successful")
     }
 })