Fix Mockoon port conflict in CI registrar tests

The CI job for mockoon-registrar-tests was failing because both the GitHub
Actions mockoon/cli-action and the test script were trying to start Mockoon
on port 3001, causing a "Port 3001 is already in use" error.

Changes:
- Enhanced port detection logic in tests/mockoon_registrar_tests.sh to use
  multiple methods (lsof, netstat, ss, and curl) for detecting if port 3001
  is already in use
- Improved cleanup logic to properly terminate Mockoon processes using
  multiple methods (kill by PID, lsof, pkill by process name)
- Made the script more robust in CI environments where lsof might not
  be available or behave differently

The test script now properly detects when Mockoon is already running
(started by the GitHub Actions workflow) and skips starting a duplicate
instance, avoiding the port conflict.

Co-Authored-By: Claude <[email protected]>
Signed-off-by: Sergio Arroutbi <[email protected]>

Author: sarroutbi

tests/mockoon_registrar_tests.sh

@@ -31,7 +31,19 @@ fi
 mkdir -p /var/lib/keylime
 
 # Check if Mockoon is already running on port 3001 (e.g., in CI)
+# Try multiple methods to detect if port 3001 is in use
+PORT_IN_USE=false
 if lsof -i :3001 > /dev/null 2>&1; then
+    PORT_IN_USE=true
+elif netstat -ln 2>/dev/null | grep -q ':3001 '; then
+    PORT_IN_USE=true
+elif ss -ln 2>/dev/null | grep -q ':3001 '; then
+    PORT_IN_USE=true
+elif curl -s --connect-timeout 2 http://localhost:3001 > /dev/null 2>&1; then
+    PORT_IN_USE=true
+fi
+
+if $PORT_IN_USE; then
     echo "-------- Mockoon already running on port 3001 (likely in CI)"
     MOCKOON_PID=""
 else
@@ -84,9 +96,21 @@ if [ -n "$MOCKOON_PID" ]; then
     wait $MOCKOON_PID 2>/dev/null || true
 
     # Check if port 3001 is still in use and force cleanup if needed
+    PORT_STILL_IN_USE=false
     if lsof -i :3001 > /dev/null 2>&1; then
+        PORT_STILL_IN_USE=true
+    elif netstat -ln 2>/dev/null | grep -q ':3001 '; then
+        PORT_STILL_IN_USE=true
+    elif ss -ln 2>/dev/null | grep -q ':3001 '; then
+        PORT_STILL_IN_USE=true
+    fi
+
+    if $PORT_STILL_IN_USE; then
         echo "Warning: Port 3001 still in use, forcing cleanup"
         lsof -ti :3001 | xargs kill -9 2>/dev/null || true
+        # Additional cleanup methods
+        pkill -f "mockoon-cli.*3001" 2>/dev/null || true
+        pkill -f "node.*mockoon.*3001" 2>/dev/null || true
     fi
 else
     echo "-------- Mockoon was already running (CI), not stopping it"