ceedling 1.2.1 → 1.2.2

data/spec/system/support/common_test_cases.rb

@@ -6,7 +6,7 @@
 # =========================================================================
 
 require 'rbconfig'
-require 'ceedling/file_wrapper'
+require 'ceedling/filesystem/file_wrapper'
 require 'ceedling/yaml_wrapper'
 
 module CommonSystemTestCases
@@ -188,8 +188,8 @@ module CommonSystemTestCases
 
         # Windows' Installer Detection reads a manifest beside each test executable
         if windows
-          manifests = Dir.glob('build/test/out/test_example_file_success/*.manifest')
-          expect(manifests).to eq(['build/test/out/test_example_file_success/test_example_file_success.exe.manifest'])
+          manifests = Dir.glob('build/test/out/test/test_example_file_success/*.manifest')
+          expect(manifests).to eq(['build/test/out/test/test_example_file_success/test_example_file_success.exe.manifest'])
         end
       end
     end
@@ -727,7 +727,7 @@ module CommonSystemTestCases
         expect(output).to match(/PASSED:\s+\d/)
         expect(output).to match(/FAILED:\s+\d/)
         expect(output).to match(/IGNORED:\s+\d/)
-        expect(File.exist?("build/artifacts/test/test_example_file_verbose.raw.log")).to eq true
+        expect(File.exist?("build/artifacts/test/test/test_example_file_verbose.raw.log")).to eq true
       end
     end
   end
@@ -880,7 +880,6 @@ module CommonSystemTestCases
         expect(@c.last_exit_status).to eq(1) # Test should fail because of crash
         expect(output).to match(/Test Executable Crashed/i)
         expect(output).to match(/Unit test failures/)
-        expect(!File.exist?('./build/test/results/test_add.fail'))
       end
     end
   end
@@ -901,9 +900,12 @@ module CommonSystemTestCases
         expect(@c.last_exit_status).to eq(1) # Test should fail because of crash
         expect(output).to match(/Test Executable Crashed/i)
         expect(output).to match(/Unit test failures/)
-        expect(File.exist?('./build/test/results/test_nosanitize_crash_sigsegv.fail')).to be(true)
-        output_rd = read_spec_file('./build/test/results/test_nosanitize_crash_sigsegv.fail')
-        expect(output_rd =~ /test_add_numbers_will_fail \(\) at test\/test_nosanitize_crash_sigsegv.c\:\d+/ )
+        expect(File.exist?('./build/test/results/test/test_nosanitize_crash_sigsegv.fail')).to be(true)
+        # Without backtraces the whole executable crashed, so every case fails as a crash at its own line
+        results = yaml_wrapper_for_test.load('./build/test/results/test/test_nosanitize_crash_sigsegv.fail')
+        crashed = results[:failures].find { |failure| failure[:test] == 'test_add_numbers_will_fail' }
+        expect(crashed).to include(message: 'Test executable crashed')
+        expect(crashed[:line]).to be > 0
       end
     end
   end
@@ -924,15 +926,15 @@ module CommonSystemTestCases
         expect(@c.last_exit_status).to eq(1) # Test should fail because of crash
         expect(output).to match(/Test Case Crashed/i)
         expect(output).to match(/Unit test failures/)
-        expect(File.exist?('./build/test/results/test_nosanitize_crash_sigsegv.fail')).to be(true)
-        output_rd = read_spec_file('./build/test/results/test_nosanitize_crash_sigsegv.fail')
+        expect(File.exist?('./build/test/results/test/test_nosanitize_crash_sigsegv.fail')).to be(true)
+        output_rd = read_spec_file('./build/test/results/test/test_nosanitize_crash_sigsegv.fail')
         expect(output_rd).to match(/Test case crashed/)
         expect(output_rd).to match(/SIGSEGV/i)
         expect(output).to match(/TESTED:\s+2/)
         expect(output).to match(/PASSED:\s+(?:0|1)/)
         expect(output).to match(/FAILED:\s+(?:1|2)/)
         expect(output).to match(/IGNORED:\s+0/)
-        log_path = './build/logs/test/test_nosanitize_crash_sigsegv/test_add_numbers_will_fail.gdb.log'
+        log_path = './build/logs/test/test/test_nosanitize_crash_sigsegv/test_add_numbers_will_fail.gdb.log'
         expect(File.exist?(log_path)).to be(true)
         expect(read_spec_file(log_path)).to match(/SIGSEGV|Segmentation fault/i)
         expect(output).to match(/test_add_numbers_will_fail\.gdb\.log/)
@@ -955,15 +957,15 @@ module CommonSystemTestCases
         expect(@c.last_exit_status).to eq(1) # Test should fail because of crash
         expect(output).to match(/Test Case Crashed/i)
         expect(output).to match(/Unit test failures/)
-        expect(File.exist?('./build/test/results/test_nosanitize_crash_sigsegv.fail')).to be(true)
-        output_rd = read_spec_file('./build/test/results/test_nosanitize_crash_sigsegv.fail')
+        expect(File.exist?('./build/test/results/test/test_nosanitize_crash_sigsegv.fail')).to be(true)
+        output_rd = read_spec_file('./build/test/results/test/test_nosanitize_crash_sigsegv.fail')
         expect(output_rd).to match(/Test case crashed/)
         expect(output_rd).to match(/SIGSEGV/i)
         expect(output).to match(/TESTED:\s+1/)
         expect(output).to match(/PASSED:\s+(?:0|1)/)
         expect(output).to match(/FAILED:\s+(?:1|2)/)
         expect(output).to match(/IGNORED:\s+0/)
-        log_path = './build/logs/test/test_nosanitize_crash_sigsegv/test_add_numbers_will_fail.gdb.log'
+        log_path = './build/logs/test/test/test_nosanitize_crash_sigsegv/test_add_numbers_will_fail.gdb.log'
         expect(File.exist?(log_path)).to be(true)
         expect(read_spec_file(log_path)).to match(/SIGSEGV|Segmentation fault/i)
         expect(output).to match(/test_add_numbers_will_fail\.gdb\.log/)
@@ -986,15 +988,15 @@ module CommonSystemTestCases
         expect(@c.last_exit_status).to eq(1) # Test should fail because of crash
         expect(output).to match(/Test Case Crashed/i)
         expect(output).to match(/Unit test failures/)
-        expect(File.exist?('./build/test/results/test_nosanitize_crash_sigsegv.fail')).to be(true)
-        output_rd = read_spec_file('./build/test/results/test_nosanitize_crash_sigsegv.fail')
+        expect(File.exist?('./build/test/results/test/test_nosanitize_crash_sigsegv.fail')).to be(true)
+        output_rd = read_spec_file('./build/test/results/test/test_nosanitize_crash_sigsegv.fail')
         expect(output_rd).to match(/Test case crashed/)
         expect(output_rd).to match(/SIGSEGV/i)
         expect(output).to match(/TESTED:\s+1/)
         expect(output).to match(/PASSED:\s+(?:0|1)/)
         expect(output).to match(/FAILED:\s+(?:1|2)/)
         expect(output).to match(/IGNORED:\s+0/)
-        log_path = './build/logs/test/test_nosanitize_crash_sigsegv/test_add_numbers_will_fail.gdb.log'
+        log_path = './build/logs/test/test/test_nosanitize_crash_sigsegv/test_add_numbers_will_fail.gdb.log'
         expect(File.exist?(log_path)).to be(true)
         expect(read_spec_file(log_path)).to match(/SIGSEGV|Segmentation fault/i)
         expect(output).to match(/test_add_numbers_will_fail\.gdb\.log/)
@@ -1021,7 +1023,7 @@ module CommonSystemTestCases
         expect(output).to match(/Unit test failures/)
         expect(output).to match(/SIGABRT/i)
         expect(output).to match(/Assertion/i)
-        log_path = './build/logs/test/test_nosanitize_crash_assert/test_add_numbers_triggers_assert.gdb.log'
+        log_path = './build/logs/test/test/test_nosanitize_crash_assert/test_add_numbers_triggers_assert.gdb.log'
         expect(File.exist?(log_path)).to be(true)
         # Windwos gdb output for an assertion failure can be quite different from Linux.
         # Windows gdb reports do not seem to cite the signal "SIGABRT" and may be quite brief,
@@ -1116,7 +1118,7 @@ module CommonSystemTestCases
         expect(output).to match(/FAILED:\s+1/)
         expect(output).to match(/IGNORED:\s+0/)
 
-        result_file = './build/test/results/test_nosanitize_crash_sigsegv_with_param.fail'
+        result_file = './build/test/results/test/test_nosanitize_crash_sigsegv_with_param.fail'
         expect(File.exist?(result_file)).to be(true)
         results = yaml_wrapper_for_test.load(result_file)
 
@@ -1158,7 +1160,7 @@ module CommonSystemTestCases
         expect(output).to match(/FAILED:\s+1/)
         expect(output).to match(/IGNORED:\s+0/)
 
-        result_file = './build/test/results/test_nosanitize_crash_sigsegv_with_param.fail'
+        result_file = './build/test/results/test/test_nosanitize_crash_sigsegv_with_param.fail'
         expect(File.exist?(result_file)).to be(true)
         results = yaml_wrapper_for_test.load(result_file)
 
@@ -1174,7 +1176,7 @@ module CommonSystemTestCases
           'test_difference_between_numbers_is_correct(0, 0, 0)'
         )
 
-        log_path = './build/logs/test/test_nosanitize_crash_sigsegv_with_param/test_add_numbers_will_fail.gdb.log'
+        log_path = './build/logs/test/test/test_nosanitize_crash_sigsegv_with_param/test_add_numbers_will_fail.gdb.log'
         expect(File.exist?(log_path)).to be(true)
         expect(read_spec_file(log_path)).to match(/SIGSEGV|Segmentation fault/i)
       end

data/spec/system/support/gcov_common_test_cases.rb

@@ -448,9 +448,12 @@ module GcovCommonTestCases
         expect(@c.last_exit_status).to eq(1) # Ceedling should exit with error because of failed test due to crash
         expect(output).to match(/crashed/i)
         expect(output).to match(/Unit test failures/)
-        expect(File.exist?('./build/gcov/results/test_nosanitize_crash_sigsegv.fail'))
-        output_rd = read_spec_file('./build/gcov/results/test_nosanitize_crash_sigsegv.fail')
-        expect(output_rd =~ /test_add_numbers_will_fail \(\) at test\/test_nosanitize_crash_sigsegv.c\:\d+/ )
+        expect(File.exist?('./build/gcov/results/test/test_nosanitize_crash_sigsegv.fail')).to be(true)
+        # gdb names the crashing case, its signal, and the log of its backtrace
+        results = yaml_wrapper_for_test.load('./build/gcov/results/test/test_nosanitize_crash_sigsegv.fail')
+        expect(results[:failures].map { |failure| failure[:test] }).to eq(['test_add_numbers_will_fail'])
+        expect(results[:failures].first[:message]).to match(/Test case crashed >> \[SIGSEGV\] Segmentation fault/)
+        expect(results[:failures].first[:message]).to include('build/logs/gcov/test/test_nosanitize_crash_sigsegv/test_add_numbers_will_fail.gdb.log')
         expect(output).to match(/TESTED:\s+2/)
         expect(output).to match(/PASSED:\s+(?:0|1)/)
         expect(output).to match(/FAILED:\s+(?:1|2)/)
@@ -482,9 +485,12 @@ module GcovCommonTestCases
         expect(@c.last_exit_status).to eq(1) # Ceedling should exit with error because of failed test due to crash
         expect(output).to match(/Test Case Crashed/i)
         expect(output).to match(/Unit test failures/)
-        expect(File.exist?('./build/gcov/results/test_nosanitize_crash_sigsegv.fail'))
-        output_rd = read_spec_file('./build/gcov/results/test_nosanitize_crash_sigsegv.fail')
-        expect(output_rd =~ /test_add_numbers_will_fail \(\) at test\/test_nosanitize_crash_sigsegv.c\:\d+/ )
+        expect(File.exist?('./build/gcov/results/test/test_nosanitize_crash_sigsegv.fail')).to be(true)
+        # gdb names the crashing case, its signal, and the log of its backtrace
+        results = yaml_wrapper_for_test.load('./build/gcov/results/test/test_nosanitize_crash_sigsegv.fail')
+        expect(results[:failures].map { |failure| failure[:test] }).to eq(['test_add_numbers_will_fail'])
+        expect(results[:failures].first[:message]).to match(/Test case crashed >> \[SIGSEGV\] Segmentation fault/)
+        expect(results[:failures].first[:message]).to include('build/logs/gcov/test/test_nosanitize_crash_sigsegv/test_add_numbers_will_fail.gdb.log')
         expect(output).to match(/TESTED:\s+1/)
         expect(output).to match(/PASSED:\s+0/)
         expect(output).to match(/FAILED:\s+1/)
@@ -522,7 +528,7 @@ module GcovCommonTestCases
 
         output = @c.ceedling_build_exec("gcov:all --exclude_test_case=test_add_numbers_will_fail")
         expect(@c.last_exit_status).to eq(0)
-        expect(File.exist?('./build/gcov/results/test_nosanitize_crash_sigsegv.pass'))
+        expect(File.exist?('./build/gcov/results/test/test_nosanitize_crash_sigsegv.pass')).to be(true)
         expect(output).to match(/TESTED:\s+2/)
         expect(output).to match(/PASSED:\s+2/)
         expect(output).to match(/FAILED:\s+0/)
@@ -624,7 +630,7 @@ module GcovCommonTestCases
 
         output = @c.ceedling_build_exec("gcov:untested_sources")
         expect(@c.last_exit_status).to eq(0), output
-        expect(File.exist?('build/gcov/out/needs_define.o')).to eq true
+        expect(File.exist?('build/gcov/out/src/needs_define.o')).to eq true
       end
     end
   end
@@ -645,7 +651,29 @@ module GcovCommonTestCases
         output = @c.ceedling_build_exec("gcov:untested_sources")
         expect(@c.last_exit_status).to eq(0)
         expect(output).not_to match(/TESTED:\s+\d/) # No test build/run occurred
-        expect(File.exist?('build/gcov/out/uncovered_example_file.o')).to eq true
+        expect(File.exist?('build/gcov/out/src/uncovered_example_file.o')).to eq true
+      end
+    end
+  end
+
+  # Untested sources compile outside any test, so only their paths keep same-named ones apart
+  def project_with_gcov_untested_sources_sharing_a_name
+    @c.with_context do
+      Dir.chdir @proj_name do
+        prep_project_yml_for_coverage
+        add_gcov_option("untested_sources", ":compile")
+        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/'
+        FileUtils.mkdir_p( ['src/a', 'src/b'] )
+        File.write( 'src/a/util.c', "int util_a(void) { return 1; }\n" )
+        File.write( 'src/b/util.c', "int util_b(void) { return 2; }\n" )
+
+        output = @c.ceedling_build_exec("gcov:untested_sources")
+        expect(@c.last_exit_status).to eq(0), output
+        expect(File.exist?('build/gcov/out/src/a/util.o')).to eq true
+        expect(File.exist?('build/gcov/out/src/b/util.o')).to eq true
+        expect(Dir.glob('build/gcov/dependencies/**/util.d').sort).to eq(['build/gcov/dependencies/src/a/util.d', 'build/gcov/dependencies/src/b/util.d'])
       end
     end
   end

data/spec/system/support/sanitize_common_test_cases.rb

@@ -33,7 +33,7 @@ module SanitizeCommonTestCases
         expect(@c.last_exit_status).to eq(0)
         expect(output).to match(/SANITIZE: .*OVERALL TEST SUMMARY/)
         expect(output).not_to match(/Sanitizers found/)
-        expect(Dir.glob('build/artifacts/sanitize/sanitize/*.log')).to be_empty
+        expect(Dir.glob('build/artifacts/sanitize/sanitize/**/*.log')).to be_empty
       end
     end
   end
@@ -68,7 +68,7 @@ module SanitizeCommonTestCases
         expect(@c.last_exit_status).to eq(1)
         expect(output).to match(/test_writes_past_its_buffer.*\n.*AddressSanitizer heap-buffer-overflow at \S*test_nosanitize_heap_buffer_overflow\.c:\d+/)
         expect(output).to match(/Sanitizers found 1 problem in 1 test/)
-        expect(read_spec_file('build/artifacts/sanitize/sanitize/test_nosanitize_heap_buffer_overflow.log')).to match(/ERROR: AddressSanitizer: heap-buffer-overflow/)
+        expect(read_spec_file('build/artifacts/sanitize/sanitize/test/test_nosanitize_heap_buffer_overflow.log')).to match(/ERROR: AddressSanitizer: heap-buffer-overflow/)
       end
     end
   end

data/spec/system/support/valgrind_common_test_cases.rb

@@ -33,8 +33,8 @@ module ValgrindCommonTestCases
         output = @c.ceedling_build_exec("valgrind:all")
         expect(@c.last_exit_status).to eq(0)
         expect(output).to match(/Wrote 1 Valgrind report/i)
-        expect(File.exist?('build/artifacts/valgrind/test_example_file_success.log')).to eq(true)
-        expect(File.size('build/artifacts/valgrind/test_example_file_success.log')).to be > 0
+        expect(File.exist?('build/artifacts/valgrind/test/test_example_file_success.log')).to eq(true)
+        expect(File.size('build/artifacts/valgrind/test/test_example_file_success.log')).to be > 0
       end
     end
   end
@@ -70,8 +70,8 @@ module ValgrindCommonTestCases
         output = @c.ceedling_build_exec("valgrind:test_example_file_success.c")
         expect(@c.last_exit_status).to eq(0)
         expect(output).to match(/Wrote 1 Valgrind report/i)
-        expect(File.exist?('build/artifacts/valgrind/test_example_file_success.log')).to eq(true)
-        expect(File.size('build/artifacts/valgrind/test_example_file_success.log')).to be > 0
+        expect(File.exist?('build/artifacts/valgrind/test/test_example_file_success.log')).to eq(true)
+        expect(File.size('build/artifacts/valgrind/test/test_example_file_success.log')).to be > 0
       end
     end
   end
@@ -97,10 +97,10 @@ module ValgrindCommonTestCases
         # Aggregate build failure registered by Valgrind plugin
         expect(output).to match(/Valgrind detected.*memory error.*across/i)
         expect(output).to match(/Wrote 1 Valgrind report/i)
-        expect(File.exist?('build/artifacts/valgrind/test_nosanitize_crash_sigsegv.log')).to eq(true)
-        expect(File.size('build/artifacts/valgrind/test_nosanitize_crash_sigsegv.log')).to be > 0
+        expect(File.exist?('build/artifacts/valgrind/test/test_nosanitize_crash_sigsegv.log')).to eq(true)
+        expect(File.size('build/artifacts/valgrind/test/test_nosanitize_crash_sigsegv.log')).to be > 0
         # Expected memory error(s) detected by Valgrind
-        expect(read_spec_file('build/artifacts/valgrind/test_nosanitize_crash_sigsegv.log')).to match(/ERROR SUMMARY:\s+[1-9]/)
+        expect(read_spec_file('build/artifacts/valgrind/test/test_nosanitize_crash_sigsegv.log')).to match(/ERROR SUMMARY:\s+[1-9]/)
       end
     end
   end
@@ -127,10 +127,10 @@ module ValgrindCommonTestCases
         # No aggregate Valgrind build failure registered
         expect(output).not_to match(/Valgrind detected.*memory error.*across/i)
         expect(output).to match(/Wrote 1 Valgrind report/i)
-        expect(File.exist?('build/artifacts/valgrind/test_nosanitize_crash_sigsegv.log')).to eq(true)
-        expect(File.size('build/artifacts/valgrind/test_nosanitize_crash_sigsegv.log')).to be > 0
+        expect(File.exist?('build/artifacts/valgrind/test/test_nosanitize_crash_sigsegv.log')).to eq(true)
+        expect(File.size('build/artifacts/valgrind/test/test_nosanitize_crash_sigsegv.log')).to be > 0
         # Expected memory error(s) detected by Valgrind
-        expect(read_spec_file('build/artifacts/valgrind/test_nosanitize_crash_sigsegv.log')).to match(/ERROR SUMMARY:\s+[1-9]/)
+        expect(read_spec_file('build/artifacts/valgrind/test/test_nosanitize_crash_sigsegv.log')).to match(/ERROR SUMMARY:\s+[1-9]/)
       end
     end
   end
@@ -149,7 +149,7 @@ module ValgrindCommonTestCases
 
         expect(@c.last_exit_status).to eq(0)
 
-        xml_path = 'build/artifacts/valgrind/test_example_file_success.xml'
+        xml_path = 'build/artifacts/valgrind/test/test_example_file_success.xml'
         expect(File.exist?(xml_path)).to eq(true)
         expect(read_spec_file(xml_path)).to match(/<valgrindoutput>/)
       end
@@ -175,7 +175,7 @@ module ValgrindCommonTestCases
         @c.ceedling_build_exec("valgrind:all")
 
         expect(@c.last_exit_status).to eq(0)
-        expect(File.exist?('build/artifacts/valgrind/test_example_file_success.log')).to eq(true)
+        expect(File.exist?('build/artifacts/valgrind/test/test_example_file_success.log')).to eq(true)
       end
     end
   end

data/spec/system/test_pipeline_stop_points_spec.rb

@@ -41,7 +41,7 @@ ceedling_system_tests do
         Dir.chdir "temp_sensor" do
           output = @c.ceedling_build_exec("gen:mocks")
 
-          expect(output).to match(/Generating mock for adc\/TestAdcModel::TaskScheduler\.h/)
+          expect(output).to match(/Generating mock for test\/adc\/TestAdcModel::TaskScheduler\.h/)
           expect(output).to_not match(/Generating runner for/)
           expect(output).to_not match(/^Compiling /)
           expect(output).to_not match(/^Linking /)
@@ -62,7 +62,7 @@ ceedling_system_tests do
         Dir.chdir "temp_sensor" do
           output = @c.ceedling_build_exec("gen:test_runner")
 
-          expect(output).to match(/Generating mock for adc\/TestAdcModel::TaskScheduler\.h/)
+          expect(output).to match(/Generating mock for test\/adc\/TestAdcModel::TaskScheduler\.h/)
           expect(output).to match(/Generating runner for TestAdcModel\.c/)
           expect(output).to_not match(/^Compiling /)
           expect(output).to_not match(/^Linking /)
@@ -80,7 +80,7 @@ ceedling_system_tests do
         Dir.chdir "temp_sensor" do
           output = @c.ceedling_build_exec("gen:mocks:TestAdcModel.c")
 
-          expect(output).to match(/Generating mock for adc\/TestAdcModel::TaskScheduler\.h/)
+          expect(output).to match(/Generating mock for test\/adc\/TestAdcModel::TaskScheduler\.h/)
           # TestAdcModel.c mocks TaskScheduler, TemperatureCalculator, and TemperatureFilter --
           # exactly its own 3 mocks, none belonging to any other test.
           expect(output.to_s.scan(/Generating mock for /).length).to eq(3)
@@ -101,7 +101,7 @@ ceedling_system_tests do
         Dir.chdir "temp_sensor" do
           output = @c.ceedling_build_exec("gen:test_runner:TestAdcModel.c")
 
-          expect(output).to match(/Generating mock for adc\/TestAdcModel::TaskScheduler\.h/)
+          expect(output).to match(/Generating mock for test\/adc\/TestAdcModel::TaskScheduler\.h/)
           # TestAdcModel.c mocks TaskScheduler, TemperatureCalculator, and TemperatureFilter --
           # exactly its own 3 mocks, none belonging to any other test.
           expect(output.to_s.scan(/Generating mock for /).length).to eq(3)

data/spec/system/test_runner_system_include_spec.rb

@@ -55,7 +55,7 @@ ceedling_system_tests do
         # 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
+        runner_path = Dir.glob('build/test/runners/**/*_runner.c').first
         expect(runner_path).not_to be_nil
 
         runner_contents = read_spec_file(runner_path)

data/spec/system/test_source_file_directive_spec.rb

@@ -121,6 +121,35 @@ ceedling_system_tests do
 
     end
 
+    # =========================================================================
+    describe "A project where one of several tests misspells a TEST_SOURCE_FILE() entry" do
+    # =========================================================================
+
+      # A misspelled entry stops the build naming its own test, with test preprocessing on or off
+      before do
+        copy_duplicate_calc_sources
+        in_project do
+          copy_fixture("test_source_file_duplicate_basenames/test_calc_alpha.c", 'test')
+          copy_fixture("test_source_file_duplicate_basenames/test_calc_beta.c", 'test')
+          copy_fixture("test_source_file_duplicate_basenames/test_calc_typo.c", 'test')
+        end
+      end
+
+      [:none, :tests].each do |preprocessing|
+        it "names the test with the misspelled entry, with test preprocessing #{preprocessing}" do
+          in_project do
+            @c.merge_project_yml_for_test({ :project => { :use_test_preprocessor => preprocessing } })
+
+            output = @c.ceedling_build_exec("test:all")
+            expect(@c.last_exit_status).not_to eq(0)
+            expect(output).to match(%r{'alpha/clac\.c' specified with TEST_SOURCE_FILE\(\) in test/test_calc_typo cannot be found})
+            expect(output).not_to match(/Undefined symbols|undefined reference/)
+          end
+        end
+      end
+
+    end
+
     # =========================================================================
     describe "A project with two same-named, header-less sources, each identified by a disambiguating TEST_SOURCE_FILE() path" do
     # =========================================================================
@@ -165,8 +194,8 @@ ceedling_system_tests do
           # is untouched by the override), so its own NOTICE still fires -- but the
           # correctly-linked pass above already proves only beta/dup.c was ever compiled,
           # not both. A duplicate-symbol link error would have failed the build otherwise.
-          expect(File.exist?('build/test/out/test_dup_override/beta/dup.o')).to be true
-          expect(File.exist?('build/test/out/test_dup_override/alpha/dup.o')).to be false
+          expect(File.exist?('build/test/out/test/test_dup_override/src/beta/dup.o')).to be true
+          expect(File.exist?('build/test/out/test/test_dup_override/src/alpha/dup.o')).to be false
 
           # A second, distinct NOTICE -- from extract_sources's own override, not the
           # header-ambiguity NOTICE above -- names the winning TEST_SOURCE_FILE() entry
@@ -195,8 +224,8 @@ ceedling_system_tests do
           expect(output).to match(/TESTED:\s+1/)
           expect(output).to match(/PASSED:\s+1/)
 
-          expect(File.exist?('build/test/out/test_dup_subtractive/alpha/dup.o')).to be false
-          expect(File.exist?('build/test/out/test_dup_subtractive/gamma/dup_gamma.o')).to be true
+          expect(File.exist?('build/test/out/test/test_dup_subtractive/src/alpha/dup.o')).to be false
+          expect(File.exist?('build/test/out/test/test_dup_subtractive/src/gamma/dup_gamma.o')).to be true
 
           expect(output).to match(/TEST_SOURCE_FILE\(.*-:alpha[\/\\]dup\.c.*\)/)
           expect(output).to match(/removed/)
@@ -222,7 +251,7 @@ ceedling_system_tests do
           expect(output).to match(/TESTED:\s+1/)
           expect(output).to match(/PASSED:\s+1/)
 
-          expect(File.exist?('build/test/out/test_dup_subtractive_noop/alpha/dup.o')).to be true
+          expect(File.exist?('build/test/out/test/test_dup_subtractive_noop/src/alpha/dup.o')).to be true
 
           expect(output).to_not match(/TEST_SOURCE_FILE\(.*removed/)
         end

data/spec/units/bin/actionator_spec.rb

@@ -213,11 +213,10 @@ describe Actionator do
       allow(@file_wrapper).to receive(:dirname) { |path| File.dirname( path ) }
       allow(@file_wrapper).to receive(:mkdir)
       allow(@file_wrapper).to receive(:write)
-      allow(@file_wrapper).to receive(:directory?).and_return( false )
     end
 
     it 'copies files and omits junk files' do
-      allow(@file_wrapper).to receive(:directory_listing).and_return([
+      allow(@file_wrapper).to receive(:tree_files).and_return([
         src_path('tree', 'keep.txt'),
         src_path('tree', 'thumbs.db'),
         src_path('tree', '.DS_Store'),
@@ -231,25 +230,25 @@ describe Actionator do
     end
 
     it 'rebases nested sources onto the destination' do
-      allow(@file_wrapper).to receive(:directory_listing).and_return([ src_path('tree', 'a', 'b', 'c.txt') ])
+      allow(@file_wrapper).to receive(:tree_files).and_return([ src_path('tree', 'a', 'b', 'c.txt') ])
 
       expect(@file_wrapper).to receive(:write).with( dest_path('out', 'a', 'b', 'c.txt'), 'contents', 'wb' )
 
       @actionator.copy_directory( 'tree', 'out', force: true )
     end
 
-    it 'skips directory entries in the listing' do
-      allow(@file_wrapper).to receive(:directory_listing).and_return([ src_path('tree', 'sub') ])
-      allow(@file_wrapper).to receive(:directory?).with( src_path('tree', 'sub') ).and_return( true )
-
-      expect(@file_wrapper).to_not receive(:write).with( /sub/, anything, anything )
+    it 'lists the source without entries named with a leading dot' do
+      skip_rule = nil
+      allow(@file_wrapper).to receive(:tree_files) { |_dir, skip:| skip_rule = skip; [] }
 
       @actionator.copy_directory( 'tree', 'out', force: true )
+
+      expect( ['.rspec', '.hidden', 'keep.c'].reject { |name| skip_rule.call( name ) } ).to eq( ['keep.c'] )
     end
 
     # Thor reported the destination directory itself before any of its contents.
     it 'reports the destination directory before its files' do
-      allow(@file_wrapper).to receive(:directory_listing).and_return([ src_path('tree', 'keep.txt') ])
+      allow(@file_wrapper).to receive(:tree_files).and_return([ src_path('tree', 'keep.txt') ])
 
       reported = []
       allow(@loginator).to receive(:console) { |message, _label| reported << message }
@@ -262,24 +261,22 @@ describe Actionator do
       ])
     end
 
-    # Issue #104: a Ceedling install path can contain literal glob metacharacters.
-    # FileWrapper#directory_listing does not escape, so Actionator has to.
-    it 'escapes glob metacharacters in the source path' do
+    # Issue #104: a Ceedling install path can contain literal glob metacharacters. The
+    # tree is listed by walking it, so the path passes through as written.
+    it 'lists a source path holding glob metacharacters as written' do
       bracketed_root = File.expand_path( '/src/[legacy]' )
       @actionator.source_root = bracketed_root
-      allow(@file_wrapper).to receive(:directory_listing).and_return([])
 
-      escaped = bracketed_root.gsub( /[*?{}\[\]]/ ) { |char| '\\' + char }
-      expect(@file_wrapper).to receive(:directory_listing).with( File.join( escaped, 'tree', '**', '*' ) )
+      expect(@file_wrapper).to receive(:tree_files).with( File.join( bracketed_root, 'tree' ), skip: Actionator::HIDDEN ).and_return([])
 
       @actionator.copy_directory( 'tree', 'out', force: true )
     end
 
     # cli_helper.rb passes component paths with a trailing separator.
     it 'tolerates a trailing separator on the source' do
-      allow(@file_wrapper).to receive(:directory_listing).and_return([ src_path('tree', 'keep.txt') ])
+      allow(@file_wrapper).to receive(:tree_files).and_return([ src_path('tree', 'keep.txt') ])
 
-      expect(@file_wrapper).to receive(:directory_listing).with( File.join( src_path('tree'), '**', '*' ) )
+      expect(@file_wrapper).to receive(:tree_files).with( src_path('tree'), skip: Actionator::HIDDEN )
       expect(@file_wrapper).to receive(:write).with( dest_path('out', 'keep.txt'), 'contents', 'wb' )
 
       @actionator.copy_directory( 'tree/', 'out', force: true )

data/spec/units/bin/app_cfg_spec.rb

@@ -44,6 +44,43 @@ describe CeedlingAppConfig do
     end
   end
 
+  describe ':launcher_root_path' do
+    it 'starts as the root of the installation this code runs from' do
+      expect(@app_cfg[:launcher_root_path]).to eq( @app_cfg[:ceedling_root_path] )
+    end
+
+    # GH #1365: `version` needs the launcher after which-Ceedling resolution moves the paths
+    it 'keeps the launcher root when #set_paths moves the application paths' do
+      launcher = @app_cfg[:ceedling_root_path]
+
+      @app_cfg.set_paths( '/tmp/some/root' )
+
+      expect(@app_cfg[:launcher_root_path]).to eq( launcher )
+    end
+  end
+
+  describe 'launch context' do
+    it 'keeps a frozen copy of the command line it was given' do
+      argv = ['test:all', '--verbosity=obnoxious']
+      app_cfg = described_class.new( argv: argv )
+      argv << 'mutated'
+
+      expect(app_cfg[:argv]).to eq( ['test:all', '--verbosity=obnoxious'] )
+      expect(app_cfg[:argv]).to be_frozen
+    end
+
+    it 'defaults to an empty command line and no hand-off' do
+      expect(@app_cfg[:argv]).to eq( [] )
+      expect(@app_cfg[:handoff]).to be_nil
+    end
+
+    it '#set_handoff keeps the markers a launcher handed this process' do
+      @app_cfg.set_handoff( {launcher: '/gem', target: '/app'} )
+
+      expect(@app_cfg[:handoff]).to eq( {launcher: '/gem', target: '/app'} )
+    end
+  end
+
   describe 'simple setters' do
     it '#set_project_config updates :project_config' do
       @app_cfg.set_project_config( {:project => {}} )