kintsugi 0.5.2 → 0.5.3

data/lib/kintsugi/merge.rb

@@ -0,0 +1,146 @@
+# Copyright (c) 2021 Lightricks. All rights reserved.
+# Created by Ben Yohay.
+# frozen_string_literal: true
+
+require "json"
+require "tmpdir"
+require "tempfile"
+require "xcodeproj"
+
+require_relative "xcodeproj_extensions"
+require_relative "apply_change_to_project"
+require_relative "error"
+
+module Kintsugi
+  class << self
+    # Resolves git conflicts of a pbxproj file specified by `project_file_path`.
+    #
+    # @param  [String] project_file_path
+    #         Project to which to apply the changes.
+    #
+    # @param  [String] changes_output_path
+    #         Path to where the changes to apply to the project are written in JSON format.
+    #
+    # @raise [ArgumentError]
+    #        If the file extension is not `pbxproj`, or the file doesn't exist, or if no rebase,
+    #        cherry-pick, or merge is in progress
+    #
+    # @raise [MergeError]
+    #        If there was an error applying the change to the project.
+    #
+    # @return [void]
+    def resolve_conflicts(project_file_path, changes_output_path)
+      validate_project(project_file_path)
+
+      base_project = copy_project_from_stage_number_to_temporary_directory(project_file_path, 1)
+      ours_project = copy_project_from_stage_number_to_temporary_directory(project_file_path, 2)
+      theirs_project = copy_project_from_stage_number_to_temporary_directory(project_file_path, 3)
+
+      change = Xcodeproj::Differ.project_diff(theirs_project, base_project, :added, :removed)
+
+      if changes_output_path
+        File.write(changes_output_path, JSON.pretty_generate(change))
+      end
+
+      apply_change_and_copy_to_original_path(ours_project, change, project_file_path)
+    end
+
+    # Merges the changes done between `theirs_project_path` and `base_project_path` to the file at
+    # `ours_project_path`. The files may not be at the original path, and therefore the
+    # `original_project_path` is required in order for the project metadata to be written properly.
+    #
+    # @param  [String] base_project_path
+    #         Path to the base version of the project.
+    #
+    # @param  [String] ours_project_path
+    #         Path to ours version of the project.
+    #
+    # @param  [String] theirs_project_path
+    #         Path to theirs version of the project.
+    #
+    # @param  [String] original_project_path
+    #         Path to the original path of the file.
+    #
+    # @raise [MergeError]
+    #        If there was an error applying the change to the project.
+    #
+    # @return [void]
+    def three_way_merge(base_project_path, ours_project_path, theirs_project_path,
+                        original_project_path)
+      original_directory_name = File.basename(File.dirname(original_project_path))
+      base_temporary_project =
+        copy_project_to_temporary_path_in_directory_with_name(base_project_path,
+                                                              original_directory_name)
+      ours_temporary_project =
+        copy_project_to_temporary_path_in_directory_with_name(ours_project_path,
+                                                              original_directory_name)
+      theirs_temporary_project =
+        copy_project_to_temporary_path_in_directory_with_name(theirs_project_path,
+                                                              original_directory_name)
+
+      change =
+        Xcodeproj::Differ.project_diff(theirs_temporary_project, base_temporary_project,
+                                       :added, :removed)
+
+      apply_change_and_copy_to_original_path(ours_temporary_project, change, ours_project_path)
+    end
+
+    private
+
+    PROJECT_FILE_NAME = "project.pbxproj"
+
+    def apply_change_and_copy_to_original_path(project, change, original_project_file_path)
+      apply_change_to_project(project, change)
+      project.save
+      FileUtils.cp(File.join(project.path, PROJECT_FILE_NAME), original_project_file_path)
+    end
+
+    def validate_project(project_file_path)
+      unless File.exist?(project_file_path)
+        raise ArgumentError, "File '#{project_file_path}' doesn't exist"
+      end
+
+      if File.extname(project_file_path) != ".pbxproj"
+        raise ArgumentError, "Wrong file extension, please provide file with extension .pbxproj\""
+      end
+
+      unless file_has_base_ours_and_theirs_versions?(project_file_path)
+        raise ArgumentError, "File '#{project_file_path}' doesn't have conflicts, " \
+          "or a 3-way merge is not possible."
+      end
+    end
+
+    def copy_project_from_stage_number_to_temporary_directory(project_file_path, stage_number)
+      project_directory_name = File.basename(File.dirname(project_file_path))
+      temp_project_file_path = File.join(Dir.mktmpdir, project_directory_name, PROJECT_FILE_NAME)
+      Dir.mkdir(File.dirname(temp_project_file_path))
+      Dir.chdir(File.dirname(project_file_path)) do
+        `git show :#{stage_number}:./#{PROJECT_FILE_NAME} > "#{temp_project_file_path}"`
+      end
+      Xcodeproj::Project.open(File.dirname(temp_project_file_path))
+    end
+
+    def copy_project_to_temporary_path_in_directory_with_name(project_file_path, directory_name)
+      temp_directory_name = File.join(Dir.mktmpdir, directory_name)
+      Dir.mkdir(temp_directory_name)
+      temp_project_file_path = File.join(temp_directory_name, PROJECT_FILE_NAME)
+      FileUtils.cp(project_file_path, temp_project_file_path)
+      Xcodeproj::Project.open(File.dirname(temp_project_file_path))
+    end
+
+    def file_has_base_ours_and_theirs_versions?(file_path)
+      Dir.chdir(`git -C "#{File.dirname(file_path)}" rev-parse --show-toplevel`.strip) do
+        file_has_version_in_stage_numbers?(file_path, [1, 2, 3])
+      end
+    end
+
+    def file_has_version_in_stage_numbers?(file_path, stage_numbers)
+      file_absolute_path = File.absolute_path(file_path)
+      actual_stage_numbers =
+        `git ls-files -u -- "#{file_absolute_path}"`.split("\n").map do |git_file_status|
+          git_file_status.split[2]
+        end
+      (stage_numbers - actual_stage_numbers.map(&:to_i)).empty?
+    end
+  end
+end

data/lib/kintsugi/version.rb

@@ -3,6 +3,6 @@
 module Kintsugi
   # This module holds the Kintsugi version information.
   module Version
-    STRING = "0.5.2"
+    STRING = "0.5.3"
   end
 end

data/lib/kintsugi/xcodeproj_extensions.rb

@@ -66,4 +66,88 @@ module Xcodeproj
       end
     end
   end
+
+  module Differ
+      # Replaces the implementation of `array_diff` with an implementation that takes into account
+      # the number of occurrences an element is found in the array.
+      # Code was mostly copied from https://github.com/CocoaPods/Xcodeproj/blob/51fb78a03f31614103815ce21c56dc25c044a10d/lib/xcodeproj/differ.rb#L111
+      def self.array_diff(value_1, value_2, options)
+      ensure_class(value_1, Array)
+      ensure_class(value_2, Array)
+      return nil if value_1 == value_2
+
+      new_objects_value_1 = array_non_unique_diff(value_1, value_2)
+      new_objects_value_2 = array_non_unique_diff(value_2, value_1)
+      return nil if value_1.empty? && value_2.empty?
+
+      matched_diff = {}
+      if id_key = options[:id_key]
+        matched_value_1 = []
+        matched_value_2 = []
+        new_objects_value_1.each do |entry_value_1|
+          if entry_value_1.is_a?(Hash)
+            id_value = entry_value_1[id_key]
+            entry_value_2 = new_objects_value_2.find do |entry|
+              entry[id_key] == id_value
+            end
+            if entry_value_2
+              matched_value_1 << entry_value_1
+              matched_value_2 << entry_value_2
+              diff = diff(entry_value_1, entry_value_2, options)
+              matched_diff[id_value] = diff if diff
+            end
+          end
+        end
+
+        new_objects_value_1 -= matched_value_1
+        new_objects_value_2 -= matched_value_2
+      end
+
+      if new_objects_value_1.empty? && new_objects_value_2.empty?
+        if matched_diff.empty?
+          nil
+        else
+          matched_diff
+        end
+      else
+        result = {}
+        result[options[:key_1]] = new_objects_value_1 unless new_objects_value_1.empty?
+        result[options[:key_2]] = new_objects_value_2 unless new_objects_value_2.empty?
+        result[:diff] = matched_diff unless matched_diff.empty?
+        result
+      end
+    end
+
+    # Returns the difference between two arrays, taking into account the number of occurrences an
+    # element is found in both arrays.
+    #
+    # @param  [Array] value_1
+    #         First array to the difference operation.
+    #
+    # @param  [Array] value_2
+    #         Second array to the difference operation.
+    #
+    # @return [Array]
+    #
+    def self.array_non_unique_diff(value_1, value_2)
+      value_2_elements_by_count = value_2.reduce({}) do |hash, element|
+        updated_element_hash = hash.key?(element) ? {element => hash[element] + 1} : {element => 1}
+        hash.merge(updated_element_hash)
+      end
+
+      value_1_elements_by_deletions =
+        value_1.to_set.map do |element|
+          times_to_delete_element = value_2_elements_by_count[element] || 0
+          [element, times_to_delete_element]
+        end.to_h
+
+      value_1.select do |element|
+        if value_1_elements_by_deletions[element].positive?
+          value_1_elements_by_deletions[element] -= 1
+          next false
+        end
+        next true
+      end
+    end
+  end
 end

data/lib/kintsugi.rb

@@ -1,167 +1,48 @@
-# Copyright (c) 2021 Lightricks. All rights reserved.
-# Created by Ben Yohay.
 # frozen_string_literal: true
 
-require "json"
-require "tmpdir"
-require "tempfile"
-require "xcodeproj"
+# Copyright (c) 2022 Lightricks. All rights reserved.
+# Created by Ben Yohay.
 
-require_relative "kintsugi/xcodeproj_extensions"
-require_relative "kintsugi/apply_change_to_project"
+require_relative "kintsugi/cli"
 require_relative "kintsugi/error"
+require_relative "kintsugi/merge"
 
 module Kintsugi
   class << self
-    # Resolves git conflicts of a pbxproj file specified by `project_file_path`.
-    #
-    # @param  [String] project_file_path
-    #         Project to which to apply the changes.
-    #
-    # @param  [String] changes_output_path
-    #         Path to where the changes to apply to the project are written in JSON format.
-    #
-    # @raise [ArgumentError]
-    #        If the file extension is not `pbxproj`, or the file doesn't exist, or if no rebase,
-    #        cherry-pick, or merge is in progress
-    #
-    # @raise [MergeError]
-    #        If there was an error applying the change to the project.
-    #
-    # @return [void]
-    def resolve_conflicts(project_file_path, changes_output_path)
-      validate_project(project_file_path)
-
-      project_in_temp_directory =
-        open_project_of_current_commit_in_temporary_directory(project_file_path)
-
-      change = change_of_conflicting_commit_with_parent(project_file_path)
-
-      if changes_output_path
-        File.write(changes_output_path, JSON.pretty_generate(change))
-      end
-
-      apply_change_and_copy_to_original_path(project_in_temp_directory, change, project_file_path)
-    end
-
-    # Merges the changes done between `theirs_project_path` and `base_project_path` to the file at
-    # `ours_project_path`. The files may not be at the original path, and therefore the
-    # `original_project_path` is required in order for the project metadata to be written properly.
-    #
-    # @param  [String] base_project_path
-    #         Path to the base version of the project.
-    #
-    # @param  [String] ours_project_path
-    #         Path to ours version of the project.
-    #
-    # @param  [String] theirs_project_path
-    #         Path to theirs version of the project.
-    #
-    # @param  [String] original_project_path
-    #         Path to the original path of the file.
-    #
-    # @raise [MergeError]
-    #        If there was an error applying the change to the project.
-    #
-    # @return [void]
-    def three_way_merge(base_project_path, ours_project_path, theirs_project_path,
-                        original_project_path)
-      original_directory_name = File.basename(File.dirname(original_project_path))
-      base_temporary_project =
-        copy_project_to_temporary_path_in_directory_with_name(base_project_path,
-                                                              original_directory_name)
-      ours_temporary_project =
-        copy_project_to_temporary_path_in_directory_with_name(ours_project_path,
-                                                              original_directory_name)
-      theirs_temporary_project =
-        copy_project_to_temporary_path_in_directory_with_name(theirs_project_path,
-                                                              original_directory_name)
-
-      change =
-        Xcodeproj::Differ.project_diff(theirs_temporary_project, base_temporary_project,
-                                       :added, :removed)
-
-      apply_change_and_copy_to_original_path(ours_temporary_project, change, ours_project_path)
-    end
-
-    private
-
-    PROJECT_FILE_NAME = "project.pbxproj"
-
-    def apply_change_and_copy_to_original_path(project, change, original_project_file_path)
-      apply_change_to_project(project, change)
-      project.save
-      FileUtils.cp(File.join(project.path, PROJECT_FILE_NAME), original_project_file_path)
-    end
-
-    def validate_project(project_file_path)
-      unless File.exist?(project_file_path)
-        raise ArgumentError, "File '#{project_file_path}' doesn't exist"
-      end
-
-      if File.extname(project_file_path) != ".pbxproj"
-        raise ArgumentError, "Wrong file extension, please provide file with extension .pbxproj\""
-      end
-
-      Dir.chdir(File.dirname(project_file_path)) do
-        unless file_has_base_ours_and_theirs_versions?(project_file_path)
-          raise ArgumentError, "File '#{project_file_path}' doesn't have conflicts, " \
-            "or a 3-way merge is not possible."
+    def run(arguments)
+      first_argument = arguments[0]
+      cli = CLI.new
+      command =
+        if name_of_subcommand?(cli.subcommands, first_argument)
+          arguments.shift
+          cli.subcommands[first_argument]
+        else
+          cli.root_command
         end
-      end
-    end
 
-    def copy_project_to_temporary_path_in_directory_with_name(project_file_path, directory_name)
-      temp_directory_name = File.join(Dir.mktmpdir, directory_name)
-      Dir.mkdir(temp_directory_name)
-      temp_project_file_path = File.join(temp_directory_name, PROJECT_FILE_NAME)
-      FileUtils.cp(project_file_path, temp_project_file_path)
-      Xcodeproj::Project.open(File.dirname(temp_project_file_path))
-    end
+      options = parse_options!(command, arguments)
 
-    def open_project_of_current_commit_in_temporary_directory(project_file_path)
-      project_directory_name = File.basename(File.dirname(project_file_path))
-      temp_directory_name = File.join(Dir.mktmpdir, project_directory_name)
-      Dir.mkdir(temp_directory_name)
-      temp_project_file_path = File.join(temp_directory_name, PROJECT_FILE_NAME)
-      Dir.chdir(File.dirname(project_file_path)) do
-        `git show HEAD:./project.pbxproj > "#{temp_project_file_path}"`
+      begin
+        command.action.call(options, arguments)
+      rescue ArgumentError => e
+        puts "#{e.class}: #{e}"
+        raise
+      rescue Kintsugi::MergeError => e
+        puts e
+        raise
       end
-      Xcodeproj::Project.open(File.dirname(temp_project_file_path))
     end
 
-    def file_has_base_ours_and_theirs_versions?(file_path)
-      Dir.chdir(`git rev-parse --show-toplevel`.strip) do
-        file_has_version_in_stage_numbers?(file_path, [1, 2, 3])
-      end
-    end
+    private
 
-    def file_has_version_in_stage_numbers?(file_path, stage_numbers)
-      file_absolute_path = File.absolute_path(file_path)
-      actual_stage_numbers =
-        `git ls-files -u -- "#{file_absolute_path}"`.split("\n").map do |git_file_status|
-          git_file_status.split[2]
-        end
-      (stage_numbers - actual_stage_numbers.map(&:to_i)).empty?
+    def parse_options!(command, arguments)
+      options = {}
+      command.option_parser.parse!(arguments, into: options)
+      options
     end
 
-    def change_of_conflicting_commit_with_parent(project_file_path)
-      Dir.chdir(File.dirname(project_file_path)) do
-        conflicting_commit_project_file_path = File.join(Dir.mktmpdir, PROJECT_FILE_NAME)
-        `git show :3:./#{PROJECT_FILE_NAME} > #{conflicting_commit_project_file_path}`
-
-        conflicting_commit_parent_project_file_path = File.join(Dir.mktmpdir, PROJECT_FILE_NAME)
-        `git show :1:./#{PROJECT_FILE_NAME} > #{conflicting_commit_parent_project_file_path}`
-
-        conflicting_commit_project = Xcodeproj::Project.open(
-          File.dirname(conflicting_commit_project_file_path)
-        )
-        conflicting_commit_parent_project =
-          Xcodeproj::Project.open(File.dirname(conflicting_commit_parent_project_file_path))
-
-        Xcodeproj::Differ.project_diff(conflicting_commit_project,
-                                       conflicting_commit_parent_project, :added, :removed)
-      end
+    def name_of_subcommand?(subcommands, argument)
+      subcommands.include?(argument)
     end
   end
 end

data/spec/kintsugi_apply_change_to_project_spec.rb

@@ -427,7 +427,8 @@ describe Kintsugi, :apply_change_to_project do
 
       theirs_project = create_copy_of_project(base_project.path, "theirs")
 
-      file_reference = theirs_project.main_group.new_reference(framework_filename)
+      file_reference = theirs_project.main_group.new_reference("bar")
+      file_reference.name = framework_filename
       build_phase = theirs_project.targets[0].frameworks_build_phase
       build_phase.files[-1].remove_from_project
       theirs_project.targets[0].frameworks_build_phase.add_file_reference(file_reference)
@@ -435,6 +436,10 @@ describe Kintsugi, :apply_change_to_project do
       changes_to_apply = get_diff(theirs_project, base_project)
 
       described_class.apply_change_to_project(base_project, changes_to_apply)
+      # This verifies we haven't created a new file reference instead of reusing the one in the
+      # hierarchy.
+      base_project.files[-1].name = "foo"
+      theirs_project.files[-1].name = "foo"
       base_project.save
 
       expect(base_project).to be_equivalent_to_project(theirs_project)
@@ -531,7 +536,7 @@ describe Kintsugi, :apply_change_to_project do
       expect(base_project).to be_equivalent_to_project(theirs_project, ignore_keys: ["containerPortal"])
     end
 
-    it "adds build file to a file reference that already exist" do
+    it "adds build file to a file reference that already exists" do
       base_project.main_group.new_reference("bar")
 
       base_project.main_group.new_reference("bar")
@@ -1062,6 +1067,20 @@ describe Kintsugi, :apply_change_to_project do
       expect(base_project).to be_equivalent_to_project(theirs_project)
     end
 
+    it "adds build phase with a simple attribute value that has non nil default" do
+      theirs_project = create_copy_of_project(base_project.path, "theirs")
+
+      theirs_project.targets[0].new_shell_script_build_phase("bar")
+      theirs_project.targets[0].build_phases.last.shell_script = "Other value"
+
+      changes_to_apply = get_diff(theirs_project, base_project)
+
+      described_class.apply_change_to_project(base_project, changes_to_apply)
+      base_project.save
+
+      expect(base_project).to be_equivalent_to_project(theirs_project)
+    end
+
     it "removes build phase" do
       base_project.targets[0].new_shell_script_build_phase("bar")
 
@@ -1346,6 +1365,41 @@ describe Kintsugi, :apply_change_to_project do
     expect(base_project).to be_equivalent_to_project(theirs_project)
   end
 
+  it "adds group to product group" do
+    base_project_path = make_temp_directory("base", ".xcodeproj")
+    base_project = Xcodeproj::Project.new(base_project_path)
+    base_project.new_target("com.apple.product-type.library.static", "foo", :ios)
+
+    base_project.save
+    theirs_project = create_copy_of_project(base_project.path, "theirs")
+
+    theirs_project.root_object.product_ref_group.new_group("foo")
+
+    changes_to_apply = get_diff(theirs_project, base_project)
+
+    described_class.apply_change_to_project(base_project, changes_to_apply)
+
+    expect(base_project).to be_equivalent_to_project(theirs_project)
+  end
+
+  it "adds localization files to product group" do
+    base_project_path = make_temp_directory("base", ".xcodeproj")
+    base_project = Xcodeproj::Project.new(base_project_path)
+    base_project.new_target("com.apple.product-type.library.static", "foo", :ios)
+
+    base_project.save
+    theirs_project = create_copy_of_project(base_project.path, "theirs")
+
+    variant_group = theirs_project.root_object.product_ref_group.new_variant_group("foo.strings")
+    variant_group.new_reference("Base").last_known_file_type = "text.plist.strings"
+
+    changes_to_apply = get_diff(theirs_project, base_project)
+
+    described_class.apply_change_to_project(base_project, changes_to_apply)
+
+    expect(base_project).to be_equivalent_to_project(theirs_project)
+  end
+
   def create_copy_of_project(project_path, new_project_prefix)
     copied_project_path = make_temp_directory(new_project_prefix, ".xcodeproj")
     FileUtils.cp(File.join(project_path, "project.pbxproj"), copied_project_path)

data/spec/kintsugi_integration_spec.rb

@@ -0,0 +1,148 @@
+# Copyright (c) 2021 Lightricks. All rights reserved.
+# Created by Ben Yohay.
+# frozen_string_literal: true
+
+require "git"
+require "json"
+require "rspec"
+require "tempfile"
+require "tmpdir"
+
+require "kintsugi"
+
+shared_examples "tests" do |git_command, project_name|
+  let(:temporary_directories_paths) { [] }
+  let(:git_directory_path) { make_temp_directory }
+  let(:git) { Git.init(git_directory_path) }
+
+  before do
+    git.config("user.email", "you@example.com")
+    git.config("user.name", "Your Name")
+  end
+
+  after do
+    temporary_directories_paths.each do |directory_path|
+      FileUtils.remove_entry(directory_path)
+    end
+  end
+
+  context "running 'git #{git_command}' with project name '#{project_name}'" do
+    it "resolves conflicts with root command" do
+      File.write(File.join(git_directory_path, ".gitattributes"), "*.pbxproj merge=Unset")
+
+      project = create_new_project_at_path(File.join(git_directory_path, project_name))
+
+      git.add(File.join(git_directory_path, ".gitattributes"))
+      git.add(project.path)
+      git.commit("Initial project")
+
+      project.new_target("com.apple.product-type.library.static", "foo", :ios)
+      project.save
+
+      git.add(all: true)
+      git.commit("Add target foo")
+      first_commit_hash = git.revparse("HEAD")
+
+      git.checkout("HEAD^")
+      project = Xcodeproj::Project.open(project.path)
+      project.new_target("com.apple.product-type.library.static", "bar", :ios)
+      project.save
+      git.add(all: true)
+      git.commit("Add target bar")
+
+      `git -C #{git_directory_path} #{git_command} #{first_commit_hash} &> /dev/null`
+      Kintsugi.run([File.join(project.path, "project.pbxproj")])
+
+      project = Xcodeproj::Project.open(project.path)
+      expect(project.targets.map(&:display_name)).to contain_exactly("foo", "bar")
+    end
+
+    it "resolves conflicts automatically with driver" do
+      git.config("merge.kintsugi.name", "Kintsugi driver")
+      git.config("merge.kintsugi.driver", "#{__dir__}/../bin/kintsugi driver %O %A %B %P")
+      File.write(File.join(git_directory_path, ".gitattributes"), "*.pbxproj merge=kintsugi")
+
+      project = create_new_project_at_path(File.join(git_directory_path, project_name))
+
+      git.add(File.join(git_directory_path, ".gitattributes"))
+      git.add(project.path)
+      git.commit("Initial project")
+
+      project.new_target("com.apple.product-type.library.static", "foo", :ios)
+      project.save
+
+      git.add(all: true)
+      git.commit("Add target foo")
+      first_commit_hash = git.revparse("HEAD")
+
+      git.checkout("HEAD^")
+      project = Xcodeproj::Project.open(project.path)
+      project.new_target("com.apple.product-type.library.static", "bar", :ios)
+      project.save
+      git.add(all: true)
+      git.commit("Add target bar")
+
+      `git -C #{git_directory_path} #{git_command} #{first_commit_hash} &> /dev/null`
+
+      project = Xcodeproj::Project.open(project.path)
+      expect(project.targets.map(&:display_name)).to contain_exactly("foo", "bar")
+    end
+
+    it "keeps conflicts if failed to resolve conflicts" do
+      File.write(File.join(git_directory_path, ".gitattributes"), "*.pbxproj merge=Unset")
+
+      project = create_new_project_at_path(File.join(git_directory_path, project_name))
+      project.new_target("com.apple.product-type.library.static", "foo", :ios)
+      project.save
+
+      git.add(File.join(git_directory_path, ".gitattributes"))
+      git.add(project.path)
+      git.commit("Initial project")
+
+      project.targets[0].build_configurations.each do |configuration|
+        configuration.build_settings["PRODUCT_NAME"] = "bar"
+      end
+      project.save
+      git.add(all: true)
+      git.commit("Change target product name to bar")
+      first_commit_hash = git.revparse("HEAD")
+
+      git.checkout("HEAD^")
+      project = Xcodeproj::Project.open(project.path)
+      project.targets[0].build_configurations.each do |configuration|
+        configuration.build_settings["PRODUCT_NAME"] = "baz"
+      end
+      project.save
+      git.add(all: true)
+      git.commit("Change target product name to baz")
+
+      `git -C #{git_directory_path} #{git_command} #{first_commit_hash} &> /dev/null`
+
+      expect {
+        Kintsugi.run([File.join(project.path, "project.pbxproj")])
+      }.to raise_error(Kintsugi::MergeError)
+      expect(`git -C #{git_directory_path} diff --name-only --diff-filter=U`.chomp)
+        .to eq("#{project_name}/project.pbxproj")
+    end
+  end
+
+  def make_temp_directory
+    directory_path = Dir.mktmpdir
+    temporary_directories_paths << directory_path
+    directory_path
+  end
+end
+
+def create_new_project_at_path(path)
+  project = Xcodeproj::Project.new(path)
+  project.save
+  project
+end
+
+describe Kintsugi, :kintsugi do
+  %w[rebase cherry-pick merge].each do |git_command|
+    ["foo.xcodeproj", "foo with space.xcodeproj"].each do |project_name|
+      it_behaves_like("tests", git_command, project_name)
+    end
+  end
+end