ceedling 1.1.10 → 1.2.0

data/spec/system/deployment_as_gem_spec.rb

@@ -67,6 +67,7 @@ ceedling_system_tests do
 
     describe "Partials" do
       test_case :test_project_partial_all_module_mocks_declaration_only_function
+      test_case :test_project_partials_enabled_still_allows_plain_treat_inlines_mocking
     end
 
     describe "Defines and configuration" do
@@ -118,6 +119,12 @@ ceedling_system_tests do
       test_case :crash_gdb_sigsegv_with_parameterized_test
     end
 
+    describe "Backtrace with GDB (hobbled)" do
+      include_context "requires a hobbled gdb"
+
+      test_case :crash_gdb_hobbled_falls_back_to_simple
+    end
+
     describe "Test filtering" do
       test_case :run_single_test_with_full_name_filter
       test_case :run_single_test_with_partial_name_filter

data/spec/system/deployment_as_vendor_spec.rb

@@ -1,12 +1,35 @@
 # =========================================================================
 #   Ceedling - Test-Centered Build System for C
 #   ThrowTheSwitch.org
-#   Copyright (c) 2010-24 Mike Karlesky, Mark VanderVoord, & Greg Williams
+#   Copyright (c) 2010-26 Mike Karlesky, Mark VanderVoord, & Greg Williams
 #   SPDX-License-Identifier: MIT
 # =========================================================================
 
 require 'spec_system_helper'
 
+##
+## Vendored Deployment Smoke
+## ===========================
+##
+## `deployment_as_gem_spec.rb` runs the full CommonSystemTestCases feature
+## battery once, under the default (gem) install mode -- preprocessing,
+## Partials, defines, Unity features, crash handling, verbosity, and test
+## filtering are all install-mode-agnostic (the same build/runtime logic runs
+## identically whether Ceedling's own files are gem-installed or vendored
+## into the project), so repeating that whole battery here a second (or
+## third) time under each vendored scaffold variant would only be
+## re-verifying behavior that install mode can't affect.
+##
+## What genuinely differs by scaffold variant -- does `--local`/`--docs`/
+## `--gitsupport` produce the right files, and does a real build actually run
+## against the vendored copy (a broken vendored $LOAD_PATH is exactly the
+## kind of thing file-presence checks alone wouldn't catch) -- keeps its own
+## dedicated coverage below, once per variant. Not "a fresh ceedling gem
+## project" (spec_system_helper.rb) here -- that shared context always
+## scaffolds a plain `new`, but every describe block below needs its own
+## differently-flagged `new --local ...` variant instead.
+##
+
 ceedling_system_tests do
   include CommonSystemTestCases
 
@@ -16,127 +39,28 @@ ceedling_system_tests do
   end
 
   after :all do
-    # Ensure version commit file is cleaned up
-    FileUtils.rm_rf( 'GIT_COMMIT_SHA' )
     @c.done!
   end
 
   before { @proj_name = unique_proj_name("vendor") }
 
   describe "Deployed in a project's `vendor` directory" do
-    before do
-      # Ensure version commit file is cleaned up
-      FileUtils.rm_rf( 'GIT_COMMIT_SHA' )
-      @c.with_context do
-        @c.ceedling_appcmd_exec("new --local --docs #{@proj_name}")
-      end
-    end
-
-    describe "Version reporting" do
-      test_case :can_report_version_no_git_commit_sha
-      test_case :can_report_version_with_git_commit_sha
-    end
+    before { @c.with_context { @c.ceedling_appcmd_exec("new --local --docs #{@proj_name}") } }
 
     describe "Project creation" do
       test_case :can_create_projects
       test_case :contains_a_vendor_directory
+      test_case :contains_vendored_licenses
       test_case :contains_documentation
     end
 
-    describe "Help system" do
-      test_case :application_commands_help
-      test_case :project_build_tasks_plugins_help
-    end
-
     describe "Basic test execution" do
       test_case :test_project_success
-      test_case :test_project_with_test_all_alias
-      test_case :test_project_success_default_task
-      test_case :test_project_fail
-      test_case :test_project_fail_alias
-      test_case :test_project_fail_default
-      test_case :test_project_with_compile_error
-      test_case :project_with_test_file_directly_including_source_file
-    end
-
-    describe "Test builds with preprocessing" do
-      test_case :test_project_preprocessing_undefined_symbols
-      test_case :test_project_preprocessing_defined_symbols
-      test_case :test_project_with_preprocessing_for_mocks
-      test_case :test_project_with_preprocessing_for_missing_mock
-      test_case :test_project_with_preprocessing_all
-      test_case :test_project_preprocessing_same_dir_mock_pairing
-    end
-
-    describe "Partials" do
-      test_case :test_project_partial_all_module_mocks_declaration_only_function
-    end
-
-    describe "Defines and configuration" do
-      test_case :test_project_with_per_file_defines
-      test_case :test_project_with_test_and_vendor_defines
-      test_case :test_project_with_both_mock_and_real_header
-    end
-
-    describe "Unity features" do
-      test_case :test_project_unity_parameterized_test_cases
-      test_case :test_project_preprocessed_unity_parameterized_test_cases
-      test_case :test_project_with_unity_exec_time
-    end
-
-    describe "Edge cases parsing" do
-      test_case :space_between_hash_and_include_is_valid
-    end
-
-    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
-
-    describe "Crash handling" do
-      test_case :crash_none_reports_executable_crashed
-      test_case :crash_none_writes_fail_results_file
-      test_case :crash_simple_sigsegv_all_test_cases
-      test_case :crash_simple_sigabrt_assert_failure
-      test_case :crash_simple_sigsegv_with_parameterized_test
-      test_case :crash_simple_diagnostic_retry_disagrees_with_main_fixture
-    end
-
-    describe "Crash handling with real UBSan" do
-      include_context "requires gcc with UBSan"
-
-      test_case :crash_simple_ubsan_diagnostic_retry_disagrees_with_main_fixture
-    end
-
-    describe "Backtrace with GDB" do
-      include_context "requires gdb"
-
-      test_case :crash_gdb_sigsegv_all_test_cases
-      test_case :crash_gdb_sigsegv_targets_test_case_filter
-      test_case :crash_gdb_sigsegv_excludes_test_case_filter
-      test_case :crash_gdb_sigabrt_assert_failure
-      test_case :crash_gdb_sigsegv_with_parameterized_test
-    end
-
-    describe "Test filtering" do
-      test_case :run_single_test_with_full_name_filter
-      test_case :run_single_test_with_partial_name_filter
-      test_case :exclusion_filter_leaves_one_test_case
-      test_case :no_tests_run_when_filter_matches_nothing
-      test_case :no_tests_run_when_filter_and_exclusion_cancel
-      test_case :run_one_test_case_with_name_filter
     end
   end
 
   describe "Deployed in a project's `vendor` directory with Git support" do
-    before do
-      @c.with_context do
-        @c.ceedling_appcmd_exec("new --local --docs --gitsupport #{@proj_name}")
-      end
-    end
+    before { @c.with_context { @c.ceedling_appcmd_exec("new --local --docs --gitsupport #{@proj_name}") } }
 
     describe "Project creation" do
       test_case :can_create_projects
@@ -147,16 +71,11 @@ ceedling_system_tests do
 
     describe "Basic test execution" do
       test_case :test_project_success
-      test_case :project_with_test_file_directly_including_source_file
     end
   end
 
   describe "Deployed in a project's `vendor` directory without docs" do
-    before do
-      @c.with_context do
-        @c.ceedling_appcmd_exec("new --local #{@proj_name}")
-      end
-    end
+    before { @c.with_context { @c.ceedling_appcmd_exec("new --local #{@proj_name}") } }
 
     describe "Project creation" do
       test_case :can_create_projects
@@ -164,43 +83,8 @@ ceedling_system_tests do
       test_case :does_not_contain_documentation
     end
 
-    describe "Help system" do
-      test_case :application_commands_help
-      test_case :project_build_tasks_plugins_help
-    end
-
     describe "Basic test execution" do
       test_case :test_project_success
-      test_case :test_project_with_test_all_alias
-      test_case :test_project_success_default_task
-      test_case :test_project_fail
-      test_case :test_project_fail_alias
-      test_case :test_project_fail_default
-      test_case :test_project_with_compile_error
-      test_case :project_with_test_file_directly_including_source_file
-    end
-
-    describe "Test builds with preprocessing" do
-      test_case :test_project_preprocessing_undefined_symbols
-      test_case :test_project_preprocessing_defined_symbols
-      test_case :test_project_with_preprocessing_for_mocks
-      test_case :test_project_with_preprocessing_for_missing_mock
-      test_case :test_project_with_preprocessing_all
-      test_case :test_project_preprocessing_same_dir_mock_pairing
-    end
-
-    describe "Partials" do
-      test_case :test_project_partial_all_module_mocks_declaration_only_function
-    end
-
-    describe "Defines and configuration" do
-      test_case :test_project_with_per_file_defines
-      test_case :test_project_with_test_and_vendor_defines
-    end
-
-    describe "Unity features" do
-      test_case :test_project_unity_parameterized_test_cases
-      test_case :test_project_with_unity_exec_time
     end
   end
-end
+end

data/spec/system/dumpconfig_spec.rb

@@ -0,0 +1,137 @@
+# =========================================================================
+#   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'
+require 'yaml'
+
+##
+## `ceedling dumpconfig` coverage
+## ==============================
+##
+## Two concerns, both otherwise untested at this tier (mixin_ordering_spec.rb and
+## ruby_replacement_spec.rb each exercise `dumpconfig` only incidentally, for their
+## own unrelated concerns):
+##
+## 1. A default (--app enabled) dumpconfig run's `:extension` config entries are
+##    FilenameExtension objects (lib/ceedling/filename_extension.rb) -- without a
+##    custom YAML serialization hook, these dump as tagged `!ruby/object:...`
+##    values instead of clean, safe YAML any tool can read back.
+##
+## 2. `--stdout` writes the same YAML directly to standard output instead of a
+##    file, for a consuming tool that wants Ceedling's resolved configuration
+##    without a temp file -- every console banner/notice dumpconfig would
+##    otherwise print must be suppressed so the stream stays pure YAML.
+##
+## 3. `--no-app` itself: dumps project config after mixins but before any
+##    application manipulation (settings, defaults, plugins, validation) --
+##    every other dumpconfig-shaped spec in this suite (mixin_loading_spec.rb,
+##    mixin_ordering_spec.rb, ruby_replacement_spec.rb) uses `--no-app` as a
+##    means to an end (a clean lens onto some other concern's merged config),
+##    not as the thing under test; this is the one place `--no-app` itself is
+##    the assertion.
+##
+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("dumpconfig") }
+
+  describe "Deployed as a gem" do
+    before do
+      @c.with_context do
+        @c.ceedling_appcmd_exec("new #{@proj_name}")
+      end
+    end
+
+    it "dumps clean YAML to a file, with :extension entries as plain arrays, not tagged Ruby objects" do
+      dump_file = nil
+      @c.with_context do
+        Dir.chdir @proj_name do
+          dump_file = File.expand_path('dump_config.yml')
+          @c.ceedling_appcmd_exec("dumpconfig #{dump_file}")
+        end
+      end
+
+      raw = read_spec_file(dump_file)
+      expect(raw).not_to include('!ruby/object')
+
+      config = YAML.safe_load(raw, permitted_classes: [Symbol])
+      expect(config.dig(:extension, :source)).to be_a(Array)
+      expect(config.dig(:extension, :source)).not_to be_empty
+    end
+
+    it "writes the same clean YAML to standard output instead of a file with --stdout" do
+      output = nil
+      @c.with_context do
+        Dir.chdir @proj_name do
+          output = @c.ceedling_appcmd_exec("dumpconfig --stdout")
+        end
+      end
+
+      # A leaked banner/notice would either break the parse outright or show up
+      # as extraneous top-level content -- a clean parse is the direct proof
+      # nothing but YAML reached the stream.
+      config = YAML.safe_load(output.to_s, permitted_classes: [Symbol])
+      expect(config.dig(:extension, :source)).to be_a(Array)
+      expect(output).not_to include('!ruby/object')
+      expect(output).not_to include('Dumped project configuration')
+    end
+
+    it "extracts just the requested section to stdout, positional arguments treated as SECTIONS" do
+      output = nil
+      @c.with_context do
+        Dir.chdir @proj_name do
+          output = @c.ceedling_appcmd_exec("dumpconfig --stdout tools test_compiler")
+        end
+      end
+
+      config = YAML.safe_load(output.to_s, permitted_classes: [Symbol])
+      expect(config.keys).to eq([:test_compiler])
+    end
+
+    it "skips application manipulation and prints a notice with --no-app" do
+      output = nil
+      dump_file = nil
+      @c.with_context do
+        Dir.chdir @proj_name do
+          dump_file = File.expand_path('dump_no_app.yml')
+          output = @c.ceedling_appcmd_exec("dumpconfig --no-app #{dump_file}")
+        end
+      end
+
+      expect(output).to match(/Skipped loading Ceedling application/)
+
+      # :extension ↳ :source only exists once app-load processing
+      # (Configurator#populate_with_defaults) merges in the project defaults and
+      # wraps every file-type entry in a FilenameExtension -- the file's own raw
+      # :extension section (just :executable, set directly by `ceedling new`'s
+      # own template) survives untouched, but :source's absence is a second,
+      # independent signal that app manipulation was truly skipped, not just
+      # that the notice printed.
+      config = YAML.safe_load(read_spec_file(dump_file), permitted_classes: [Symbol])
+      expect(config.dig(:extension, :source)).to be_nil
+    end
+
+    it "fails with a clear error and non-zero exit status when FILEPATH is omitted without --stdout" do
+      @c.with_context do
+        Dir.chdir @proj_name do
+          @c.ceedling_appcmd_exec("dumpconfig")
+        end
+      end
+
+      expect(@c.last_exit_status).not_to eq(0)
+      expect(@c.raw_output).to match(/FILEPATH is a required parameter/)
+    end
+  end
+end

data/spec/system/encoding_stress_spec.rb

@@ -77,6 +77,9 @@ ceedling_system_tests do
       # proving the stress condition has teeth, not just that the encoding name matches.
       # This is the same failure mode (`invalid byte sequence`) that encoding-safety
       # guards elsewhere in Ceedling's source-scanning code exist to prevent.
+      # Deliberately a bare read, unlike every other read in the system specs, which go
+      # through read_spec_file's explicit UTF-8. The point here is that the naive read
+      # still raises, so this one must not be "fixed" to read explicitly.
       non_ascii_fixture = File.expand_path('../../../examples/temp_sensor/test/TestMain.c', __FILE__)
       expect {
         File.read(non_ascii_fixture).match(/test/)
@@ -131,8 +134,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+68/)
-          expect(output).to match(/PASSED:\s+68/)
+          expect(output).to match(/TESTED:\s+71/)
+          expect(output).to match(/PASSED:\s+71/)
         end
       end
     end
@@ -144,8 +147,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+68/)
-          expect(output).to match(/PASSED:\s+68/)
+          expect(output).to match(/TESTED:\s+71/)
+          expect(output).to match(/PASSED:\s+71/)
         end
       end
     end

data/spec/system/example_cipher_quest_spec.rb

@@ -7,6 +7,22 @@
 
 require 'spec_system_helper'
 
+##
+## cipher_quest Example Project
+## ===============================
+##
+## `files:*`/`paths:*` reporting, previously checked here too, now lives
+## solely in files_paths_reporting_spec.rb. The inline-YAML mixin duplicates
+## of each release build are dropped -- inline-YAML *syntax* is proven once by
+## mixin_ordering_spec.rb/mixin_loading_spec.rb, and repeating it per release
+## scenario here added nothing the sigil-file builds below don't already
+## prove. What's kept: `test:all`, the `#error`-without-mixin failure (a guard
+## specific to this project's own shipped main.c, no analog elsewhere), and
+## all three sigil-file release builds -- ROT13, Caesar, and full-featured are
+## functionally distinct feature-selection scenarios, not redundant with each
+## other.
+##
+
 ceedling_system_tests do
   include CommonSystemTestCases
 
@@ -21,24 +37,21 @@ ceedling_system_tests do
 
   before { @proj_name = "cipher_quest" }
 
-  describe "Command: `ceedling examples`" do
-    before do
-      @c.with_context do
-        @output = @c.ceedling_appcmd_exec("examples")
-      end
-    end
-
-    it "should list cipher_quest as an available example" do
-      expect(@output).to match(/cipher_quest/)
-    end
-  end
-
   describe "Command: `ceedling example cipher_quest`" do
     describe "cipher_quest" do
       before do
         @c.with_context do
           output = @c.ceedling_appcmd_exec("example cipher_quest")
           expect(output).to match(/created/)
+
+          # `ceedling example` only overwrites src/, test/, mixin/, and project.yml --
+          # it never touches build/. Without clobbering it here, a dependency cache
+          # populated by an earlier example in this describe block would carry over
+          # and make delta-build staleness tracking correctly (but unhelpfully, for
+          # these tests) treat this "fresh" example as already fully built.
+          Dir.chdir "cipher_quest" do
+            @c.ceedling_build_exec("clobber")
+          end
         end
       end
 
@@ -73,16 +86,6 @@ ceedling_system_tests do
         end
       end
 
-      it "should build a release binary with the ROT13 inline YAML mixin" do
-        @c.with_context do
-          Dir.chdir "cipher_quest" do
-            @output = @c.ceedling_build_exec("release --mixin \"=defines: {release: ['CIPHER_ROT13']}\"")
-            expect(@output).to match(/Merging command line inline YAML mixin/)
-            expect(@output).to match(/cipher_quest\.out/)
-          end
-        end
-      end
-
       it "should build a release binary with the Caesar cipher file mixin via sigil" do
         @c.with_context do
           Dir.chdir "cipher_quest" do
@@ -93,16 +96,6 @@ ceedling_system_tests do
         end
       end
 
-      it "should build a release binary with the Caesar cipher inline YAML mixin" do
-        @c.with_context do
-          Dir.chdir "cipher_quest" do
-            @output = @c.ceedling_build_exec("release --mixin \"=defines: {release: ['CIPHER_CAESAR']}\"")
-            expect(@output).to match(/Merging command line inline YAML mixin/)
-            expect(@output).to match(/cipher_quest\.out/)
-          end
-        end
-      end
-
       it "should build a release binary with the full-featured file mixin via sigil" do
         @c.with_context do
           Dir.chdir "cipher_quest" do
@@ -113,74 +106,6 @@ ceedling_system_tests do
         end
       end
 
-      it "should build a release binary with the full-featured inline YAML mixin" do
-        @c.with_context do
-          Dir.chdir "cipher_quest" do
-            @output = @c.ceedling_build_exec("release --mixin \"=defines: {release: ['CIPHER_ROT13', 'CIPHER_CAESAR', 'ANALYZER_ENABLED']}\"")
-            expect(@output).to match(/Merging command line inline YAML mixin/)
-            expect(@output).to match(/cipher_quest\.out/)
-          end
-        end
-      end
-
-      it "should be able to report header files found in paths" do
-        @c.with_context do
-          Dir.chdir "cipher_quest" do
-            @output = @c.ceedling_build_exec("files:header")
-            expect(@output).to match(/Header files:/i)
-            expect(@output).to match(/text_utils\.h/i)
-            expect(@output).to match(/cipher\.h/i)
-            expect(@output).to match(/analyzer\.h/i)
-          end
-        end
-      end
-
-      it "should be able to report source files found in paths" do
-        @c.with_context do
-          Dir.chdir "cipher_quest" do
-            @output = @c.ceedling_build_exec("files:source")
-            expect(@output).to match(/Source files:/i)
-            expect(@output).to match(/text_utils\.c/i)
-            expect(@output).to match(/cipher\.c/i)
-            expect(@output).to match(/analyzer\.c/i)
-            expect(@output).to match(/main\.c/i)
-          end
-        end
-      end
-
-      it "should be able to report test files found in paths" do
-        @c.with_context do
-          Dir.chdir "cipher_quest" do
-            @output = @c.ceedling_build_exec("files:test")
-            expect(@output).to match(/Test files:/i)
-            expect(@output).to match(/TestTextUtils\.c/i)
-            expect(@output).to match(/TestCipherRot13\.c/i)
-            expect(@output).to match(/TestCipherCaesar\.c/i)
-            expect(@output).to match(/TestAnalyzer\.c/i)
-          end
-        end
-      end
-
-      it "should be able to report include paths" do
-        @c.with_context do
-          Dir.chdir "cipher_quest" do
-            @output = @c.ceedling_build_exec("paths:include")
-            expect(@output).to match(/Include paths:/i)
-            expect(@output).to match(/src/i)
-          end
-        end
-      end
-
-      it "should be able to report test paths" do
-        @c.with_context do
-          Dir.chdir "cipher_quest" do
-            @output = @c.ceedling_build_exec("paths:test")
-            expect(@output).to match(/Test paths:/i)
-            expect(@output).to match(/test/i)
-          end
-        end
-      end
-
     end
   end
 end