ceedling 1.1.5 → 1.1.7

data/spec/support/system/system_context.rb

@@ -6,6 +6,7 @@
 # =========================================================================
 
 require 'fileutils'
+require 'bundler'
 require 'ceedling/encodinator'
 require_relative 'gem_dir_layout'
 
@@ -44,16 +45,26 @@ class SystemContext
     )
 
     Dir.chdir(shared_dir) do
-      saved = ENV.to_hash
       begin
-        %w{BUNDLE_GEMFILE BUNDLE_BIN_PATH RUBYOPT}.each { |k| ENV.delete(k) }
-        deploy_output  = `bundle config set --local path '#{shared_gem.install_dir}' 2>&1`
-        # --prefer-local: Without it, Bundler resolves gems (e.g. `erb`) fresh from
-        # rubygems repository even when Ruby's default-gem copy satisfies the Gemfile constraint.
-        deploy_output += `bundle install --prefer-local 2>&1`
+        # Bundler.with_unbundled_env restores the pre-`bundle exec` environment for this
+        # block -- clearing not just BUNDLE_GEMFILE/BUNDLE_BIN_PATH/RUBYOPT (the previous,
+        # incomplete hand-rolled list here) but also GEM_HOME/GEM_PATH/RUBYLIB/PATH and every
+        # other var Bundler sets when this spec suite itself runs under `bundle exec` against
+        # a non-default GEM_HOME (e.g. `bundle install --path vendor/bundle`). Any one of
+        # those left in place is enough on its own for the `bundle exec ruby -S ceedling
+        # version` verification below to load the *outer* project's bundler/gems instead of
+        # this shared gem's own `--local path` install, and fail to find this Gemfile's exact
+        # pinned versions there.
+        deploy_output = Bundler.with_unbundled_env do
+          output  = `bundle config set --local path '#{shared_gem.install_dir}' 2>&1`
+          # --prefer-local: Without it, Bundler resolves gems (e.g. `erb`) fresh from
+          # rubygems repository even when Ruby's default-gem copy satisfies the Gemfile constraint.
+          output += `bundle install --prefer-local 2>&1`
+          output
+        end
         raise VerificationFailed, "bundle install failed:\n#{deploy_output}" unless $?.success?
 
-        verify = `bundle exec ruby -S ceedling version 2>&1`
+        verify = Bundler.with_unbundled_env { `bundle exec ruby -S ceedling version 2>&1` }
         unless $?.success?
           raise VerificationFailed,
             "Ceedling does not appear to be installed or ready for use.\n" \
@@ -62,8 +73,6 @@ class SystemContext
       rescue
         FileUtils.rm_rf(shared_dir)
         raise
-      ensure
-        ENV.replace(saved)
       end
     end
 
@@ -226,8 +235,11 @@ class SystemContext
     # encoding regardless of their actual content, so any project source content the
     # subprocess echoes (verbose logging, compiler output, etc.) can make later regex
     # matching against this string raise. Sanitize once here, the same way Ceedling itself
-    # sanitizes file content it scans.
-    @raw_output       = (stdout + stderr).clean_encoding
+    # sanitizes file content it scans -- reused for both the raw output and the failure
+    # report below, since a real compiler's own diagnostic text (e.g. clang's curly-quoted
+    # identifiers) is exactly the kind of non-ASCII content this needs to survive too.
+    stdout, stderr    = stdout.clean_encoding, stderr.clean_encoding
+    @raw_output       = stdout + stderr
     @console_summary  = compose_failure_report(stdout, stderr)
 
     SystemTestOutput.new(@raw_output)
@@ -241,7 +253,8 @@ class SystemContext
 
     @last_cmd         = cmd
     @last_exit_status = status.exitstatus
-    @raw_output       = (stdout + stderr).clean_encoding
+    stdout, stderr    = stdout.clean_encoding, stderr.clean_encoding
+    @raw_output       = stdout + stderr
     @console_summary  = compose_failure_report(stdout, stderr)
 
     SystemTestOutput.new(@raw_output)

data/spec/system/conditional_include_macro_visibility_spec.rb

@@ -0,0 +1,152 @@
+# =========================================================================
+#   Ceedling - Test-Centered Build System for C
+#   ThrowTheSwitch.org
+#   Copyright (c) 2010-26 Mike Karlesky, Mark VanderVoord, & Greg Williams
+#   SPDX-License-Identifier: MIT
+# =========================================================================
+
+require 'spec_system_helper'
+
+##
+## Conditional-#include Macro-Visibility Tests (directives-only preprocessing)
+## =============================================================================
+##
+## Ceedling discovers a file's #includes via two preprocessor passes reconciled
+## into one list: a "bare" pass and an accurate pass (gcc -E -fdirectives-only
+## against the real file with real search paths -- genuinely opens every
+## header). Reconciliation keeps an accurate-pass entry only if bare also found
+## it, to filter out headers reached only via a deeper, nested path.
+##
+## "Bare" is itself two things unioned together: a gcc -M -MG -MP pass against
+## an isolated, sibling-free copy of the file (so it never actually opens any
+## header the file #includes, but can still resolve an #include whose own
+## target is a macro, since command-line -D defines are visible even in
+## isolation), and a plain literal text scan of the file's own #include lines
+## (no conditional evaluation at all, so it sees past a guard the isolated gcc
+## pass can't evaluate -- but also can't resolve a macro-computed #include
+## target, since there's no literal filename in the source text to find).
+## Neither replaces the other; each catches what the other structurally can't.
+##
+## These tests characterize that reconciliation's behavior for #include
+## directives guarded by an #if/#ifdef, across the different places the
+## guarding macro can come from -- some already handled correctly, one
+## (issue #1223) fixed by adding the text-scan half of "bare" above.
+##
+## Test assets: assets/tests_with_conditional_includes/
+##   - widget.c/.h: three scenarios that already work correctly:
+##     (a) conditional include gated on a project :defines macro
+##     (b) conditional include gated on a same-file #define
+##     (c) a header reached only transitively (nested_wrapper.h's own
+##         #include, never itself directly #include'd by widget.c) --
+##         must not appear duplicated in as a false top-level entry
+##   - widget_feature.c/.h + feature_config.h + feature_extra.h: issue #1223
+##     itself -- a conditional include gated on a macro defined by an
+##     *earlier #include in the same file* (feature_config.h's FEATURE_LEVEL),
+##     which the isolated bare pass can never see, silently dropping
+##     feature_extra.h and leaving FEATURE_EXTRA_MACRO undeclared.
+##
+
+ceedling_system_tests do
+
+  before :all do
+    @c = SystemContext.new
+    @c.deploy_gem
+  end
+
+  after :all do
+    @c.done!
+  end
+
+  before { @proj_name = unique_proj_name("cond_inc") }
+
+  describe "Deployed as a gem" do
+    before do
+      @c.with_context do
+        @c.ceedling_appcmd_exec("new #{@proj_name}")
+      end
+    end
+
+    # =========================================================================
+    describe "Conditional includes that already resolve correctly today" do
+    # =========================================================================
+
+      before do
+        @c.with_context do
+          Dir.chdir @proj_name do
+            FileUtils.cp test_asset_path("tests_with_conditional_includes/src/widget.h"), 'src/'
+            FileUtils.cp test_asset_path("tests_with_conditional_includes/src/widget.c"), 'src/'
+            FileUtils.cp test_asset_path("tests_with_conditional_includes/src/project_flag_extra.h"), 'src/'
+            FileUtils.cp test_asset_path("tests_with_conditional_includes/src/local_flag_extra.h"), 'src/'
+            FileUtils.cp test_asset_path("tests_with_conditional_includes/src/nested_wrapper.h"), 'src/'
+            FileUtils.cp test_asset_path("tests_with_conditional_includes/src/nested_extra.h"), 'src/'
+            FileUtils.cp test_asset_path("tests_with_conditional_includes/test/test_widget.c"), 'test/'
+
+            settings = {
+              :project => { :use_partials => true },
+              :defines => { :test => ['PROJECT_FLAG'] }
+            }
+            @c.merge_project_yml_for_test(settings)
+          end
+        end
+      end
+
+      it "resolves a project-:defines-gated include, a same-file-#define-gated include, and correctly excludes a transitively-nested header from duplication" do
+        @c.with_context do
+          Dir.chdir @proj_name do
+            output = @c.ceedling_build_exec("test:widget")
+            expect(@c.last_exit_status).to eq(0)
+            expect(output).to match(/TESTED:\s+3/)
+            expect(output).to match(/PASSED:\s+3/)
+            expect(output).to match(/FAILED:\s+0/)
+
+            # (c) nested_extra.h is reached only transitively, through
+            # nested_wrapper.h's own #include -- confirm it is not promoted
+            # into a spurious, duplicated top-level #include in the generated
+            # Partial implementation (the exact filtering property the #1223
+            # fix must not break).
+            generated = Dir.glob('build/test/partials/**/*_impl.c').first
+            expect(generated).not_to be_nil
+            contents = File.read(generated)
+            expect(contents.scan(/#include\s+"nested_extra\.h"/).length).to eq(0)
+          end
+        end
+      end
+
+    end
+
+    # =========================================================================
+    describe "Conditional include gated on a macro from an earlier #include in the same file (issue #1223)" do
+    # =========================================================================
+
+      before do
+        @c.with_context do
+          Dir.chdir @proj_name do
+            FileUtils.cp test_asset_path("tests_with_conditional_includes/src/widget_feature.h"), 'src/'
+            FileUtils.cp test_asset_path("tests_with_conditional_includes/src/widget_feature.c"), 'src/'
+            FileUtils.cp test_asset_path("tests_with_conditional_includes/src/feature_config.h"), 'src/'
+            FileUtils.cp test_asset_path("tests_with_conditional_includes/src/feature_extra.h"), 'src/'
+            FileUtils.cp test_asset_path("tests_with_conditional_includes/test/test_widget_feature.c"), 'test/'
+
+            settings = { :project => { :use_partials => true } }
+            @c.merge_project_yml_for_test(settings)
+          end
+        end
+      end
+
+      it "keeps FEATURE_EXTRA_MACRO visible in the generated Partial so the build compiles" do
+        @c.with_context do
+          Dir.chdir @proj_name do
+            output = @c.ceedling_build_exec("test:widget_feature")
+            expect(@c.last_exit_status).to eq(0)
+            expect(output).to match(/TESTED:\s+1/)
+            expect(output).to match(/PASSED:\s+1/)
+            expect(output).to match(/FAILED:\s+0/)
+          end
+        end
+      end
+
+    end
+
+  end
+
+end

data/spec/system/deployment_as_gem_spec.rb

@@ -87,6 +87,8 @@ ceedling_system_tests do
 
     describe "Verbosity and output" do
       test_case :test_project_with_named_verbosity
+      test_case :test_project_with_warnings_verbosity_prints_overall_summary
+      test_case :test_project_with_errors_verbosity_prints_overall_summary_on_passing_run
       test_case :test_project_with_numerical_verbosity
       test_case :report_tests_raw_output_log_plugin
     end

data/spec/system/deployment_as_vendor_spec.rb

@@ -90,6 +90,8 @@ ceedling_system_tests do
 
     describe "Verbosity and output" do
       test_case :test_project_with_named_verbosity
+      test_case :test_project_with_warnings_verbosity_prints_overall_summary
+      test_case :test_project_with_errors_verbosity_prints_overall_summary_on_passing_run
       test_case :test_project_with_numerical_verbosity
       test_case :report_tests_raw_output_log_plugin
     end

data/spec/system/encoding_stress_spec.rb

@@ -131,8 +131,8 @@ ceedling_system_tests do
           @c.merge_project_yml_for_test({ :test_build => { :preprocess_force_fallback => false } })
           output = @c.ceedling_build_exec("test:all")
           expect(@c.last_exit_status).to eq(0)
-          expect(output).to match(/TESTED:\s+67/)
-          expect(output).to match(/PASSED:\s+67/)
+          expect(output).to match(/TESTED:\s+68/)
+          expect(output).to match(/PASSED:\s+68/)
         end
       end
     end
@@ -144,8 +144,8 @@ ceedling_system_tests do
           @c.merge_project_yml_for_test({ :test_build => { :preprocess_force_fallback => true } })
           output = @c.ceedling_build_exec("test:all")
           expect(@c.last_exit_status).to eq(0)
-          expect(output).to match(/TESTED:\s+67/)
-          expect(output).to match(/PASSED:\s+67/)
+          expect(output).to match(/TESTED:\s+68/)
+          expect(output).to match(/PASSED:\s+68/)
         end
       end
     end

data/spec/system/example_wondrous_forest_spec.rb

@@ -46,8 +46,8 @@ ceedling_system_tests do
         @c.with_context do
           Dir.chdir "wondrous_forest" do
             @output = @c.ceedling_build_exec("test:all")
-            expect(@output).to match(/TESTED:\s+67/)
-            expect(@output).to match(/PASSED:\s+67/)
+            expect(@output).to match(/TESTED:\s+68/)
+            expect(@output).to match(/PASSED:\s+68/)
             expect(@output).to match(/FAILED:\s+0/)
           end
         end

data/spec/system/gcov_deployment_spec.rb

@@ -33,8 +33,10 @@ ceedling_system_tests do
         end
       end
 
+      test_case :project_with_gcov_enabled_test_all_does_not_require_gcovr
       test_case :project_with_gcov_success
       test_case :project_with_gcov_fail
+      test_case :project_with_gcov_fail_under_decision
       test_case :gcov_console_report_with_system_header
       test_case :gcov_console_report_with_partial
       # TODO: Restore these tests when the :abort_on_uncovered option is restored in the Gcov plugin

data/spec/system/support/common_test_cases.rb

@@ -114,7 +114,11 @@ module CommonSystemTestCases
     @c.with_context do
       Dir.chdir @proj_name do
         all_docs = Dir["docs/*"]
-        expect(all_docs).to contain_exactly('docs/ceedling', 'docs/unity', 'docs/cmock', 'docs/c_exception', 'docs/license.txt')
+        # docs/ceedling only exists when this repo's own local HTML docs bundle
+        # (site-local/) was available to copy in -- see html_docs_bundle_available?.
+        expected = ['docs/unity', 'docs/cmock', 'docs/c_exception', 'docs/license.txt']
+        expected << 'docs/ceedling' if html_docs_bundle_available?
+        expect(all_docs).to contain_exactly(*expected)
       end
     end
   end
@@ -197,6 +201,48 @@ module CommonSystemTestCases
     end
   end
 
+  # The Overall Test Summary must still print at :warnings verbosity
+  # (`--verbosity=warnings`, Verbosity::COMPLAIN) on an all-passing run, not just
+  # when verbosity is :normal or higher, or when a test fails.
+  def test_project_with_warnings_verbosity_prints_overall_summary
+    @c.with_context do
+      Dir.chdir @proj_name do
+        FileUtils.cp test_asset_path("example_file.h"), 'src/'
+        FileUtils.cp test_asset_path("example_file.c"), 'src/'
+        FileUtils.cp test_asset_path("test_example_file_success.c"), 'test/'
+
+        output = @c.ceedling_build_exec("--verbosity=warnings")
+        expect(@c.last_exit_status).to eq(0)
+        expect(output).to match(/OVERALL TEST SUMMARY/)
+        expect(output).to match(/TESTED:\s+\d/)
+        expect(output).to match(/PASSED:\s+\d/)
+        expect(output).to match(/FAILED:\s+\d/)
+        expect(output).to match(/IGNORED:\s+\d/)
+      end
+    end
+  end
+
+  # ERRORS is the true floor for the Overall Test Summary -- a build's own
+  # test results are core information, visible even at the most restrictive
+  # verbosity short of :silent, whether the run passed or failed.
+  def test_project_with_errors_verbosity_prints_overall_summary_on_passing_run
+    @c.with_context do
+      Dir.chdir @proj_name do
+        FileUtils.cp test_asset_path("example_file.h"), 'src/'
+        FileUtils.cp test_asset_path("example_file.c"), 'src/'
+        FileUtils.cp test_asset_path("test_example_file_success.c"), 'test/'
+
+        output = @c.ceedling_build_exec("--verbosity=errors")
+        expect(@c.last_exit_status).to eq(0)
+        expect(output).to match(/OVERALL TEST SUMMARY/)
+        expect(output).to match(/TESTED:\s+\d/)
+        expect(output).to match(/PASSED:\s+\d/)
+        expect(output).to match(/FAILED:\s+\d/)
+        expect(output).to match(/IGNORED:\s+\d/)
+      end
+    end
+  end
+
   def test_project_with_numerical_verbosity
     @c.with_context do
       Dir.chdir @proj_name do

data/spec/system/support/gcov_common_test_cases.rb

@@ -10,6 +10,38 @@ require_relative 'gcov_helpers'
 module GcovCommonTestCases
   include GcovHelpers
 
+  # #1252 -- Enabling the gcov plugin must not make gcovr a hard dependency of every
+  # build. Deliberately does NOT call prep_project_yml_for_coverage (which conditionally
+  # strips :utilities ➡️ gcovr when the host lacks a real gcovr) -- the point here is
+  # gcovr enabled in config but genuinely unreachable, reproducing #1252 regardless of
+  # what's actually installed on the machine running this spec.
+  def project_with_gcov_enabled_test_all_does_not_require_gcovr
+    @c.with_context do
+      Dir.chdir @proj_name do
+        @c.uncomment_project_yml_option_for_test("- gcov")
+        # Default :gcov ↳ :utilities already includes gcovr and :reports already
+        # includes HtmlBasic -- exactly the enabled-but-unused shape #1252 reported.
+        @c.merge_project_yml_for_test(
+          { :tools => { :gcov_gcovr_report => { :executable => 'nonexistent_gcovr_binary_for_1252_test' } } }
+        )
+        FileUtils.cp test_asset_path("example_file.h"), 'src/'
+        FileUtils.cp test_asset_path("example_file.c"), 'src/'
+        FileUtils.cp test_asset_path("test_example_file_success.c"), 'test/'
+
+        # An ordinary test:all run must succeed -- it never needs gcovr at all.
+        output = @c.ceedling_build_exec("test:all")
+        expect(@c.last_exit_status).to eq(0)
+        expect(output).to_not match(/nonexistent_gcovr_binary_for_1252_test/)
+
+        # A real gcov: build still needs gcovr and must still fail loudly when it's missing --
+        # the fix defers gcovr's tool validation, it doesn't remove it.
+        output = @c.ceedling_build_exec("gcov:all")
+        expect(@c.last_exit_status).to_not eq(0)
+        expect(output).to match(/nonexistent_gcovr_binary_for_1252_test/)
+      end
+    end
+  end
+
   def project_with_gcov_success
     @c.with_context do
       Dir.chdir @proj_name do
@@ -46,6 +78,26 @@ module GcovCommonTestCases
     end
   end
 
+  # gcovr's own --fail-under-decision is a no-op without --decisions also present --
+  # Ceedling supplies --decisions automatically whenever :fail_under_decision is
+  # configured, so a passing run reports success instead of gcovr's own opaque
+  # "need also option --decision" CLI error.
+  def project_with_gcov_fail_under_decision
+    @c.with_context do
+      Dir.chdir @proj_name do
+        prep_project_yml_for_coverage
+        @c.merge_project_yml_for_test( { :gcov => { :gcovr => { :fail_under_decision => 1 } } } )
+        FileUtils.cp test_asset_path("example_file.h"), 'src/'
+        FileUtils.cp test_asset_path("example_file.c"), 'src/'
+        FileUtils.cp test_asset_path("test_example_file_success.c"), 'test/'
+
+        output = @c.ceedling_build_exec("gcov:all")
+        expect(@c.last_exit_status).to eq(0)
+        expect(output).to_not match(/need also option --decision/)
+      end
+    end
+  end
+
   # TODO: Restore this test when the :abort_on_uncovered option is restored in the Gcov plugin
   # def project_with_gcov_fail_because_of_uncovered_files
   #   @c.with_context do

data/spec/system/test_runner_system_include_spec.rb

@@ -0,0 +1,90 @@
+# =========================================================================
+#   Ceedling - Test-Centered Build System for C
+#   ThrowTheSwitch.org
+#   Copyright (c) 2010-26 Mike Karlesky, Mark VanderVoord, & Greg Williams
+#   SPDX-License-Identifier: MIT
+# =========================================================================
+
+require 'spec_system_helper'
+
+##
+## Generated Runner Preserves System #include Directory/Brackets (issue #1236)
+## =============================================================================
+##
+## A test file directly #include-ing a system header with a directory component
+## (e.g. `#include <sys/stat.h>`) previously produced a generated runner containing
+## `#include "stat.h"` -- the directory component and the system-include `<>`
+## delimiters were both lost, since GeneratorTestRunner#generate rendered every
+## include (mocks aside) down to its bare basename with no `<>` for system headers.
+## The test file itself compiled fine; only the separately generated runner file
+## failed, with a "No such file or directory" error on the mangled include.
+##
+
+TEST_RUNNER_SYSTEM_INCLUDE_C = <<~C
+  #include <sys/stat.h>
+
+  #include "unity.h"
+
+  void setUp(void) {}
+  void tearDown(void) {}
+
+  void test_runner_preserves_system_include_path(void)
+  {
+    struct stat value;
+    TEST_ASSERT_EQUAL_UINT(sizeof(value), sizeof(struct stat));
+  }
+C
+
+ceedling_system_tests do
+
+  before :all do
+    @c = SystemContext.new
+    @c.deploy_gem
+  end
+
+  after :all do
+    @c.done!
+  end
+
+  before { @proj_name = unique_proj_name("runner_sys_inc") }
+
+  describe "Deployed as a gem" do
+    before do
+      @c.with_context do
+        @c.ceedling_appcmd_exec("new #{@proj_name}")
+      end
+    end
+
+    before do
+      @c.with_context do
+        Dir.chdir @proj_name do
+          File.write('test/test_runner_system_include.c', TEST_RUNNER_SYSTEM_INCLUDE_C)
+        end
+      end
+    end
+
+    it "preserves a system include's directory component and angle brackets in the generated runner" do
+      @c.with_context do
+        Dir.chdir @proj_name do
+          output = @c.ceedling_build_exec("test:all")
+
+          expect(@c.last_exit_status).to eq(0)
+          expect(output).to match(/TESTED:\s+1/)
+          expect(output).to match(/PASSED:\s+1/)
+          expect(output).to match(/FAILED:\s+0/)
+
+          # The generated runner is what actually failed to compile in the reported
+          # issue -- assert on its content directly rather than just overall build
+          # success, since only the runner (not the test file itself) was affected.
+          runner_path = Dir.glob('build/test/runners/*_runner.c').first
+          expect(runner_path).not_to be_nil
+
+          runner_contents = File.read(runner_path)
+          expect(runner_contents).to include('#include <sys/stat.h>')
+          expect(runner_contents).not_to include('#include "stat.h"')
+        end
+      end
+    end
+  end
+
+end

data/spec/units/generators/generator_test_results_backtrace_spec.rb

@@ -130,6 +130,7 @@ SIMPLE_ASSERT_CRASH_OUTPUT =
 SIMPLE_PASS_OUTPUT   = "test_lib.c:3:test_empty:PASS\n---------\n1 Tests 0 Failures 0 Ignored\nOK\n"
 SIMPLE_FAIL_OUTPUT   = "test_lib.c:8:test_bad:FAIL: Expected 1 Was 2\n---------\n1 Tests 1 Failures 0 Ignored\nFAIL\n"
 SIMPLE_IGNORE_OUTPUT = "test_lib.c:12:test_skip:IGNORE\n---------\n1 Tests 0 Failures 1 Ignored\nOK\n"
+SIMPLE_IGNORE_WITH_MESSAGE_OUTPUT = "test_lib.c:12:test_skip:IGNORE: Not yet implemented\n---------\n1 Tests 0 Failures 1 Ignored\nOK\n"
 
 
 describe GeneratorTestResultsBacktrace do
@@ -255,6 +256,31 @@ describe GeneratorTestResultsBacktrace do
       expect(expected_output_lines).to include('test_lib.c:9:test_asserting:FAIL: Expected 1 Was 2')
     end
 
+    # Ignored-crash false positive: a TEST_IGNORE_MESSAGE(...) test case carries a
+    # trailing message, same as FAIL. Before this fix, only FAIL tolerated one --
+    # a message-carrying IGNORE fell through to "unresolved" and was misreported
+    # as crash evidence (a second @file_wrapper.write, for a companion_case that
+    # never actually crashed).
+    it 'handles an IGNORE test case with a trailing message and does not write a log' do
+      allow(@tool_executor).to receive(:exec).and_return(
+        { output: GDB_NO_SIGNAL_OUTPUT, time: 0.1, exit_code: 1, stderr: '', status: @ok_status },
+        { output: "test_lib.c:9:test_asserting:IGNORE: Not yet implemented\n---------\n1 Tests 0 Failures 1 Ignored\nOK\n",
+          time: 0.1, exit_code: 0, stderr: '', status: @ok_status }
+      )
+
+      expect(@file_wrapper).to receive(:write).once.with(%r{test_crashes_elsewhere}, anything, anything)
+
+      expected_output_lines = []
+      allow(@generator_test_results).to receive(:regenerate_test_executable_stdout) do |**kwargs|
+        expected_output_lines = kwargs[:output]
+        'regenerated'
+      end
+
+      @backtrace.do_gdb( filename, executable, shell_result, [companion_case] + test_cases, context: :test )
+
+      expect(expected_output_lines).to include('test_lib.c:9:test_asserting:IGNORE: Not yet implemented')
+    end
+
     it "does not trust a matched PASS line when the retry's own real status contradicts it" do
       # Regression test for issue #1198: a diagnostic retry can print a clean Unity
       # result line while its own real process outcome says otherwise.
@@ -471,6 +497,31 @@ describe GeneratorTestResultsBacktrace do
       expect(expected_output_lines).to include('test_lib.c:12:test_skip:IGNORE')
     end
 
+    # Ignored-crash false positive: a TEST_IGNORE_MESSAGE(...) test case carries a
+    # trailing message, same as FAIL (handled just above). Before this fix, only FAIL
+    # tolerated one -- a message-carrying IGNORE fell through to the "no result line"
+    # branch and was misreported as a second crashed test case (test_skip, alongside
+    # the real crash attributed to test_crashes_elsewhere).
+    it 'handles an IGNORE test case with a trailing message' do
+      test_cases_ignore = [{ test: 'test_crashes_elsewhere', line_number: 20 }, { test: 'test_skip', line_number: 12 }]
+
+      allow(@tool_executor).to receive(:exec).and_return(
+        { output: SIMPLE_ASSERT_CRASH_OUTPUT, time: 0.1, exit_code: 134, stderr: '', status: @ok_status },
+        { output: SIMPLE_IGNORE_WITH_MESSAGE_OUTPUT, time: 0.02, exit_code: 0, stderr: '', status: @ok_status }
+      )
+
+      expected_output_lines = []
+      allow(@generator_test_results).to receive(:regenerate_test_executable_stdout) do |**kwargs|
+        expected_output_lines = kwargs[:output]
+        'regenerated'
+      end
+
+      @backtrace.do_simple( filename, executable, shell_result, test_cases_ignore, context: :test )
+
+      expect(expected_output_lines).to include('test_lib.c:12:test_skip:IGNORE: Not yet implemented')
+      expect(expected_output_lines).to_not include(a_string_matching(/test_skip:FAIL/))
+    end
+
     it "does not trust a matched PASS line when the retry's own real status contradicts it" do
       # Regression test for issue #1198: a diagnostic retry can print a clean Unity
       # result line while its own real process outcome says otherwise.

data/spec/units/generators/generator_test_runner_spec.rb

@@ -8,6 +8,7 @@
 require 'spec_helper'
 require 'ceedling/generators/generator_test_runner'
 require 'ceedling/parsing_parcels'
+require 'ceedling/includes/includes'
 
 describe GeneratorTestRunner do
   before(:each) do
@@ -141,4 +142,72 @@ describe GeneratorTestRunner do
       )
     end
   end
+
+  # ===========================================================================
+  describe '#generate' do
+  # ===========================================================================
+    # Regression coverage for https://github.com/ThrowTheSwitch/Ceedling/issues/1236:
+    # a system #include with a directory component (e.g. <sys/stat.h>) was rendered
+    # into the generated runner as a bare, unbracketed basename ("stat.h"), losing
+    # both the directory and the user/system distinction, breaking runner compilation.
+    #
+    # @unity_runner_generator is a real UnityTestRunnerGenerator, constructed directly
+    # inside #initialize rather than injected -- swapped out for a double here so these
+    # specs assert on exactly what GeneratorTestRunner itself hands off, independent of
+    # Unity's own generated file formatting.
+    def build_and_capture_includes(includes:, mocks: [])
+      runner = build_runner( test_file_contents: "void test_Thing(void) {}\n" )
+
+      captured = nil
+      fake_generator = double('unity_runner_generator')
+      allow(fake_generator).to receive(:generate) do |*args|
+        captured = args[4]
+      end
+      runner.instance_variable_set(:@unity_runner_generator, fake_generator)
+
+      runner.generate(
+        module_name: 'test_thing',
+        runner_filepath: '/build/test/runners/test_thing_runner.c',
+        mocks: mocks,
+        includes: includes
+      )
+
+      return captured
+    end
+
+    it 'renders a system include with a directory component in angle brackets, directory intact' do
+      captured = build_and_capture_includes( includes: [ SystemInclude.new('sys/stat.h') ] )
+
+      expect( captured ).to eq( ['<sys/stat.h>'] )
+    end
+
+    it 'prefers a reconciled system include\'s include_path over its resolved filepath' do
+      # Mirrors what Includes.reconcile actually produces: filepath is the real,
+      # resolved (possibly absolute) location; include_path is the original,
+      # as-written directive text that must be what's rendered.
+      reconciled = SystemInclude.new('/usr/include/sys/stat.h', include_path: 'sys/stat.h')
+
+      captured = build_and_capture_includes( includes: [ reconciled ] )
+
+      expect( captured ).to eq( ['<sys/stat.h>'] )
+    end
+
+    it 'renders a user include with a directory component down to its bare filename, unchanged from prior behavior' do
+      # Unlike a system header, a project's own directories are exposed to the
+      # compiler as a flat list of -I search paths -- rendering a user include's
+      # own directory component here breaks resolution rather than fixing anything,
+      # so this behavior is intentionally unchanged by the issue #1236 fix.
+      captured = build_and_capture_includes( includes: [ UserInclude.new('sub/header.h') ] )
+
+      expect( captured ).to eq( ['header.h'] )
+    end
+
+    it 'renders a mixed list of system and user includes each in their own correct form' do
+      captured = build_and_capture_includes(
+        includes: [ SystemInclude.new('sys/stat.h'), UserInclude.new('sub/header.h') ]
+      )
+
+      expect( captured ).to eq( ['<sys/stat.h>', 'header.h'] )
+    end
+  end
 end