- Fix parallel test execution conflicts by moving test files to build dir
- Improve error handling in ci-build.sh with proper exit codes
- Update test file paths for out-of-tree build compatibility
- Add test-files and test-logs to .gitignore
- Add clean-functional-tests target to Makefile.am

@@ -91,2 +91,4 @@
 
 This will run the same tests that the CI runs and refuse the push if it fails.
+The CI script runs 4 parallel builds with different configurations, each with Valgrind and functional tests.
+Use `./ci-build.sh --coverage-only` to run only build 1 with coverage collection.

@@ -26,0 +40,4 @@
+
+# Parse STATS line from build log and set global variables
+# Usage: parse_build_stats <log_file>
+# Sets: STAT_UNIT_P, STAT_UNIT_F, STAT_FUNC_P, STAT_FUNC_F,

@@ -216,3 +261,3 @@
 
         # shellcheck disable=SC2086
-        ../configure -C $features $extra_args
+        if ! ../configure -C $features $extra_args; then

@@ -219,3 +267,2 @@
 
-        $MAKE CC="${CC}"
-        $MAKE check-functional-parallel
+        if ! $MAKE CC="${CC}"; then

@@ -224,0 +356,4 @@
+
+        # Save coverage.info before cleanup (for CI artifact)
+        if [ "$run_coverage" = "yes" ] && [ -f coverage.info ]; then
+            cp coverage.info ../coverage.info

@@ -243,0 +400,4 @@
+                run_valgrind="yes"
+                extra_flags="--enable-valgrind"
+            else
+                extra_flags=""

@@ -138,1 +138,4 @@
 {
+    const char *group_env = getenv("PROF_TEST_GROUP");
+    int group = group_env ? atoi(group_env) : 0;
+    int dir_id = (group >= 1 && group <= 4) ? group : stub_port;

@@ -229,0 +245,4 @@
+    
+    /* Calculate port base: each build uses a different range of 4 ports
+     * Build 0/1: 5230-5233, Build 2: 5234-5237, Build 3: 5238-5241, Build 4: 5242-5245 */
+    int port_base = 5230 + ((build_idx > 0 ? build_idx - 1 : 0) * 4);

@@ -234,0 +255,4 @@
+    if (group >= 1 && group <= 4) {
+        /* Static allocation: each group gets a dedicated port */
+        stub_port = port_base + group - 1;
+        fprintf(stderr, "[PROF_TEST] Build %d, Group %d: trying port %d\n", build_idx, group, stub_port);

@@ -234,2 +260,3 @@
+        if (stbbr_start(STBBR_LOGDEBUG, stub_port, 0) == 0) {
             started = TRUE;
-            break;
+            fprintf(stderr, "[PROF_TEST] Started stabber on port %d\n", stub_port);

@@ -238,2 +268,3 @@
+    /* Fallback to dynamic allocation if static failed or group=0 */
     if (!started) {
-        assert_true(FALSE); // could not start stabber on any port in range
+        fprintf(stderr, "[PROF_TEST] Using dynamic port allocation\n");

@@ -240,0 +272,4 @@
+            if (stbbr_start(STBBR_LOGDEBUG, p, 0) == 0) {
+                stub_port = p;
+                started = TRUE;
+                fprintf(stderr, "[PROF_TEST] Started stabber on port %d\n", stub_port);

@@ -240,0 +280,4 @@
+    
+    if (!started) {
+        fprintf(stderr, "[PROF_TEST] ERROR: could not start stabber on any port\n");
+        assert_true(FALSE);

@@ -247,1 +293,3 @@
-             "./tests/functionaltests/files/%d/xdg_data_home", stub_port);
+             "./test-files/%d/xdg_data_home", dir_id);
+    
+    fprintf(stderr, "[PROF_TEST] Group %d using directories: config=%s, data=%s\n", 

@@ -399,3 +448,3 @@
         }
         
-        usleep(50000);  /* 50ms */
+        usleep(50000);

@@ -229,0 +249,4 @@
+    /* Calculate port base: each build uses a different range of TEST_GROUPS ports.
+     * Build 0 (local/default): 5230-5233, Build 1: 5230-5233, Build 2: 5234-5237, etc.
+     * Build 0 and 1 share the same range because build 0 is for local runs (no parallel builds),
+     * while builds 1-4 are used in CI where they run in parallel. */

@@ -235,1 +267,3 @@
-            break;
+            printf("[PROF_TEST] Started stabber on port %d\n", stub_port);
+        } else {
+            printf("[PROF_TEST] Failed to start stabber on port %d\n", stub_port);