(maint) Fix expect_upload for module relative paths outside files

lib/bolt_spec/plans/mock_executor.rb

@@ -49,7 +49,11 @@ def module_file_id(file)
         path = Pathname.new(file)
         relative = path.relative_path_from(Pathname.new(modpath.first))
         segments = relative.to_path.split('/')
-        ([segments[0]] + segments[2..-1]).join('/')
+        if segments[1] == 'files'
+          ([segments[0]] + segments[2..-1]).join('/')
+        else
+          segments.join('/')
+        end
       end
 
       def run_command(targets, command, options = {}, _position = [])

spec/bolt_spec/plan_spec.rb

@@ -207,6 +207,7 @@ def expect_action
       with_tempfile_containing('prep', '') do |file|
         @source = file.path
         allow_upload(@source).with_targets(targets)
+        expect_upload('plans/resources/bar').with_destination('/o')
         example.run
       end
     end

spec/bolt_spec/plans/mock_executor_spec.rb

@@ -11,4 +11,20 @@
 
     expect(missing_methods.empty?).to be(true), message
   end
+
+  context '#module_file_id' do
+    let(:executor) { BoltSpec::Plans::MockExecutor.new('/some/path/to/modules') }
+
+    it 'returns nil if path is outside of modulepath' do
+      expect(executor.module_file_id('/some/other/path')).to be_nil
+    end
+
+    it 'handles module relative paths relative to module/files returning module/path excluding the files dir' do
+      expect(executor.module_file_id('/some/path/to/modules/amodule/files/dingo')).to eq('amodule/dingo')
+    end
+
+    it 'handles module relative paths outside of module/files' do
+      expect(executor.module_file_id('/some/path/to/modules/amodule/files/../other/dingo')).to eq('amodule/other/dingo')
+    end
+  end
 end

spec/fixtures/bolt_spec/plans/plans/upload.pp

@@ -1,4 +1,5 @@
 plan plans::upload(TargetSpec $nodes, String $source) {
   upload_file($source, '/b', $nodes)
+  upload_file('plans/files/../resources/bar', '/o', $nodes)
   return upload_file('plans/script', '/d', $nodes)
 }

spec/fixtures/bolt_spec/plans/resources/bar

@@ -0,0 +1 @@
+Some other file outside of files/.