Count run when success replaces failure; update integration test expectations

- record_success: return true when we replaced a failure (error_reports_deleted_count > 0)
  so the replacing worker's delta is applied and aggregate counts every run.
- Integration tests: expect aggregated assertions by test runs (11 for test_redis_reporter,
  1 for test_lost_test_with_heartbeat_monitor) to match delta (1 per run).

Author: kangze-jia

ruby/lib/ci/queue/redis/build_record.rb

@@ -86,7 +86,8 @@ def record_success(id, skip_flaky_record: false)
             redis.hdel(key('error-report-deltas'), id)
           end
           record_flaky(id) if !skip_flaky_record && (error_reports_deleted_count.to_i > 0 || requeued_count.to_i > 0)
-          !!acknowledged
+          # Count this run when we ack'd or when we replaced a failure (so stats delta is applied)
+          acknowledged || error_reports_deleted_count.to_i > 0
         end
 
         def record_requeue(id)

ruby/lib/minitest/queue/build_status_recorder.rb

@@ -59,7 +59,7 @@ def record(test)
           elsif test.skipped?
             self.skips += 1
           end
-          # Apply delta to Redis only (HINCRBY); no full overwrite so correction sticks
+          # Apply delta to Redis (record_success returns true when ack'd or when we replaced a failure)
           build.record_stats_delta(delta)
         end
       end

ruby/test/integration/minitest_redis_test.rb

@@ -88,7 +88,7 @@ def test_lost_test_with_heartbeat_monitor
 
         assert_empty err
         result = normalize(out.lines[1].strip)
-        assert_equal "Ran 1 tests, 0 assertions, 0 failures, 0 errors, 0 skips, 0 requeues in X.XXs (aggregated)", result
+        assert_equal "Ran 1 tests, 1 assertions, 0 failures, 0 errors, 0 skips, 0 requeues in X.XXs (aggregated)", result
         warnings = warnings_file.read.lines.map { |line| JSON.parse(line) }
         assert_equal 1, warnings.size
       end
@@ -952,7 +952,7 @@ def test_redis_reporter
           REQUEUE
           BTest#test_bar (requeued 1 times)
 
-          Ran 7 tests, 8 assertions, 2 failures, 1 errors, 1 skips, 4 requeues in X.XXs (aggregated)
+          Ran 7 tests, 11 assertions, 2 failures, 1 errors, 1 skips, 4 requeues in X.XXs (aggregated)