fix: separate app and server to fix open handle warning in tests

Author: vikashkrdeveloper

package.json

@@ -4,7 +4,7 @@
   "description": "Simple CI/CD backend testing",
   "main": "server.js",
   "scripts": {
-    "start": "node server.js",
+    "start": "node src/start.js",
     "test": "jest --detectOpenHandles"
   },
   "dependencies": {

server.js

@@ -1,10 +1,15 @@
 const request = require('supertest');
-const app = require('./server');
+const app = require('./src/server');  // Only the app, no server listening!
 
 describe('GET /', () => {
   it('responds with Hello CI/CD World!', async () => {
-    const response = await request(app).get('/');
-    expect(response.text).toBe('Hello CI/CD World!');
-    expect(response.statusCode).toBe(200);
+    const res = await request(app).get('/');
+    expect(res.text).toBe('Hello CI/CD World!');
   });
 });
+describe('GET /nonexistent', () => {
+  it('responds with 404 Not Found', async () => {
+    const res = await request(app).get('/nonexistent');
+    expect(res.statusCode).toBe(404);
+  });
+});

server.test.js

@@ -0,0 +1,8 @@
+const express = require('express');
+const app = express();
+
+app.get('/', (req, res) => {
+  res.send('Hello CI/CD World!');
+});
+
+module.exports = app;

src/server.js

@@ -0,0 +1,8 @@
+const app = require('./server');
+const port = 3000;
+
+app.listen(port, () => {
+  console.log(`Server running on port ${port}`);
+  console.log(`http://localhost:${port}`);
+  console.log('Press Ctrl+C to stop the server');
+});