Refactoring towards RevisionDir, step 4

Author: beatmadsen

lib/files_in_my_diff/tmp_dir.rb

@@ -51,9 +51,11 @@ def create!
       def decorate(changes)
         raise DirectoryError, "Directory for #{@sha} not yet created" if @dir.nil?
 
-        # TODO
-        decorated = changes
-        { dir: @dir, sha: @sha, changes: decorated }
+        d_changes = changes.map do |change|
+          change => { path:, type: }
+          { full_path: File.join(@dir, path), relative_path: path, type: }
+        end
+        { dir: @dir, sha: @sha, changes: d_changes }
       end
     end
   end

test/tmp_dir/revision_dir_test.rb

@@ -27,6 +27,20 @@ def test_that_decorating_fails_if_dir_not_created
         assert_raises(TmpDir::DirectoryError) { subject.decorate([]) }
       end
 
+      def test_that_decorate_adds_full_paths_to_tmp_dir_for_each_change
+        subject = subject()
+        subject.create!
+        result = subject.decorate([{ path: 'a/b/c', type: :modified }, { path: 'd/e/f', type: :added }])
+
+        assert_equal(
+          { dir: 'some_tmpdir/files_in_my_diff/ab12cd34', sha: 'ab12cd34', changes: [
+            { full_path: 'some_tmpdir/files_in_my_diff/ab12cd34/a/b/c', relative_path: 'a/b/c', type: :modified },
+            { full_path: 'some_tmpdir/files_in_my_diff/ab12cd34/d/e/f', relative_path: 'd/e/f', type: :added },
+          ], },
+          result,
+        )
+      end
+
       private
 
       def subject(sha: 'ab12cd34', file_strategy: file_strategy_stub)