ceedling 1.2.1 → 1.2.2

data/spec/integration/test_naming_spec.rb

@@ -0,0 +1,198 @@
+# =========================================================================
+#   Ceedling - Test-Centered Build System for C
+#   ThrowTheSwitch.org
+#   Copyright (c) 2010-26 Mike Karlesky, Mark VanderVoord, & Greg Williams
+#   SPDX-License-Identifier: MIT
+# =========================================================================
+
+# How tests are named, across project layouts with same-named tests (GH #1364). A project's YAML runs through the real
+# configuration pipeline in a temporary project, so test paths and collections come from
+# real globs. A real FilePathUtils names each test, and a real TestBuildSetup prepares its
+# directories, as stage 1 of a build does.
+
+require 'spec_helper'
+require 'configurator_integration_helper'
+require 'ceedling/filesystem/file_path_utils'
+require 'ceedling/filesystem/file_finder'
+require 'ceedling/filesystem/file_finder_helper'
+require 'ceedling/test_invoker/test_build_setup'
+require 'ceedling/test_invoker/test_invoker_types'
+
+describe 'Test naming (integration)' do
+  include ConfiguratorIntegrationHelpers
+
+  # Runs each test of a batch in turn, in place of the thread pool
+  class TestNamingBatchinator
+    def exec(things:, **_options)
+      things.each { |thing| yield thing }
+    end
+  end
+
+  # name => [test paths, `:paths ↳ :test` entries, `:files ↳ :test` entries]
+  TEST_NAMING_LAYOUTS = {
+    'flat and nested under one root' => [%w[test/test_calc.c test/unit/test_calc.c], %w[test/**], []],
+    'the same subdirectory under two roots' => [%w[test/unit/test_calc.c extra/unit/test_calc.c], %w[test/unit/** extra/**], []],
+    'overlapping roots' => [%w[test/test_calc.c test/unit/test_calc.c], %w[test/** test/unit/**], []],
+    'the same layout under two roots' => [%w[test/foo/test_x.c extra/foo/test_x.c], %w[test/** extra/**], []],
+    'a :files test outside every root' => [%w[test/test_calc.c other/test_calc.c], %w[test/**], %w[other/test_calc.c]]
+  }.freeze
+
+  def project_yaml(test_paths, test_files)
+    return <<~YAML
+      :project:
+        :build_root: build
+      :paths:
+        :test: #{test_paths.inspect}
+        :source: [src/**]
+        :include: [src/**]
+      :files:
+        :test: #{test_files.inspect}
+    YAML
+  end
+
+  def project_files(tests)
+    files = { 'src/calc.c' => "int calc(void) { return 1; }\n", 'src/calc.h' => "int calc(void);\n" }
+    tests.each { |test| files[test] = "#include \"unity.h\"\nvoid test_it(void) {}\n" }
+    return files
+  end
+
+  # Stage 1 of a build of `tests`, with real naming and real directories
+  def prepare(configurator, tests)
+    file_wrapper    = FileWrapper.new( loginator: double( 'loginator', log: nil ), verbosinator: double( 'verbosinator', should_output?: false ) )
+    file_path_utils = FilePathUtils.new( configurator: configurator, file_wrapper: file_wrapper )
+
+    collaborators = TestBuildSetup.constructor_keys.to_h { |key| [key, double( key.to_s )] }
+    setup = TestBuildSetup.new( collaborators.merge(
+      configurator: configurator, file_wrapper: file_wrapper, file_path_utils: file_path_utils, batchinator: TestNamingBatchinator.new
+    ) )
+
+    state = TestInvokerTypes::PipelineState.new( tests: tests, testables: {}, context: :test, options: [], lock: Mutex.new )
+    setup.stage_prepare_build_paths( state )
+    return [state, file_path_utils]
+  end
+
+  def finder(configurator, file_path_utils)
+    loginator = double( 'loginator', log: nil )
+    return FileFinder.new(
+      configurator: configurator, file_finder_helper: FileFinderHelper.new( loginator: loginator ), file_path_utils: file_path_utils,
+      file_wrapper: FileWrapper.new( loginator: loginator, verbosinator: double( 'verbosinator', should_output?: false ) ), yaml_wrapper: nil
+    )
+  end
+
+  # Each test's own object resolves to that test, by name and from its object path alone
+  def expect_own_objects_resolve(configurator, state, file_path_utils)
+    finder = finder( configurator, file_path_utils )
+    state.testables.each_value do |testable|
+      object = File.join( testable.paths[:build], File.basename( testable.filepath ).ext( '.o' ) )
+      expect(finder.find_build_input_file( filepath: object, context: :test, test: testable.name )).to eq( testable.filepath )
+      expect(finder.find_build_input_file( filepath: object, context: :test )).to eq( testable.filepath )
+    end
+  end
+
+  TEST_NAMING_LAYOUTS.each do |layout, (tests, test_paths, test_files)|
+    context "with #{layout}" do
+      it 'names each test by its path, inside the build directory, with results where summaries look' do
+        in_temp_project( project_files( tests ) ) do
+          configurator = configure( project_yaml( test_paths, test_files ) )[:configurator]
+          expect(configurator.collection_all_tests).to match_array( tests )
+
+          state, file_path_utils = prepare( configurator, configurator.collection_all_tests )
+
+          expect(state.testables.keys.map( &:to_s )).to match_array( tests.map { |test| test.ext( '' ) } )
+
+          build = File.expand_path( 'build' ) + '/'
+          state.testables.each_value do |testable|
+            testable.paths.each_value { |path| expect(File.expand_path( path )).to start_with( build ) }
+          end
+
+          expected = state.testables.values.map { |testable| file_path_utils.form_pass_results_filepath( testable.paths[:results], testable.filepath ) }
+          expect(file_path_utils.form_pass_results_filelist( configurator.project_test_results_path, tests )).to match_array( expected )
+          expect(expected.uniq.size).to eq( tests.size )
+
+          expect_own_objects_resolve( configurator, state, file_path_utils )
+        end
+      end
+    end
+  end
+
+  # One test's name continues another's, so its directories nest inside the other's
+  context 'with a test directory named like a test beside it' do
+    it "keeps each test's mocks its own" do
+      tests = %w[test/test_foo.c test/test_foo/test_bar.c]
+      in_temp_project( project_files( tests ) ) do
+        configurator = configure( project_yaml( %w[test/**], [] ) )[:configurator]
+        state, file_path_utils = prepare( configurator, configurator.collection_all_tests )
+        expect_own_objects_resolve( configurator, state, file_path_utils )
+
+        mock = "#{configurator.cmock_mock_prefix}widget"
+        state.testables.each_value do |testable|
+          FileUtils.mkdir_p( testable.paths[:mocks] )
+          File.write( File.join( testable.paths[:mocks], "#{mock}.c" ), "\n" )
+        end
+
+        finder = finder( configurator, file_path_utils )
+        state.testables.each_value do |testable|
+          object = File.join( testable.paths[:build], "#{mock}.o" )
+          expect(finder.find_build_input_file( filepath: object, context: :test, test: testable.name )).to eq( File.join( testable.paths[:mocks], "#{mock}.c" ) )
+        end
+      end
+    end
+  end
+
+  context 'with tests outside the project' do
+    def outside_files
+      return project_files( [] ).transform_keys { |path| File.join( 'app', path ) }.merge(
+        'shared/test/unit/test_s.c' => "void test_it(void) {}\n",
+        'other/test_o.c'            => "void test_it(void) {}\n"
+      )
+    end
+
+    it 'keeps their names and directories inside the build directory' do
+      in_temp_project( outside_files ) do
+        Dir.chdir( 'app' ) do
+          configurator = configure( project_yaml( %w[../shared/test/**], %w[../other/test_o.c] ) )[:configurator]
+
+          state, file_path_utils = prepare( configurator, configurator.collection_all_tests )
+
+          expect(state.testables.keys.map( &:to_s )).to match_array( %w[_up_/shared/test/unit/test_s _up_/other/test_o] )
+          expect_own_objects_resolve( configurator, state, file_path_utils )
+          build = File.expand_path( 'build' ) + '/'
+          state.testables.each_value do |testable|
+            testable.paths.each_value { |path| expect(File.expand_path( path )).to start_with( build ) }
+          end
+        end
+      end
+    end
+
+    # A project directory named like the marker a test outside the project is named by keeps
+    # a name of its own
+    it 'names a test below a project directory called `_up_` apart from one above the project' do
+      files = outside_files.merge( 'app/_up_/other/test_o.c' => "void test_it(void) {}\n" )
+
+      in_temp_project( files ) do
+        Dir.chdir( 'app' ) do
+          configurator = configure( project_yaml( %w[../shared/test/** _up_/**], %w[../other/test_o.c] ) )[:configurator]
+
+          state, _file_path_utils = prepare( configurator, ['_up_/other/test_o.c', '../other/test_o.c'] )
+
+          expect(state.testables.keys.map( &:to_s )).to match_array( %w[__up_/other/test_o _up_/other/test_o] )
+        end
+      end
+    end
+
+    it 'stops a build that has two tests sharing one name, naming both' do
+      files = outside_files.merge( 'app/test/test_d.c' => "void test_it(void) {}\n", 'app/test/test_d.cpp' => "void test_it(void) {}\n" )
+
+      in_temp_project( files ) do
+        Dir.chdir( 'app' ) do
+          yaml = project_yaml( %w[test/**], [] ) + ":extension:\n  :source: ['.c', '.cpp']\n"
+          configurator = configure( yaml )[:configurator]
+
+          expect { prepare( configurator, ['test/test_d.c'] ) }.to raise_error( CeedlingException ) { |error|
+            expect(error.message).to include( 'test/test_d.c', 'test/test_d.cpp', '`test/test_d`' )
+          }
+        end
+      end
+    end
+  end
+end

data/spec/integration/test_source_file_checks_spec.rb

@@ -0,0 +1,166 @@
+# =========================================================================
+#   Ceedling - Test-Centered Build System for C
+#   ThrowTheSwitch.org
+#   Copyright (c) 2010-26 Mike Karlesky, Mark VanderVoord, & Greg Williams
+#   SPDX-License-Identifier: MIT
+# =========================================================================
+
+# Checking TEST_SOURCE_FILE() entries against a real project. Real test files are scanned by
+# the real context extractor, and the real resolver looks each entry up through the real
+# file finder in a temporary project configured by the real configuration pipeline.
+#
+# Each test's entries are checked once, under that test's own name, with test preprocessing
+# on or off, so a misspelled entry stops the build naming the test that holds it (GH #1364).
+
+require 'spec_helper'
+require 'configurator_integration_helper'
+require 'ceedling/parsing_parcels'
+require 'ceedling/includes/include_factory'
+require 'ceedling/filesystem/file_finder'
+require 'ceedling/filesystem/file_finder_helper'
+require 'ceedling/filesystem/file_path_utils'
+require 'ceedling/test_context_extractor'
+require 'ceedling/test_invoker/test_source_file_directive_resolver'
+require 'ceedling/test_invoker/test_build_setup'
+require 'ceedling/test_invoker/test_invoker_types'
+
+describe 'TEST_SOURCE_FILE() checks (integration)' do
+  include ConfiguratorIntegrationHelpers
+
+  # Runs each test of a batch in turn, in place of the thread pool
+  class TestSourceFileChecksBatchinator
+    def exec(things:, **_options)
+      things.each { |key, value| yield key, value }
+    end
+  end
+
+  def test_file(entries)
+    lines = entries.map { |entry| "TEST_SOURCE_FILE(\"#{entry}\")\n" }.join
+    return "#include \"unity.h\"\n#{lines}void test_it(void) {}\n"
+  end
+
+  def project_files
+    return {
+      'src/calc.c'        => "int calc(void) { return 1; }\n",
+      'src/calc.h'        => "int calc(void);\n",
+      'src/a/dup.c'       => "int dup(void) { return 1; }\n",
+      'lib/a/dup.c'       => "int dup(void) { return 2; }\n",
+      'lib/extra.c'       => "int extra(void) { return 7; }\n",
+      'test/test_m1.c'    => test_file( ['calc.c'] ),
+      'test/test_m2.c'    => test_file( ['calc.c'] ),
+      'test/unit/test_m3.c' => test_file( ['../../lib/extra.c'] )
+    }
+  end
+
+  def project_yaml(preprocessor: :none)
+    return <<~YAML
+      :project:
+        :build_root: build
+        :use_test_preprocessor: :#{preprocessor}
+      :paths:
+        :test: [test/**]
+        :source: [src/**, lib/**]
+        :include: [src/**]
+    YAML
+  end
+
+  def objects
+    graph        = configure( project_yaml )
+    configurator = graph[:configurator]
+    loginator    = graph[:loginator]
+    file_wrapper = graph[:file_wrapper]
+
+    file_path_utils = FilePathUtils.new( configurator: configurator, file_wrapper: file_wrapper )
+    file_finder     = FileFinder.new(
+      configurator: configurator, file_finder_helper: FileFinderHelper.new( loginator: loginator ),
+      file_path_utils: file_path_utils, file_wrapper: file_wrapper, yaml_wrapper: graph[:yaml_wrapper]
+    )
+    extractor = TestContextExtractor.new(
+      configurator: configurator, parsing_parcels: ParsingParcels.new, include_factory: IncludeFactory.new( configurator: configurator ),
+      partializer_config: double( 'partializer_config' ).as_null_object, file_path_utils: file_path_utils, file_wrapper: file_wrapper,
+      yaml_wrapper: graph[:yaml_wrapper], loginator: loginator
+    )
+    resolver = TestSourceFileDirectiveResolver.new(
+      test_context_extractor: extractor, file_finder: file_finder, configurator: configurator, loginator: loginator
+    )
+
+    return { configurator: configurator, file_path_utils: file_path_utils, file_finder: file_finder, extractor: extractor, resolver: resolver }
+  end
+
+  def scan(extractor, tests)
+    tests.each { |test| extractor.collect_simple_context_from_file( test, nil, TestContextExtractor::Context::BUILD_DIRECTIVE_SOURCE_FILES ) }
+  end
+
+  it "passes a test's real entries, including one reached through .." do
+    in_temp_project( project_files ) do
+      o = objects
+      tests = %w[test/test_m1.c test/test_m2.c test/unit/test_m3.c]
+      scan( o[:extractor], tests )
+
+      tests.each do |test|
+        expect { o[:resolver].validate!( test: test.ext( '' ), test_filepath: test, context: :test ) }.to_not raise_error
+      end
+    end
+  end
+
+  it 'names the test that holds a missing entry' do
+    in_temp_project( project_files.merge( 'test/test_m2.c' => test_file( ['calc_typo.c'] ) ) ) do
+      o = objects
+      scan( o[:extractor], %w[test/test_m1.c test/test_m2.c] )
+
+      expect { o[:resolver].validate!( test: 'test/test_m1', test_filepath: 'test/test_m1.c', context: :test ) }.to_not raise_error
+      expect { o[:resolver].validate!( test: 'test/test_m2', test_filepath: 'test/test_m2.c', context: :test ) }.to raise_error(
+        CeedlingException, %r{'calc_typo\.c' specified with TEST_SOURCE_FILE\(\) in test/test_m2 cannot be found}
+      )
+    end
+  end
+
+  it 'stops for a misspelled removal entry, which otherwise leaves its file in the build' do
+    in_temp_project( project_files.merge( 'test/test_m1.c' => test_file( ['-:clac.c'] ) ) ) do
+      o = objects
+      scan( o[:extractor], %w[test/test_m1.c] )
+
+      expect { o[:resolver].validate!( test: 'test/test_m1', test_filepath: 'test/test_m1.c', context: :test ) }.to raise_error(
+        CeedlingException, /'clac\.c'.*cannot be found/
+      )
+    end
+  end
+
+  it 'notes an entry matching several files once, however many tests name it' do
+    files = project_files.merge( %w[test/test_m1.c test/test_m2.c test/unit/test_m3.c].to_h { |test| [test, test_file( ['a/dup.c'] )] } )
+
+    in_temp_project( files ) do
+      o = objects
+      tests = %w[test/test_m1.c test/test_m2.c test/unit/test_m3.c]
+      scan( o[:extractor], tests )
+
+      tests.each do |test|
+        o[:resolver].validate!( test: test.ext( '' ), test_filepath: test, context: :test )
+        o[:resolver].resolve( test, :test )
+      end
+
+      expect( logged.grep( /Multiple files matched 'a\/dup\.c'/ ).size ).to eq( 1 )
+    end
+  end
+
+  # Without test preprocessing, stage 2 scans the entries and checks them
+  it 'checks each test under its own name in stage 2 without test preprocessing' do
+    in_temp_project( project_files.merge( 'test/unit/test_m3.c' => test_file( ['nowhere.c'] ) ) ) do
+      o = objects
+      setup = TestBuildSetup.new( TestBuildSetup.constructor_keys.to_h { |key| [key, double( key.to_s ).as_null_object] }.merge(
+        configurator: o[:configurator], batchinator: TestSourceFileChecksBatchinator.new, test_context_extractor: o[:extractor],
+        file_path_utils: o[:file_path_utils], file_finder: o[:file_finder], test_source_file_directive_resolver: o[:resolver],
+        file_wrapper: FileWrapper.new( loginator: double( 'loginator', log: nil ), verbosinator: double( 'verbosinator', should_output?: false ) ),
+        dependinator: double( 'dependinator', register: nil, stale?: true, mark_fresh: nil ),
+        reportinator: double( 'reportinator', generate_progress: '', generate_skip_summary: nil )
+      ) )
+      tests = %w[test/test_m1.c test/test_m2.c test/unit/test_m3.c]
+      state = TestInvokerTypes::PipelineState.new( tests: tests, testables: {}, context: :test, options: [], lock: Mutex.new )
+      setup.stage_prepare_build_paths( state )
+
+      expect { setup.stage_collect_test_context( state ) }.to raise_error(
+        CeedlingException, %r{'nowhere\.c' specified with TEST_SOURCE_FILE\(\) in test/unit/test_m3 cannot be found}
+      )
+    end
+  end
+end

data/spec/integration/tool_validator_spec.rb

@@ -17,7 +17,7 @@ require 'spec_helper'
 require 'spec_integration_helper'
 require 'fileutils'
 require 'ceedling/tool_validator'
-require 'ceedling/file_wrapper'
+require 'ceedling/filesystem/file_wrapper'
 require 'ceedling/system_wrapper'
 require 'ceedling/ruby_expandinator'
 require 'ceedling/filename_extension'

data/spec/integration/upgrade_docs_spec.rb

@@ -0,0 +1,84 @@
+# =========================================================================
+#   Ceedling - Test-Centered Build System for C
+#   ThrowTheSwitch.org
+#   Copyright (c) 2010-26 Mike Karlesky, Mark VanderVoord, & Greg Williams
+#   SPDX-License-Identifier: MIT
+# =========================================================================
+
+# GH #1371: `upgrade` never refreshed documentation that `new --docs` exported, because it
+# looked for a file exports stopped writing in 1.1.0. For a 1.0.x export, which it did
+# find, it removed all of docs/, the project's own files included.
+#
+# These run the real launcher against real projects. Nothing compiles, so no toolchain is
+# needed.
+
+require 'spec_helper'
+require 'tmpdir'
+require 'fileutils'
+require 'open3'
+require 'rbconfig'
+
+describe '`upgrade` and exported documentation' do
+  REPO = File.expand_path( '../..', __dir__ )
+  UNITY_PAGE = File.join( 'unity', 'UnityGettingStartedGuide.md' )
+
+  def ceedling(*args)
+    stdout, stderr, status = Open3.capture3(
+      {'CEEDLING_DECORATORS' => 'false'}, RbConfig.ruby, File.join( REPO, 'bin', 'ceedling' ), *args, chdir: @tmp
+    )
+    raise "`ceedling #{args.join( ' ' )}` failed:\n#{stdout}#{stderr}" unless status.success?
+  end
+
+  def docs(*entry)
+    return File.join( @tmp, 'proj', 'docs', *entry )
+  end
+
+  def write(entry, text='')
+    FileUtils.mkdir_p( File.dirname( docs( entry ) ) )
+    File.write( docs( entry ), text )
+  end
+
+  around(:each) do |example|
+    Dir.mktmpdir( 'upgrade_docs' ) { |dir| @tmp = dir; example.run }
+  end
+
+  it 'refreshes an export of 1.1.0 or later and keeps the project\'s own files' do
+    ceedling( 'new', '--local', '--docs', 'proj' )
+    write( 'notes.md', 'mine' )
+    write( File.join( 'unity', 'stale.md' ) )
+    write( UNITY_PAGE, 'edited' )
+
+    ceedling( 'upgrade', 'proj' )
+
+    expect(File.exist?( docs( 'unity', 'stale.md' ) )).to be( false )
+    expect(File.read( docs( UNITY_PAGE ) )).to eq( File.read( File.join( REPO, 'vendor', 'unity', 'docs', 'UnityGettingStartedGuide.md' ) ) )
+    expect(File.read( docs( 'notes.md' ) )).to eq( 'mine' )
+  end
+
+  it 'replaces a 1.0.x export with the current layout and keeps the project\'s own files' do
+    ceedling( 'new', '--local', 'proj' )
+    write( 'CeedlingPacket.md' )
+    write( 'Changelog.md' )
+    write( File.join( 'plugins', 'gcov.md' ) )
+    write( File.join( 'unity', 'stale.md' ) )
+    write( 'license.txt' )
+    write( 'notes.md', 'mine' )
+
+    ceedling( 'upgrade', 'proj' )
+
+    ['CeedlingPacket.md', 'Changelog.md', 'plugins', File.join( 'unity', 'stale.md' )].each do |entry|
+      expect(File.exist?( docs( entry ) )).to be( false ), entry
+    end
+    expect(File.exist?( docs( UNITY_PAGE ) )).to be( true )
+    expect(File.read( docs( 'notes.md' ) )).to eq( 'mine' )
+  end
+
+  it 'leaves a docs/ holding only the project\'s own files alone' do
+    ceedling( 'new', '--local', 'proj' )
+    write( 'notes.md', 'mine' )
+
+    ceedling( 'upgrade', 'proj' )
+
+    expect(Dir.children( docs )).to eq( ['notes.md'] )
+  end
+end

data/spec/integration/vendor_copy_transient_io_retry_spec.rb

@@ -8,26 +8,29 @@
 # ConfiguratorSetup#vendor_frameworks_and_support_files with a real FileWrapper on a real
 # filesystem. A build brings build/vendor up to date with the gem's frameworks. It writes only
 # a file that differs from its source, and replaces it whole by renaming a temporary copy over
-# it, so a concurrent build never reads a half-written header (GH #1361). A transient copy
-# fault, such as a path briefly the wrong type (GH #1292), is retried, and one that never
+# it, so a concurrent build never reads a half-written header (GH #1361). An entry of the
+# wrong type, such as a directory where a file belongs, is replaced (GH #1292, #1376). A
+# transient copy fault, such as a scanner holding a file open, is retried, and one that never
 # clears names the destination.
 #
 # Faults are injected at FileWrapper#replace_file, the one write of each file, so the retry
-# loop in FileWrapper#sync_tree is what runs.
+# loop in FileTree is what runs.
 
 require 'spec_helper'
 require 'spec_integration_helper'
+require 'file_tree_helper'
 require 'ceedling/config/configurator_setup'
 require 'ceedling/constants'
 
 describe 'Vendor copy into build/vendor (integration)' do
   include IntegrationSpecHelpers
+  include FileTreeHelper
 
   # A real FileWrapper whose #replace_file raises for files beneath `fault_dir` the first
   # `raises_left` times. Named apart from other specs' faulting wrappers, since top-level
   # classes share one RSpec process.
   class ReplaceFaultingFileWrapper < FileWrapper
-    def initialize(fault_dir:, raises_left: 1, error: Errno::EISDIR, **kwargs)
+    def initialize(fault_dir:, raises_left: 1, error: Errno::EACCES, **kwargs)
       super(**kwargs)
       @fault_dir = fault_dir
       @raises_left = raises_left
@@ -37,7 +40,7 @@ describe 'Vendor copy into build/vendor (integration)' do
     def replace_file(source, destination)
       if destination.start_with?( @fault_dir ) && @raises_left > 0
         @raises_left -= 1
-        raise @error, "simulated transient type-mismatch on #{destination}"
+        raise @error, "simulated fault on #{destination}"
       end
       super
     end
@@ -52,6 +55,7 @@ describe 'Vendor copy into build/vendor (integration)' do
     ConfiguratorSetup.new(
       configurator_builder:   IntegrationSpecHelpers::NULL,
       configurator_validator: IntegrationSpecHelpers::NULL,
+      configurator_paths_validator: IntegrationSpecHelpers::NULL,
       loginator:              IntegrationSpecHelpers::NULL,
       reportinator:           IntegrationSpecHelpers::NULL,
       file_wrapper:           file_wrapper,
@@ -175,4 +179,69 @@ describe 'Vendor copy into build/vendor (integration)' do
       }.to raise_error( CeedlingException, /#{Regexp.escape( dest )}/ )
     end
   end
+  # GH #1376: a deduplicated file the reporter's lstat called a directory left one where any
+  # vendored file belongs, not only the framework's own .c file
+  it "replaces a directory where any vendored file belongs" do
+    with_source_tree({}) do |dir|
+      config = flattened_config( dir )
+      dest = config[:project_build_vendor_unity_path]
+      FileUtils.mkdir_p( File.join( dest, 'unity.h' ) )
+      write( File.join( dest, 'internal', 'unity_internals.h', 'junk' ) )
+
+      vendor( file_wrapper, dir, config )
+
+      expect( File.read( File.join( dest, 'unity.h' ) ) ).to eq( "// stand-in for unity.h\n" )
+      expect( File.file?( File.join( dest, 'internal', 'unity_internals.h' ) ) ).to be(true)
+    end
+  end
+
+  it "vendors every file as a file when lstat reports files as directories" do
+    with_source_tree({}) do |dir|
+      config = flattened_config( dir )
+      dest = config[:project_build_vendor_unity_path]
+      lie_about_lstat( %r{gem_vendor/} )
+
+      begin
+        vendor( file_wrapper, dir, config )
+      ensure
+        # The temporary directory is removed by an lstat-based remover
+        allow(File).to receive(:lstat).and_call_original
+      end
+
+      expect( File.file?( File.join( dest, 'unity.c' ) ) ).to be(true)
+      expect( File.file?( File.join( dest, 'internal', 'unity_internals.h' ) ) ).to be(true)
+    end
+  end
+
+  # A read-only gem install would otherwise vendor read-only files, which Windows will not
+  # let a later rename replace
+  it "vendors owner-writable files from a read-only source, and replaces a read-only destination" do
+    with_source_tree({}) do |dir|
+      config = flattened_config( dir )
+      dest = config[:project_build_vendor_unity_path]
+      source = File.join( config[:unity_vendor_path], 'unity', 'src', 'unity.h' )
+      File.chmod( 0o444, source )
+      vendor( file_wrapper, dir, config )
+      expect( File.stat( File.join( dest, 'unity.h' ) ).mode & 0o200 ).to eq( 0o200 )
+
+      File.chmod( 0o644, source )
+      File.write( source, "// a newer unity.h\n" )
+      File.chmod( 0o444, File.join( dest, 'unity.h' ) )
+      vendor( file_wrapper, dir, config )
+
+      expect( File.read( File.join( dest, 'unity.h' ) ) ).to eq( "// a newer unity.h\n" )
+    end
+  end
+
+  it "fails at once on an error retrying cannot clear" do
+    with_source_tree({}) do |dir|
+      config = flattened_config( dir )
+      dest = config[:project_build_vendor_unity_path]
+      faulting = file_wrapper( fault_dir: dest, raises_left: 999, error: Errno::ENOSPC )
+      allow(faulting).to receive(:sleep)
+
+      expect { vendor( faulting, dir, config ) }.to raise_error( Errno::ENOSPC )
+      expect(faulting).not_to have_received(:sleep)
+    end
+  end
 end

data/spec/support/file_tree_helper.rb

@@ -0,0 +1,88 @@
+# =========================================================================
+#   Ceedling - Test-Centered Build System for C
+#   ThrowTheSwitch.org
+#   Copyright (c) 2010-26 Mike Karlesky, Mark VanderVoord, & Greg Williams
+#   SPDX-License-Identifier: MIT
+# =========================================================================
+
+require 'delegate'
+require 'fileutils'
+require 'tmpdir'
+
+# Support for specs that copy real file trees (`filesystem: true`).
+#
+# CEEDLING_TEST_FS_ROOT names a directory on another filesystem, such as a tmpfs, a vfat
+# image, a UNC share, or a VHD volume. These specs then build their trees there instead of
+# the system temp directory, so CI can run one suite on several filesystems.
+module FileTreeHelper
+  ROOT_ENV = 'CEEDLING_TEST_FS_ROOT'
+
+  # A stat result that claims a directory. Reproduces a filesystem whose `lstat` reports a
+  # regular file as a directory.
+  class LyingStat < SimpleDelegator
+    def directory?
+      true
+    end
+
+    def file?
+      false
+    end
+
+    def symlink?
+      false
+    end
+
+    def ftype
+      'directory'
+    end
+
+    def mode
+      (__getobj__.mode & ~0o170000) | 0o040000
+    end
+  end
+
+  # A temporary directory on the filesystem under test, removed afterwards
+  def with_tree_root
+    root = ENV[ROOT_ENV].to_s
+    dir = root.empty? ? Dir.mktmpdir( 'tree' ) : Dir.mktmpdir( 'tree', root )
+    yield File.realpath( dir )
+  ensure
+    FileUtils.rm_rf( dir ) if dir
+  end
+
+  def write(path, content = File.basename( path ))
+    FileUtils.mkdir_p( File.dirname( path ) )
+    File.binwrite( path, content )
+  end
+
+  def symlink_or_skip(target, link)
+    File.symlink( target, link )
+  rescue NotImplementedError, SystemCallError => e
+    skip "symlinks are unavailable here (#{e.class})"
+  end
+
+  # Every entry below `dir`, dotfiles included, as sorted relative paths
+  def tree_of(dir)
+    Dir.glob( '**/*', File::FNM_DOTMATCH, base: dir ).reject { |path| path == '.' || path.end_with?( '/.' ) }.sort
+  end
+
+  # Makes `File.lstat` lie, claiming a directory, for each path matching `pattern`
+  def lie_about_lstat(pattern)
+    allow(File).to receive(:lstat).and_wrap_original do |original, path, *rest|
+      status = original.call( path, *rest )
+      path.to_s.match?( pattern ) ? LyingStat.new( status ) : status
+    end
+  end
+
+  # Fails any `File.lstat` call, proving code under test never classifies by it
+  def forbid_lstat
+    allow(File).to receive(:lstat).and_raise( RuntimeError, 'File.lstat must not be called' )
+  end
+
+  # The publish step FileWrapper#replace_file performs: a whole copy renamed into place
+  PUBLISH = lambda do |source, destination|
+    temp = "#{destination}.publishing"
+    FileUtils.cp( source, temp )
+    File.rename( temp, destination )
+  end
+end