macOS CI: 1 worker, task test longer sleep, tools on macOS

- API/task workers: 1+1 (halves 84s startup to ~42s)
- Task tests: 60s pickup wait on darwin instead of 1s
- Server detection: HTTP readiness check (not just port open)
- Tools job: add macOS matrix (skip build-logger, bazel)

Author: gamesh411

.github/workflows/test.yml

@@ -46,23 +46,31 @@ jobs:
 
   tools:
     name: Tools (report-converter, etc.)
-    runs-on: ubuntu-24.04
+    runs-on: ${{ matrix.os }}
+    continue-on-error: ${{ matrix.os == 'macos-latest' }}
+
+    strategy:
+      matrix:
+        os: [ubuntu-24.04, macos-latest]
 
     steps:
       - uses: actions/checkout@v2
       - uses: actions/setup-python@v4
         with:
           python-version: '3.10'
       - name: Setup Bazel
+        if: runner.os == 'Linux'
         uses: abhinavsingh/setup-bazel@v3
         with:
           version: 4.0.0
       - name: Install common dependencies
+        if: runner.os == 'Linux'
         run: |
           sudo apt-get update -q
           sudo apt-get install gcc-multilib
 
       - name: Run build-logger tests
+        if: runner.os == 'Linux'
         working-directory: analyzer/tools/build-logger
         run: |
           pip install -r requirements_py/dev/requirements.txt
@@ -95,6 +103,7 @@ jobs:
           make test
 
       - name: Run bazel-compile-commands tests
+        if: runner.os == 'Linux'
         working-directory: tools/bazel
         run: |
           pip install -r requirements_py/dev/requirements.txt
@@ -215,8 +224,8 @@ jobs:
           cd web
           make test_matrix_sqlite
         env:
-          CC_TEST_API_WORKERS: "2"
-          CC_TEST_TASK_WORKERS: "2"
+          CC_TEST_API_WORKERS: "1"
+          CC_TEST_TASK_WORKERS: "1"
 
       - name: Run unit tests coverage
         working-directory: web

web/tests/functional/tasks/test_task_management.py

@@ -13,20 +13,22 @@
 from datetime import datetime, timezone
 import os
 import shutil
-import unittest
+import sys
 import time
+import unittest
 from typing import List, Optional, cast
 
 import multiprocess
 
-import sys
-
 from codechecker_api_shared.ttypes import RequestFailed, Ternary
 from codechecker_api.codeCheckerServersideTasks_v6.ttypes import \
     AdministratorTaskInfo, TaskFilter, TaskInfo, TaskStatus
 
 from libtest import codechecker, env
 
+# On macOS, spawn workers need time to import before processing tasks.
+_TASK_PICKUP_WAIT = 60 if sys.platform == "darwin" else 1
+
 
 # Stop events for the CodeChecker servers.
 STOP_SERVER = multiprocess.Event()
@@ -39,9 +41,6 @@
 # Note: Test names in this file follow a strict ordinal convention, because
 # the assertions are created with a specific execution history!
 
-@unittest.skipIf(sys.platform == "darwin",
-                 "Spawn workers take ~84s to import on macOS CI; "
-                 "task timing assumptions (1s) are incompatible.")
 class TaskManagementAPITests(unittest.TestCase):
     def setup_class(self):
         global TEST_WORKSPACE
@@ -139,7 +138,7 @@ def setup_method(self, _):
     def test_task_1_query_status(self):
         task_token = self._anonymous_task_client.createDummyTask(2, False)
 
-        time.sleep(1)
+        time.sleep(_TASK_PICKUP_WAIT)
         task_info: TaskInfo = self._anonymous_task_client.getTaskInfo(
             task_token)
         self.assertEqual(task_info.token, task_token)
@@ -166,7 +165,7 @@ def test_task_1_query_status(self):
     def test_task_2_query_status_of_failed(self):
         task_token = self._anonymous_task_client.createDummyTask(2, True)
 
-        time.sleep(1)
+        time.sleep(_TASK_PICKUP_WAIT)
         task_info: TaskInfo = self._anonymous_task_client.getTaskInfo(
             task_token)
         self.assertEqual(task_info.token, task_token)
@@ -183,7 +182,7 @@ def test_task_2_query_status_of_failed(self):
     def test_task_3_cancel(self):
         task_token = self._anonymous_task_client.createDummyTask(3, False)
 
-        time.sleep(1)
+        time.sleep(_TASK_PICKUP_WAIT)
         cancel_req: bool = self._privileged_task_client.cancelTask(task_token)
         self.assertTrue(cancel_req)
 
@@ -366,7 +365,7 @@ def test_task_5_info_query_filters(self):
         # Let every task terminate. We should only need 1 second per task,
         # running likely in a multithreaded environment.
         # Let's have some leeway, though...
-        time.sleep(2)
+        time.sleep(_TASK_PICKUP_WAIT + 2)
 
         task_infos = self._privileged_task_client.getTasks(TaskFilter(
             enqueuedAfterEpoch=current_time_epoch,

web/tests/libtest/codechecker.py

@@ -687,17 +687,17 @@ def wait_for_server_start(stdoutfile, port=None):
                           f"{n}s. Output:")
                     print(out[-2000:])
 
-        # Fallback: check if port is accepting connections.
+        # Fallback: check if server can handle HTTP requests.
         if port and n > 3:
-            import socket
+            import urllib.request
             try:
-                s = socket.create_connection(
-                    ("localhost", port), timeout=1)
-                s.close()
-                print(f"Server port {port} open after {n}s "
-                      "(file detection missed it)")
+                urllib.request.urlopen(
+                    f"http://localhost:{port}/", timeout=1)
+            except urllib.error.HTTPError:
+                # Any HTTP response (even 404) means server is ready.
+                print(f"Server responding on port {port} after {n}s")
                 return
-            except (ConnectionRefusedError, OSError):
+            except (ConnectionRefusedError, OSError, urllib.error.URLError):
                 pass
 
         if n > server_start_timeout.total_seconds():