refactor: move REST tests from Docker to regular test suite

- Add test-server:writable script without --read-only flag
- Add test:rest script to run REST tests
- Include test:rest in main test command
- Update GitHub workflow to run REST tests with writable server
- Remove REST tests from docker-compose acceptance tests

This fixes the data cleanup issues caused by --read-only mode
in Docker environment. REST tests don't require Docker isolation
and can run like other unit/runner tests.

Results: 37/38 tests pass (vs 7 failures in Docker)

Author: kobenguyent

.github/workflows/acceptance-tests.yml

@@ -37,11 +37,6 @@ jobs:
         run: docker-compose build
         working-directory: test
 
-      # Run rest tests using docker-compose
-      - name: Run REST Tests
-        run: docker-compose run --rm test-rest
-        working-directory: test
-
       # Run WebDriverIO acceptance tests using docker-compose
       - name: Run WebDriverIO Acceptance Tests
         run: docker-compose run --rm test-acceptance.webdriverio

.github/workflows/test.yml

@@ -36,7 +36,16 @@ jobs:
             curl -sSf http://localhost:3001/api/users && break
             sleep 1
           done
+      - name: Start test server
+        run: nohup npm run test-server:writable &
+      - name: Wait for test server
+        run: |
+          for i in {1..20}; do
+            curl -sSf http://localhost:8010/posts && break
+            sleep 1
+          done
       - run: npm run test:unit
+      - run: npm run test:rest
       - name: Stop mock server
         run: npm run mock-server:stop
 

package.json

@@ -49,6 +49,7 @@
   "repository": "Codeception/codeceptjs",
   "scripts": {
     "test-server": "node bin/test-server.js test/data/rest/db.json --host 0.0.0.0 -p 8010 --read-only",
+    "test-server:writable": "node bin/test-server.js test/data/rest/db.json --host 0.0.0.0 -p 8010",
     "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",
@@ -58,8 +59,9 @@
     "prettier": "prettier --config prettier.config.js --write bin/**/*.js lib/**/*.js test/**/*.js translations/**/*.js runok.cjs",
     "docs": "./runok.cjs docs",
     "test:unit": "mocha test/unit --recursive --timeout 10000 --reporter @testomatio/reporter/mocha",
+    "test:rest": "mocha test/rest --recursive --timeout 20000 --reporter @testomatio/reporter/mocha",
     "test:runner": "mocha test/runner --recursive --timeout 10000 --reporter @testomatio/reporter/mocha",
-    "test": "npm run test:unit && npm run test:runner",
+    "test": "npm run test:unit && npm run test:rest && npm run test:runner",
     "test:appium-quick": "mocha test/helper/Appium_test.js --grep 'quick' --reporter @testomatio/reporter/mocha",
     "test:appium-other": "mocha test/helper/Appium_test.js --grep 'second' --reporter @testomatio/reporter/mocha",
     "test:ios:appium-quick": "mocha test/helper/Appium_ios_test.js --grep 'quick' --reporter @testomatio/reporter/mocha",