ceedling 1.2.0 → 1.2.2

data/spec/units/{file_path_utils_spec.rb → filesystem/file_path_utils_spec.rb}

@@ -6,8 +6,8 @@
 # =========================================================================
 
 require 'spec_helper'
-require 'ceedling/file_path_utils'
-require 'ceedling/file_wrapper'
+require 'ceedling/filesystem/file_path_utils'
+require 'ceedling/filesystem/file_wrapper'
 require 'ceedling/filename_extension'
 
 describe FilePathUtils do
@@ -37,6 +37,12 @@ describe FilePathUtils do
       expect(result).to match_array(['a'])
     end
 
+    # The path handling audit around GH #1370: `..` holds `../sib` but not `../../far/x`
+    it 'keeps a path above the project that no other path holds' do
+      result = FilePathUtils.collapse_to_common_parents(['..', '../../far/x', '../sib'])
+      expect(result).to match_array(['..', '../../far/x'])
+    end
+
     it 'handles deeply nested redundant paths' do
       result = FilePathUtils.collapse_to_common_parents(['src', 'src/a/b/c', 'lib'])
       expect(result).to match_array(['src', 'lib'])
@@ -78,6 +84,11 @@ describe FilePathUtils do
       expect( FilePathUtils.standardize_in_place( 'foo/bar/baz' ) ).to eq( 'foo/bar/baz' )
     end
 
+    # GH #1370 audit (G21)
+    it 'reads `~` as the home directory' do
+      expect( FilePathUtils.standardize_in_place( +'~/inc' ) ).to eq( File.join( Dir.home, 'inc' ) )
+    end
+
     it 'returns nil unchanged for a nil argument' do
       expect( FilePathUtils.standardize_in_place( nil ) ).to be_nil
     end
@@ -365,25 +376,25 @@ describe FilePathUtils do
         .to eq('build/test/runners/test_foo_runner.c')
     end
 
-    it 'forms a flat runner filepath for a flat test\'s own identity' do
-      expect( @fpu.form_runner_filepath_from_test('test/test_foo.c', name: 'test_foo') )
+    it 'forms a runner filepath in the runners directory itself for a test at the project root' do
+      expect( @fpu.form_runner_filepath_from_test('test_foo.c', name: 'test_foo') )
         .to eq('build/test/runners/test_foo_runner.c')
     end
 
-    it 'mirrors the runner beneath the test\'s own mirrored subdirectory, distinguishing same-named tests' do
-      expect( @fpu.form_runner_filepath_from_test('test/unit/test_foo.c', name: 'unit/test_foo') )
-        .to eq('build/test/runners/unit/test_foo_runner.c')
+    it 'places the runner beneath the directories of the test\'s name, distinguishing same-named tests' do
+      expect( @fpu.form_runner_filepath_from_test('test/unit/test_foo.c', name: 'test/unit/test_foo') )
+        .to eq('build/test/runners/test/unit/test_foo_runner.c')
 
-      expect( @fpu.form_runner_filepath_from_test('test/integration/test_foo.c', name: 'integration/test_foo') )
-        .to eq('build/test/runners/integration/test_foo_runner.c')
+      expect( @fpu.form_runner_filepath_from_test('test/integration/test_foo.c', name: 'test/integration/test_foo') )
+        .to eq('build/test/runners/test/integration/test_foo_runner.c')
     end
 
-    it 'forms a flat runner directory for a flat test' do
+    it 'forms the runners directory itself for a test at the project root' do
       expect( @fpu.form_test_runners_path('test_foo') ).to eq('build/test/runners')
     end
 
-    it 'forms the runner directory itself beneath a nested test\'s own mirrored subdirectory' do
-      expect( @fpu.form_test_runners_path('unit/test_foo') ).to eq('build/test/runners/unit')
+    it 'forms the runner directory beneath the directories of the test\'s name' do
+      expect( @fpu.form_test_runners_path('test/unit/test_foo') ).to eq('build/test/runners/test/unit')
     end
   end
 
@@ -449,12 +460,54 @@ describe FilePathUtils do
         .to eq('build/gcov/dependencies/AdcCalc.d')
     end
 
+    # A context's own objects, outside any test, are placed by their sources' paths too
+    it 'follows a context\'s own object below its output root when no test identity is given' do
+      expect( @fpu.form_test_dependencies_filepath('build/gcov/out/src/a/util.o', context: :gcov) )
+        .to eq('build/gcov/dependencies/src/a/util.d')
+    end
+
     it 'forms the list filepath directly alongside the object, wherever it was mirrored to' do
       expect( @fpu.form_test_build_list_filepath('build/test/out/TestFoo/calculators/AdcCalc.o') )
         .to eq('build/test/out/TestFoo/calculators/AdcCalc.lst')
     end
   end
 
+  describe '#form_test_object_filepath' do
+    before(:each) do
+      @configurator = double('configurator')
+      @file_wrapper = double('file_wrapper')
+      allow(@file_wrapper).to receive(:check_path_length)
+      @fpu = described_class.new({
+        :configurator => @configurator,
+        :file_wrapper => @file_wrapper
+      })
+
+      allow(@configurator).to receive(:project_build_root).and_return('build')
+      allow(@configurator).to receive(:extension_object).and_return( FilenameExtension.new('.o') )
+      allow(@configurator).to receive(:project_use_partials).and_return( true )
+      allow(@configurator).to receive(:project_use_mocks).and_return( true )
+      allow(@configurator).to receive(:project_test_partials_path).and_return('build/test/partials')
+      allow(@configurator).to receive(:cmock_mock_path).and_return('build/test/mocks')
+    end
+
+    it 'places a test\'s object where its build does, by the source\'s path from the project root' do
+      expect( @fpu.form_test_object_filepath('src/a/util.c', name: 'test/test_util', context: :gcov) )
+        .to eq('build/gcov/out/test/test_util/src/a/util.o')
+    end
+
+    # Untested sources compile outside any test, so same-named ones keep apart only by path
+    it 'places a context\'s own objects apart, by their sources\' paths, when no test is named' do
+      expect( @fpu.form_test_object_filepath('src/a/util.c', context: :gcov) ).to eq('build/gcov/out/src/a/util.o')
+      expect( @fpu.form_test_object_filepath('src/b/util.c', context: :gcov) ).to eq('build/gcov/out/src/b/util.o')
+    end
+
+    it 'places a test\'s generated Partial below its own subdirectory, as its build does' do
+      expect(
+        @fpu.form_test_object_filepath('build/test/partials/test/test_util/drv/ceedling_partial_util_impl.c', name: 'test/test_util', context: :gcov)
+      ).to eq('build/gcov/out/test/test_util/drv/ceedling_partial_util_impl.o')
+    end
+  end
+
   describe '#form_test_preprocess_build_directives_path, #form_test_build_directives_cache_filepath, #form_preprocessed_source_files_cache_filepath' do
     before(:each) do
       @configurator = double('configurator')
@@ -497,19 +550,28 @@ describe FilePathUtils do
       })
 
       allow(@configurator).to receive(:project_release_build_output_path).and_return('build/release/out')
+      allow(@configurator).to receive(:project_build_root).and_return('build')
       allow(@configurator).to receive(:extension_object).and_return( FilenameExtension.new('.o') )
-      allow(@configurator).to receive(:paths_source).and_return( ['drivers'] )
     end
 
-    it 'mirrors each object beneath whichever configured source root matched its file, distinguishing same-named sources' do
-      objects = @fpu.form_release_build_objects_filelist( ['drivers/foo/bar.c', 'drivers/baz/bar.c'] )
-      expect( objects ).to include('build/release/out/foo/bar.o')
-      expect( objects ).to include('build/release/out/baz/bar.o')
+    it 'places each object by its source\'s path from the project root, distinguishing same-named sources' do
+      objects = @fpu.form_release_build_objects_filelist( ['drivers/foo/bar.c', 'drivers/baz/bar.c', 'lib/bar.c'] )
+      expect( objects ).to eq(['build/release/out/drivers/foo/bar.o', 'build/release/out/drivers/baz/bar.o', 'build/release/out/lib/bar.o'])
     end
 
-    it 'leaves a file matching no configured source root flat' do
-      objects = @fpu.form_release_build_objects_filelist( ['vendor/unity.c'] )
-      expect( objects ).to eq(['build/release/out/unity.o'])
+    it 'places a source above the project root below the output root' do
+      objects = @fpu.form_release_build_objects_filelist( ['../shared/src/crc.c'] )
+      expect( objects ).to eq(['build/release/out/_up_/shared/src/crc.o'])
+    end
+
+    it 'leaves a source below the build root flat' do
+      objects = @fpu.form_release_build_objects_filelist( ['build/vendor/c_exception/lib/CException.c'] )
+      expect( objects ).to eq(['build/release/out/CException.o'])
+    end
+
+    it 'refuses two sources that would build to one object, naming both' do
+      expect { @fpu.form_release_build_objects_filelist( ['build/vendor/c_exception/lib/CException.c', 'CException.c'] ) }
+        .to raise_error( CeedlingException, /`build\/vendor\/c_exception\/lib\/CException.c` and `CException.c`.*`build\/release\/out\/CException.o`/ )
     end
   end
 
@@ -526,21 +588,16 @@ describe FilePathUtils do
       })
 
       allow(@configurator).to receive(:project_release_dependencies_path).and_return('build/release/dependencies')
+      allow(@configurator).to receive(:project_build_root).and_return('build')
       allow(@configurator).to receive(:extension_dependencies).and_return( FilenameExtension.new('.d') )
-      allow(@configurator).to receive(:paths_source).and_return( ['drivers'] )
     end
 
-    it 'mirrors each dependency file the same way as its corresponding object' do
+    it 'places each dependency file the same way as its corresponding object' do
       deps = @fpu.form_release_dependencies_filelist( ['drivers/foo/bar.c', 'drivers/baz/bar.c'] )
-      expect( deps ).to include('build/release/dependencies/foo/bar.d')
-      expect( deps ).to include('build/release/dependencies/baz/bar.d')
+      expect( deps ).to eq(['build/release/dependencies/drivers/foo/bar.d', 'build/release/dependencies/drivers/baz/bar.d'])
     end
   end
 
-  # Every preprocessed artifact for a test lands below that test's own subdirectory.
-  # That alone kept artifacts apart while a test had at most one source of any given
-  # basename. Two same-basename sources in one test overwrite each other, so the
-  # source's own namespace below its configured root is mirrored as well.
   # A mock's generated files sit below its test's mock root, in the subdirectory mirroring
   # whatever it mocks. Two places need that directory -- the stage that compiles the mock
   # and the stage that generates it -- and they must agree, or a mock is compiled from a
@@ -580,46 +637,35 @@ describe FilePathUtils do
 
       allow(@configurator).to receive(:project_test_preprocess_files_path).and_return('build/test/preprocess/files')
       allow(@configurator).to receive(:project_test_preprocess_includes_path).and_return('build/test/preprocess/includes')
-      allow(@configurator).to receive(:paths_source).and_return( ['src/**'] )
-      allow(@configurator).to receive(:paths_include).and_return( ['include/**'] )
-      allow(@configurator).to receive(:paths_support).and_return( [] )
-      allow(@configurator).to receive(:paths_test).and_return( ['test/**'] )
+      allow(@configurator).to receive(:project_build_root).and_return('build')
       allow(@configurator).to receive(:project_use_partials).and_return( true )
       allow(@configurator).to receive(:project_use_mocks).and_return( true )
       allow(@configurator).to receive(:project_test_partials_path).and_return('build/test/partials')
       allow(@configurator).to receive(:cmock_mock_path).and_return('build/test/mocks')
     end
 
-    # A file sitting directly in a configured root has no namespace of its own, so its
-    # artifact stays flat -- the layout every project has always seen.
-    it 'keeps a file directly in a configured root flat' do
-      expect( @fpu.form_preprocessed_file_filepath('src/foo.c', 'test_foo') )
-        .to eq('build/test/preprocess/files/test_foo/foo.c')
-      expect( @fpu.form_preprocessed_file_full_expansion_filepath('src/foo.c', 'test_foo') )
-        .to eq('build/test/preprocess/files/test_foo/full_expansion/foo.c')
-      expect( @fpu.form_preprocessed_file_raw_directives_only_filepath('src/foo.c', 'test_foo') )
-        .to eq('build/test/preprocess/files/test_foo/directives_only/raw/foo.c')
-      expect( @fpu.form_preprocessed_file_compacted_directives_only_filepath('src/foo.c', 'test_foo') )
-        .to eq('build/test/preprocess/files/test_foo/directives_only/foo.c')
-      expect( @fpu.form_preprocessed_file_fallback_expansion_filepath('src/foo.c', 'test_foo') )
-        .to eq('build/test/preprocess/files/test_foo/fallback_expansion/foo.c')
-      expect( @fpu.form_preprocessed_includes_list_filepath('src/foo.c', 'test_foo') )
-        .to eq('build/test/preprocess/includes/test_foo/foo.c.yml')
-    end
-
-    it 'mirrors a nested source below the test subdirectory' do
+    # Each artifact of a test sits below the test's own subdirectory, at its file's path
+    # from the project root, so two same-named files in one test never share one
+    it 'places a file by its path from the project root below the test subdirectory' do
       expect( @fpu.form_preprocessed_file_filepath('src/drivers/uart/config.c', 'test_both') )
-        .to eq('build/test/preprocess/files/test_both/drivers/uart/config.c')
+        .to eq('build/test/preprocess/files/test_both/src/drivers/uart/config.c')
       expect( @fpu.form_preprocessed_file_full_expansion_filepath('src/drivers/uart/config.c', 'test_both') )
-        .to eq('build/test/preprocess/files/test_both/full_expansion/drivers/uart/config.c')
+        .to eq('build/test/preprocess/files/test_both/full_expansion/src/drivers/uart/config.c')
       expect( @fpu.form_preprocessed_file_raw_directives_only_filepath('src/drivers/uart/config.c', 'test_both') )
-        .to eq('build/test/preprocess/files/test_both/directives_only/raw/drivers/uart/config.c')
+        .to eq('build/test/preprocess/files/test_both/directives_only/raw/src/drivers/uart/config.c')
       expect( @fpu.form_preprocessed_file_compacted_directives_only_filepath('src/drivers/uart/config.c', 'test_both') )
-        .to eq('build/test/preprocess/files/test_both/directives_only/drivers/uart/config.c')
+        .to eq('build/test/preprocess/files/test_both/directives_only/src/drivers/uart/config.c')
       expect( @fpu.form_preprocessed_file_fallback_expansion_filepath('src/drivers/uart/config.c', 'test_both') )
-        .to eq('build/test/preprocess/files/test_both/fallback_expansion/drivers/uart/config.c')
+        .to eq('build/test/preprocess/files/test_both/fallback_expansion/src/drivers/uart/config.c')
       expect( @fpu.form_preprocessed_includes_list_filepath('src/drivers/uart/config.c', 'test_both') )
-        .to eq('build/test/preprocess/includes/test_both/drivers/uart/config.c.yml')
+        .to eq('build/test/preprocess/includes/test_both/src/drivers/uart/config.c.yml')
+    end
+
+    it 'keeps a header far above the project below the test subdirectory' do
+      far = File.expand_path( '../../far/inc/sdk.h' )
+      expect( @fpu.form_preprocessed_file_filepath( far, 'test_both' ) )
+        .to eq( File.join( 'build/test/preprocess/files/test_both', BuildSubpath.of_directory( File.dirname( far ) ), 'sdk.h' ) )
+      expect( BuildSubpath.of_directory( File.dirname( far ) ) ).to_not include( '..' )
     end
 
     # The defect itself: two sources sharing a basename in one test.
@@ -641,9 +687,9 @@ describe FilePathUtils do
       @fpu.form_preprocessed_includes_list_filepath('src/drivers/uart/config.c', 'test_both')
     end
 
-    it 'mirrors a nested header below its own include root' do
+    it 'places a header by its path from the project root' do
       expect( @fpu.form_preprocessed_file_filepath('include/drivers/uart/config.h', 'test_both') )
-        .to eq('build/test/preprocess/files/test_both/drivers/uart/config.h')
+        .to eq('build/test/preprocess/files/test_both/include/drivers/uart/config.h')
     end
 
     # A generated Partial or mock sits below its own test's generated root rather than a
@@ -676,9 +722,7 @@ describe FilePathUtils do
       ).to eq('build/test/preprocess/files/test_both/drivers/spi/mock_config.h')
     end
 
-    # A generated file sits below no configured root, so nothing mirrors it and its
-    # artifact stays where it has always been.
-    it 'keeps a generated file flat' do
+    it 'keeps a generated file directly in its test root flat' do
       expect( @fpu.form_preprocessed_file_filepath('build/test/mocks/test_both/Mockconfig.h', 'test_both') )
         .to eq('build/test/preprocess/files/test_both/Mockconfig.h')
     end
@@ -697,27 +741,42 @@ describe FilePathUtils do
       })
 
       allow(@configurator).to receive(:extension_object).and_return( FilenameExtension.new('.o') )
-      allow(@configurator).to receive(:paths_source).and_return( ['drivers'] )
-      allow(@configurator).to receive(:paths_support).and_return( ['support'] )
+      allow(@configurator).to receive(:project_build_root).and_return('build')
     end
 
-    it 'mirrors a module-under-test or support source beneath its matching configured root' do
+    it 'places a test, module-under-test or support source by its path from the project root' do
       objects = @fpu.form_test_build_objects_filelist(
-        'build/test/out/TestFoo', ['drivers/foo/bar.c', 'drivers/baz/bar.c']
+        'build/test/out/TestFoo', ['test/TestFoo.c', 'drivers/foo/bar.c', 'drivers/baz/bar.c', 'support/qux.c']
       )
-      expect( objects ).to include('build/test/out/TestFoo/foo/bar.o')
-      expect( objects ).to include('build/test/out/TestFoo/baz/bar.o')
+      expect( objects ).to eq([
+        'build/test/out/TestFoo/test/TestFoo.o',
+        'build/test/out/TestFoo/drivers/foo/bar.o',
+        'build/test/out/TestFoo/drivers/baz/bar.o',
+        'build/test/out/TestFoo/support/qux.o'
+      ])
     end
 
-    it 'leaves the test file itself, mocks, and other unmatched inputs flat' do
+    # A source collected by one spelling and configured by another is still one source
+    it 'builds one file named by two spellings once, as no collision' do
       objects = @fpu.form_test_build_objects_filelist(
-        'build/test/out/TestFoo', ['test/TestFoo.c', 'MockBar.c', 'support/qux.c']
+        'build/test/out/TestFoo', ['test/support/helper.c', './test/support/helper.c', 'build/vendor/c_exception/lib/CException.c', './build/vendor/c_exception/lib/CException.c']
       )
-      expect( objects ).to eq([
-        'build/test/out/TestFoo/TestFoo.o',
-        'build/test/out/TestFoo/MockBar.o',
-        'build/test/out/TestFoo/qux.o'
-      ])
+      expect( objects ).to eq(['build/test/out/TestFoo/test/support/helper.o', 'build/test/out/TestFoo/CException.o'])
+    end
+
+    # A source far from the project gains a directory for each level its path climbs, which
+    # can reach Windows' path length limit
+    it 'checks the length of each object\'s whole path, once' do
+      expect(@file_wrapper).to receive(:check_path_length).with( 'build/test/out/TestFoo/_up_/_up_/far/src/x.o', origin: 'FilePathUtils#build_filepath' ).once
+
+      2.times { @fpu.form_test_build_objects_filelist( 'build/test/out/TestFoo', ['../../far/src/x.c'] ) }
+    end
+
+    it 'leaves a runner or vendored framework source below the build root flat' do
+      objects = @fpu.form_test_build_objects_filelist(
+        'build/test/out/TestFoo', ['build/test/runners/TestFoo_runner.c', 'build/vendor/unity/src/unity.c']
+      )
+      expect( objects ).to eq(['build/test/out/TestFoo/TestFoo_runner.o', 'build/test/out/TestFoo/unity.o'])
     end
 
     # Generated Partial sources sit below their own build root, not a configured source
@@ -739,7 +798,7 @@ describe FilePathUtils do
       ])
     end
 
-    it 'leaves inputs flat when no extra root is supplied' do
+    it 'leaves a generated source flat when its root is not supplied' do
       objects = @fpu.form_test_build_objects_filelist(
         'build/test/out/TestFoo',
         ['build/test/partials/TestFoo/drivers/uart/ceedling_partial_config_impl.c']
@@ -762,20 +821,82 @@ describe FilePathUtils do
       })
 
       allow(@configurator).to receive(:extension_testpass).and_return( FilenameExtension.new('.pass') )
-      allow(@configurator).to receive(:paths_test).and_return( ['test'] )
+      allow(@configurator).to receive(:project_build_root).and_return( 'build' )
+      allow(@file_wrapper).to receive(:check_path_length)
     end
 
-    it 'mirrors each expected results file beneath the same test identity its own build directory uses' do
+    it 'mirrors each expected results file beneath the same test name its own build directory uses' do
       results = @fpu.form_pass_results_filelist(
         'build/test/results', ['test/unit/test_foo.c', 'test/integration/test_foo.c']
       )
-      expect( results ).to include('build/test/results/unit/test_foo.pass')
-      expect( results ).to include('build/test/results/integration/test_foo.pass')
+      expect( results ).to include('build/test/results/test/unit/test_foo.pass')
+      expect( results ).to include('build/test/results/test/integration/test_foo.pass')
     end
 
-    it 'leaves a flat test\'s own results file flat, undisturbed by mirroring' do
+    it 'places a test at its root beneath that root, as its name does' do
       results = @fpu.form_pass_results_filelist('build/test/results', ['test/test_bar.c'])
-      expect( results ).to eq(['build/test/results/test_bar.pass'])
+      expect( results ).to eq(['build/test/results/test/test_bar.pass'])
+    end
+
+    # GH #1364: same-named tests at a root and below it, and under two roots
+    it 'gives same-named tests distinct results files' do
+      tests = ['test/test_calc.c', 'test/unit/test_calc.c', 'extra/unit/test_calc.c']
+      results = @fpu.form_pass_results_filelist( 'build/test/results', tests )
+      expect( results.uniq.size ).to eq(3)
+    end
+
+    it 'agrees with the results path each test is given' do
+      ['test/test_calc.c', 'test/unit/test_calc.c', '../shared/test/test_s.c'].each do |test|
+        name = @fpu.form_test_name( test )
+        expected = @fpu.form_pass_results_filepath( @fpu.form_test_results_path( name, context: 'test' ), test )
+        expect( @fpu.form_pass_results_filelist( 'build/test/results', [test] ) ).to eq([expected])
+      end
+    end
+  end
+
+  describe '#form_test_name' do
+    before(:each) do
+      @fpu = described_class.new({ :configurator => double('configurator'), :file_wrapper => double('file_wrapper') })
+    end
+
+    it 'names a test by its path from the project root, without its extension' do
+      expect( @fpu.form_test_name( 'test/test_calc.c' ) ).to eq('test/test_calc')
+      expect( @fpu.form_test_name( 'test/unit/test_calc.c' ) ).to eq('test/unit/test_calc')
+    end
+
+    # GH #1364: the same layout under two roots and a :files test outside every root
+    it 'keeps the root, so same-named tests under different roots get different names' do
+      names = ['test/foo/test_x.c', 'extra/foo/test_x.c', 'other/test_x.c'].map { |test| @fpu.form_test_name( test ) }
+      expect( names ).to eq(['test/foo/test_x', 'extra/foo/test_x', 'other/test_x'])
+    end
+
+    it 'gives one name however the path is written' do
+      expect( @fpu.form_test_name( './test/unit/test_calc.c' ) ).to eq('test/unit/test_calc')
+      expect( @fpu.form_test_name( 'test\\unit\\test_calc.c' ) ).to eq('test/unit/test_calc')
+      expect( @fpu.form_test_name( File.join( Dir.pwd, 'test/unit/test_calc.c' ) ) ).to eq('test/unit/test_calc')
+    end
+
+    it 'keeps a test outside the project inside the build directory' do
+      expect( @fpu.form_test_name( '../shared/test/unit/test_s.c' ) ).to eq('_up_/shared/test/unit/test_s')
+      expect( @fpu.form_test_name( '../../other/test_o.c' ) ).to eq('_up_/_up_/other/test_o')
+    end
+
+    it 'names an absolute path outside the project by its path from the project' do
+      outside = File.join( File.dirname( Dir.pwd ), 'shared', 'test', 'test_s.c' )
+      expect( @fpu.form_test_name( outside ) ).to eq('_up_/shared/test/test_s')
+    end
+
+    # A drive other than the working directory's, which on Windows may itself be C: or D:
+    it 'places a test on another Windows drive beneath its drive letter' do
+      drive = Dir.pwd.match?( /\A[Qq]:/ ) ? 'R' : 'Q'
+
+      expect( @fpu.form_test_name( "#{drive}:/work/test/test_x.c" ) ).to eq("_abs_/#{drive}/work/test/test_x")
+      expect( @fpu.form_test_name( "#{drive}:\\work\\test\\test_x.c" ) ).to eq("_abs_/#{drive}/work/test/test_x")
+    end
+
+    it 'names a test on the working directory\'s own Windows drive by its relative path', if: Dir.pwd.match?( /\A[A-Za-z]:/ ) do
+      inside = File.join( Dir.pwd, 'test', 'test_x.c' ).tr( '/', '\\' )
+      expect( @fpu.form_test_name( inside ) ).to eq('test/test_x')
     end
   end
 
@@ -790,19 +911,15 @@ describe FilePathUtils do
       })
 
       allow(@configurator).to receive(:project_test_preprocess_includes_path).and_return('build/test/preprocess/includes')
-      # The test file sits directly in its root, so nothing is mirrored here.
+      allow(@configurator).to receive(:project_build_root).and_return('build')
       allow(@configurator).to receive(:project_use_partials).and_return( false )
       allow(@configurator).to receive(:project_use_mocks).and_return( false )
-      allow(@configurator).to receive(:paths_source).and_return( [] )
-      allow(@configurator).to receive(:paths_support).and_return( [] )
-      allow(@configurator).to receive(:paths_include).and_return( [] )
-      allow(@configurator).to receive(:paths_test).and_return( ['test'] )
     end
 
     it 'uses the fixed internal .yml extension regardless of a project-configured :extension ↳ :yaml setting' do
       expect(@configurator).to_not receive(:extension_yaml)
       expect( @fpu.form_preprocessed_includes_list_filepath('test/TestFoo.c', 'TestFoo') )
-        .to eq('build/test/preprocess/includes/TestFoo/TestFoo.c.yml')
+        .to eq('build/test/preprocess/includes/TestFoo/test/TestFoo.c.yml')
     end
   end
 

data/spec/units/filesystem/file_system_wrapper_spec.rb

@@ -0,0 +1,33 @@
+# =========================================================================
+#   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_helper'
+require 'tmpdir'
+require 'fileutils'
+require 'ceedling/filesystem/file_system_wrapper'
+require 'ceedling/filesystem/working_dir'
+
+describe FileSystemWrapper do
+  around(:each) do |example|
+    Dir.mktmpdir do |dir|
+      @ws = File.realpath( dir )
+      FileUtils.mkdir_p( File.join( @ws, 'app/dep' ) )
+      Dir.chdir( File.join( @ws, 'app' ) ) { WorkingDirTracking.untracked { example.run } }
+    end
+  end
+
+  # A plugin can change directory through this object, and paths it resolves inside must
+  # resolve against the directory it changed to
+  it 'changes directory for the block as WorkingDir.chdir does' do
+    WorkingDir.path
+
+    inside = described_class.new.cd( 'dep' ) { WorkingDir.path }
+
+    expect(inside).to eq( File.join( @ws, 'app/dep' ) )
+    expect(WorkingDir.path).to eq( File.join( @ws, 'app' ) )
+  end
+end