Add tests that specificly test read_tree, write_tree, and commit_tree git commands

jcouball · jcouball · commit 079893db4d43 · 2023-12-26T09:53:53.000-08:00
diff --git a/tests/units/test_tree_ops.rb b/tests/units/test_tree_ops.rb
@@ -5,6 +5,134 @@
 class TestTreeOps < Test::Unit::TestCase
 
   def test_read_tree
+    treeish = 'testbranch1'
+    expected_command_line = ['read-tree', treeish]
+    git_cmd = :read_tree
+    git_cmd_args = [treeish]
+    assert_command_line(expected_command_line, git_cmd, git_cmd_args)
+  end
+
+  def test_read_tree_with_prefix
+    treeish = 'testbranch1'
+    prefix = 'foo'
+    expected_command_line = ['read-tree', "--prefix=#{prefix}", treeish]
+    git_cmd = :read_tree
+    git_cmd_args = [treeish, prefix: prefix]
+    assert_command_line(expected_command_line, git_cmd, git_cmd_args)
+  end
+
+  def test_write_tree
+    expected_command_line = ['write-tree']
+    git_output = 'aa7349e'
+    git_cmd = :write_tree
+    git_cmd_args = []
+    result = assert_command_line(expected_command_line, git_cmd, git_cmd_args, git_output)
+    # the git output should be returned from Git::Base#write_tree
+    assert_equal(git_output, result)
+  end
+
+  def test_commit_tree_with_default_message
+    tree = 'tree-ref'
+
+    expected_message = 'commit tree tree-ref'
+    tempfile_path = 'foo'
+    mock_tempfile = mock('tempfile')
+    Tempfile.stubs(:new).returns(mock_tempfile)
+    mock_tempfile.stubs(:path).returns(tempfile_path)
+    mock_tempfile.expects(:write).with(expected_message)
+    mock_tempfile.expects(:close)
+
+    redirect_value = windows_platform? ? "< \"#{tempfile_path}\"" : "< '#{tempfile_path}'"
+
+    expected_command_line = ['commit-tree', tree, redirect: redirect_value]
+    git_cmd = :commit_tree
+    git_cmd_args = [tree]
+    assert_command_line(expected_command_line, git_cmd, git_cmd_args)
+  end
+
+  def test_commit_tree_with_message
+    tree = 'tree-ref'
+    message = 'this is my message'
+
+    tempfile_path = 'foo'
+    mock_tempfile = mock('tempfile')
+    Tempfile.stubs(:new).returns(mock_tempfile)
+    mock_tempfile.stubs(:path).returns(tempfile_path)
+    mock_tempfile.expects(:write).with(message)
+    mock_tempfile.expects(:close)
+
+    redirect_value = windows_platform? ? "< \"#{tempfile_path}\"" : "< '#{tempfile_path}'"
+
+    expected_command_line = ['commit-tree', tree, redirect: redirect_value]
+    git_cmd = :commit_tree
+    git_cmd_args = [tree, message: message]
+    assert_command_line(expected_command_line, git_cmd, git_cmd_args)
+  end
+
+  def test_commit_tree_with_parent
+    tree = 'tree-ref'
+    message = 'this is my message'
+    parent = 'parent-commit'
+
+    tempfile_path = 'foo'
+    mock_tempfile = mock('tempfile')
+    Tempfile.stubs(:new).returns(mock_tempfile)
+    mock_tempfile.stubs(:path).returns(tempfile_path)
+    mock_tempfile.expects(:write).with(message)
+    mock_tempfile.expects(:close)
+
+    redirect_value = windows_platform? ? "< \"#{tempfile_path}\"" : "< '#{tempfile_path}'"
+
+    expected_command_line = ['commit-tree', tree, "-p", parent, redirect: redirect_value]
+    git_cmd = :commit_tree
+    git_cmd_args = [tree, parent: parent, message: message]
+
+    assert_command_line(expected_command_line, git_cmd, git_cmd_args)
+  end
+
+  def test_commit_tree_with_parents
+    tree = 'tree-ref'
+    message = 'this is my message'
+    parents = 'commit1'
+
+    tempfile_path = 'foo'
+    mock_tempfile = mock('tempfile')
+    Tempfile.stubs(:new).returns(mock_tempfile)
+    mock_tempfile.stubs(:path).returns(tempfile_path)
+    mock_tempfile.expects(:write).with(message)
+    mock_tempfile.expects(:close)
+
+    redirect_value = windows_platform? ? "< \"#{tempfile_path}\"" : "< '#{tempfile_path}'"
+
+    expected_command_line = ['commit-tree', tree, '-p', 'commit1', redirect: redirect_value]
+    git_cmd = :commit_tree
+    git_cmd_args = [tree, parents: parents, message: message]
+
+    assert_command_line(expected_command_line, git_cmd, git_cmd_args)
+  end
+
+  def test_commit_tree_with_multiple_parents
+    tree = 'tree-ref'
+    message = 'this is my message'
+    parents = ['commit1', 'commit2']
+
+    tempfile_path = 'foo'
+    mock_tempfile = mock('tempfile')
+    Tempfile.stubs(:new).returns(mock_tempfile)
+    mock_tempfile.stubs(:path).returns(tempfile_path)
+    mock_tempfile.expects(:write).with(message)
+    mock_tempfile.expects(:close)
+
+    redirect_value = windows_platform? ? "< \"#{tempfile_path}\"" : "< '#{tempfile_path}'"
+
+    expected_command_line = ['commit-tree', tree, '-p', 'commit1', '-p', 'commit2', redirect: redirect_value]
+    git_cmd = :commit_tree
+    git_cmd_args = [tree, parents: parents, message: message]
+
+    assert_command_line(expected_command_line, git_cmd, git_cmd_args)
+  end
+
+  def test_tree_ops
     in_bare_repo_clone do |g|
       g.branch('testbranch1').in_branch('tb commit 1') do
         new_file('test-file1', 'blahblahblah2')
@@ -79,12 +207,7 @@ def test_read_tree
       assert(c.commit?)
       assert_equal('b40f7a9072cdec637725700668f8fdebe39e6d38', c.gtree.sha)
 
-      tmp = Tempfile.new('tesxt')
-      tmppath = tmp.path
-      tmp.close
-      tmp.unlink
-
-      g.with_index(tmppath) do
+      g.with_temp_index do
         g.read_tree('testbranch1', :prefix => 'b1/')
         g.read_tree('testbranch3', :prefix => 'b3/')
         index = g.ls_files
@@ -111,7 +234,6 @@ def test_read_tree
         g.checkout_index(:all => true)
         assert(File.directory?('b1'))
       end
-
     end
   end
 end
