Add read-only mode to test-server to prevent data persistence

- Added readOnly config option to TestServer class
- Skip saving to disk when in read-only mode
- Added --read-only flag to bin/test-server.js
- Updated npm test-server script to use read-only mode by default
- This ensures db.json never gets modified during test runs
- Fixes data accumulation issue in CI where DELETE requests appeared to fail
- Removed start-json-server.sh script (no longer needed)

Author: kobenguyent

bin/test-server.js

@@ -17,6 +17,7 @@ const args = process.argv.slice(2)
 let dbFile = path.join(__dirname, '../test/data/rest/db.json')
 let port = 8010
 let host = '0.0.0.0'
+let readOnly = false
 
 // Simple argument parsing
 for (let i = 0; i < args.length; i++) {
@@ -26,15 +27,20 @@ for (let i = 0; i < args.length; i++) {
     port = parseInt(args[++i])
   } else if (arg === '--host') {
     host = args[++i]
+  } else if (arg === '--read-only' || arg === '-r') {
+    readOnly = true
   } else if (!arg.startsWith('-')) {
     dbFile = path.resolve(arg)
   }
 }
 
 // Create and start server
-const server = new TestServer({ port, host, dbFile })
+const server = new TestServer({ port, host, dbFile, readOnly })
 
 console.log(`Starting test server with db file: ${dbFile}`)
+if (readOnly) {
+  console.log('Running in READ-ONLY mode - changes will not be persisted to disk')
+}
 
 server
   .start()

lib/test-server.js

@@ -13,6 +13,7 @@ class TestServer {
     this.port = config.port || 8010
     this.host = config.host || 'localhost'
     this.dbFile = config.dbFile || path.join(__dirname, '../test/data/rest/db.json')
+    this.readOnly = config.readOnly || false
     this.lastModified = null
     this.data = this.loadData()
 
@@ -49,6 +50,10 @@ class TestServer {
   }
 
   saveData() {
+    if (this.readOnly) {
+      console.log('[Save] Skipping save - running in read-only mode')
+      return
+    }
     try {
       fs.writeFileSync(this.dbFile, JSON.stringify(this.data, null, 2))
       console.log('[Save] Data saved to file')

package.json

@@ -48,7 +48,7 @@
   },
   "repository": "Codeception/codeceptjs",
   "scripts": {
-    "test-server": "node bin/test-server.js test/data/rest/db.json --host 0.0.0.0 -p 8010",
+    "test-server": "node bin/test-server.js test/data/rest/db.json --host 0.0.0.0 -p 8010 --read-only",
     "mock-server:start": "node test/mock-server/start-mock-server.js",
     "mock-server:stop": "kill -9 $(lsof -t -i:3001)",
     "test:with-mock-server": "npm run mock-server:start && npm test",

test/data/rest/db.json

@@ -1 +1 @@
-{"comments":[],"posts":[{"id":1,"title":"json-server","author":"davert"}]}
+{"posts":[{"id":1,"title":"json-server","author":"davert"}],"user":{"name":"davert"}}

test/docker-compose.yml

@@ -70,7 +70,7 @@ services:
   json_server:
     <<: *test-service
     entrypoint: []
-    command: /bin/bash /codecept/test/scripts/start-json-server.sh
+    command: npm run test-server
     ports:
       - '8010:8010' # Expose to host
     restart: always # Automatically restart the container if it fails or becomes unhealthy