ace-test-runner 0.19.1 → 0.25.5

data/lib/ace/test_runner/suite/process_monitor.rb

@@ -97,7 +97,7 @@ module Ace
               exit_status = thread.value.exitstatus
 
               collect_remaining_output(process_info)
-              results = build_results(process_info, elapsed)
+              results = build_results(process_info, elapsed, exit_status)
               close_streams(process_info)
 
               # Final callback
@@ -182,8 +182,8 @@ module Ace
         def build_command(package, options)
           cmd_parts = ["ace-test"]
 
-          # Always run in single batch for suite execution
-          # This avoids nested group headers and improves performance
+          # Suite package execution intentionally bypasses grouped mode so each
+          # package runs its full target scope as one batch under suite orchestration.
           cmd_parts << "--run-in-single-batch"
 
           # Add format (use progress if compact is specified since ace-test doesn't have compact format)
@@ -201,6 +201,9 @@ module Ace
             cmd_parts << "--report-dir" << pkg_report_dir
           end
 
+          target = options["target"]
+          cmd_parts << target if target && !target.to_s.empty?
+
           cmd_parts
         end
 
@@ -275,11 +278,12 @@ module Ace
           end
         end
 
-        def build_results(process_info, elapsed)
+        def build_results(process_info, elapsed, exit_status)
           return timeout_results(process_info, elapsed) if process_info[:timeout_triggered]
           return interrupted_results(elapsed) if process_info[:terminated_by] == :interrupt
-          results = load_summary_results(process_info, elapsed)
-          return results if results
+          fresh_summary = load_summary_results(process_info, elapsed)
+          return fresh_summary if exit_status == 0 && fresh_summary
+          return failed_process_results(process_info, elapsed, exit_status, fresh_summary) if exit_status != 0
 
           parse_results(process_info[:output])
         end
@@ -308,6 +312,26 @@ module Ace
           }
         end
 
+        def failed_process_results(process_info, elapsed, exit_status, fresh_summary)
+          return fresh_summary if fresh_summary && fresh_summary[:success] == false
+
+          parsed = parse_results([process_info[:output], process_info[:stderr_output]].join("\n"))
+          parsed[:duration] ||= elapsed
+          parsed[:success] = false
+          parsed[:error] ||= failure_message_for(process_info, exit_status)
+          parsed[:errors] = 1 if parsed[:failures].to_i == 0 && parsed[:errors].to_i == 0
+          parsed
+        end
+
+        def failure_message_for(process_info, exit_status)
+          stderr = process_info[:stderr_output].to_s.strip
+          stdout = process_info[:output].to_s.strip
+          message = stderr.empty? ? stdout : stderr
+          return message unless message.empty?
+
+          "ace-test exited with status #{exit_status}"
+        end
+
         def load_summary_results(process_info, elapsed)
           package = process_info[:package]
           reports_dir = Atoms::ReportPathResolver.report_directory(
@@ -317,6 +341,7 @@ module Ace
           )
           summary_file = reports_dir ? File.join(reports_dir, "summary.json") : nil
           return nil unless summary_file && File.exist?(summary_file)
+          return nil unless summary_fresh_for_run?(summary_file, process_info[:start_time])
 
           json_data = JSON.parse(File.read(summary_file))
           results = {
@@ -324,6 +349,7 @@ module Ace
             assertions: json_data["assertions"] || 0,
             failures: json_data["failed"] || 0,
             errors: json_data["errors"] || 0,
+            skipped: json_data["skipped"] || 0,
             duration: json_data["duration"] || elapsed,
             success: json_data["success"] || false
           }
@@ -341,6 +367,12 @@ module Ace
           nil
         end
 
+        def summary_fresh_for_run?(summary_file, start_time)
+          File.mtime(summary_file) >= (start_time - 0.001)
+        rescue StandardError
+          false
+        end
+
         def parse_progress(process_info, chunk)
           # Count dots, F, E, S in the output for progress
           dots = chunk.scan(/[.FES]/).join

data/lib/ace/test_runner/suite/result_aggregator.rb

@@ -47,6 +47,9 @@ module Ace
 
         def collect_results
           @packages.map do |package|
+            runtime = runtime_result(package)
+            next runtime if runtime_result_overrides_summary?(package, runtime)
+
             reports_dir = Atoms::ReportPathResolver.report_directory(
               package["path"],
               report_root: @report_root,
@@ -91,7 +94,7 @@ module Ace
                 }
               end
             else
-              runtime_result(package) || {
+              runtime || {
                 package: package["name"],
                 path: package["path"],
                 report_root: @report_root,
@@ -106,6 +109,13 @@ module Ace
           end
         end
 
+        def runtime_result_overrides_summary?(package, runtime)
+          return false unless runtime
+
+          status = @runtime_results[package["name"]] || {}
+          status[:timed_out] || status[:interrupted] || runtime[:success] == false
+        end
+
         def collect_failed_packages(results)
           results.select { |r| !r[:success] }.map do |result|
             {

data/lib/ace/test_runner/version.rb

@@ -2,6 +2,6 @@
 
 module Ace
   module TestRunner
-    VERSION = '0.19.1'
+    VERSION = '0.25.5'
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: ace-test-runner
 version: !ruby/object:Gem::Version
-  version: 0.19.1
+  version: 0.25.5
 platform: ruby
 authors:
 - Michal Czyz
 bindir: exe
 cert_chain: []
-date: 2026-03-29 00:00:00.000000000 Z
+date: 2026-04-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ace-support-cli
@@ -189,9 +189,9 @@ files:
 - lib/ace/test_runner/formatters/progress_formatter.rb
 - lib/ace/test_runner/models/test_configuration.rb
 - lib/ace/test_runner/models/test_failure.rb
-- lib/ace/test_runner/models/test_group.rb
 - lib/ace/test_runner/models/test_report.rb
 - lib/ace/test_runner/models/test_result.rb
+- lib/ace/test_runner/models/test_target.rb
 - lib/ace/test_runner/molecules/cli_argument_parser.rb
 - lib/ace/test_runner/molecules/config_loader.rb
 - lib/ace/test_runner/molecules/deprecation_fixer.rb
@@ -206,7 +206,7 @@ files:
 - lib/ace/test_runner/molecules/test_executor.rb
 - lib/ace/test_runner/organisms/agent_reporter.rb
 - lib/ace/test_runner/organisms/report_generator.rb
-- lib/ace/test_runner/organisms/sequential_group_executor.rb
+- lib/ace/test_runner/organisms/sequential_target_executor.rb
 - lib/ace/test_runner/organisms/test_orchestrator.rb
 - lib/ace/test_runner/rake_task.rb
 - lib/ace/test_runner/suite.rb