kintsugi 0.6.2 → 0.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3cad34bb224a6760f052ff90ac433095e7fb071e7b59ca9018719e3c6a8f5c96
-  data.tar.gz: a0b9675697896041b08d60186804e3c112e51c0dd61d66d26ce9f9310c626f4e
+  metadata.gz: f9f20babc40a6ddf7228d2455e23d165a2d696929f5507fc8de0a9878353280f
+  data.tar.gz: 8ffd34dd7f504193c0a35d1347fa92c6edde6976602b5e62658ba9a637b6bc81
 SHA512:
-  metadata.gz: 359e1799a2f313daf547645342677eca83f58a76dc857d2f72b6f97b06318455385cec7b0145344c380ec659376ff3228add2f6c91499f1bef1cd20d63beae11
-  data.tar.gz: 04c9e862353bef092c14464de1e781c3d2168ad958fb8c5d08ed53f74e4090921023f674e6759eff997fbe2994af9514a0d546de4a8d652b19e5a37b32ce3970
+  metadata.gz: d8477aaafe2a4afed13fce37f5a5963b79ab2d3e26e4816abf4cd7b082b5a9a09cd189260713bbd5f82eaf9f8f8227b92bcd5dc79ad90acf1c0cc5d4ee9e8971
+  data.tar.gz: d63cf3848b727c894a9c294b75023c46e12b5672f03144bb7068477edc7765372b2e5e586c03bc69560f71a8145b7c2b3074a5246a4726a965b98a87050a59cb

data/lib/kintsugi/apply_change_to_project.rb

@@ -103,6 +103,12 @@ module Kintsugi
         group_type = Module.const_get("Xcodeproj::Project::#{change["isa"]}")
         containing_group = path.empty? ? project.main_group : project[path]
 
+        if containing_group.nil?
+          raise MergeError, "Trying to add or move a group with change #{change} to a group that " \
+            "no longer exists with path '#{path}'. This is considered a conflict that should be " \
+            "resolved manually."
+        end
+
         next if !Settings.allow_duplicates &&
           !find_group_in_group(containing_group, group_type, change).nil?
 
@@ -144,6 +150,12 @@ module Kintsugi
         containing_group = path.empty? ? project.main_group : project[path]
         change_key = file_reference_key(change)
 
+        if containing_group.nil?
+          raise MergeError, "Trying to add or move a file with change #{change} to a group that " \
+            "no longer exists with path '#{path}'. This is considered a conflict that should be " \
+            "resolved manually."
+        end
+
         if (removal_keys_to_references[change_key] || []).empty?
           apply_file_addition(containing_group, change, "rootObject/mainGroup/#{path}")
         elsif addition_keys_to_paths[change_key].length == 1 &&

data/lib/kintsugi/version.rb

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

data/spec/kintsugi_apply_change_to_project_spec.rb

@@ -159,6 +159,59 @@ describe Kintsugi, :apply_change_to_project do
       expect(base_project).to be_equivalent_to_project(theirs_project)
     end
 
+    it "raises if trying to move file to another group that no longer exists" do
+      base_project.main_group.find_subpath("new_group", true)
+      base_project.save
+
+      theirs_project = create_copy_of_project(base_project.path, "theirs")
+      new_group = theirs_project.main_group.find_subpath("new_group")
+      file_reference = theirs_project.main_group.find_file_by_path(filepath)
+      file_reference.move(new_group)
+
+      changes_to_apply = get_diff(theirs_project, base_project)
+
+      base_project.main_group.find_subpath("new_group").remove_from_project
+
+      expect {
+        described_class.apply_change_to_project(base_project, changes_to_apply)
+      }.to raise_error(Kintsugi::MergeError)
+    end
+
+    it "raises if trying to add file to a group that no longer exists" do
+      base_project.main_group.find_subpath("new_group", true)
+      base_project.save
+
+      theirs_project = create_copy_of_project(base_project.path, "theirs")
+      theirs_project.main_group.find_subpath("new_group").new_reference("foo")
+
+      changes_to_apply = get_diff(theirs_project, base_project)
+
+      base_project.main_group.find_subpath("new_group").remove_from_project
+
+      expect {
+        described_class.apply_change_to_project(base_project, changes_to_apply)
+      }.to raise_error(Kintsugi::MergeError)
+    end
+
+    it "does nothing if trying to remove a file from a group that no longer exists" do
+      base_project.main_group.find_subpath("new_group", true).new_reference("foo")
+      base_project.save
+
+      theirs_project = create_copy_of_project(base_project.path, "theirs")
+      theirs_project.main_group.find_subpath("new_group/foo").remove_from_project
+
+      changes_to_apply = get_diff(theirs_project, base_project)
+
+      base_project.main_group.find_subpath("new_group").remove_from_project
+
+      base_project.save
+      expected_project = create_copy_of_project(base_project.path, "expected")
+
+      described_class.apply_change_to_project(base_project, changes_to_apply)
+
+      expect(base_project).to be_equivalent_to_project(expected_project)
+    end
+
     it "raises when a file is split into two" do
       base_project.main_group.find_subpath("new_group", true)
       base_project.main_group.find_subpath("new_group2", true)
@@ -219,6 +272,39 @@ describe Kintsugi, :apply_change_to_project do
       expect(base_project).to be_equivalent_to_project(theirs_project)
     end
 
+    it "raises when trying to add a group to a group that no longer exists" do
+      base_project.main_group.find_subpath("new_group", true)
+      base_project.save
+
+      theirs_project = create_copy_of_project(base_project.path, "theirs")
+      theirs_project["new_group"].find_subpath("sub_group", true)
+
+      changes_to_apply = get_diff(theirs_project, base_project)
+
+      base_project.main_group.find_subpath("new_group").remove_from_project
+
+      expect {
+        described_class.apply_change_to_project(base_project, changes_to_apply)
+      }.to raise_error(Kintsugi::MergeError)
+    end
+
+    it "raises when trying to move a group to a group that no longer exists" do
+      base_project.main_group.find_subpath("new_group", true)
+      base_project.main_group.find_subpath("other_group", true)
+      base_project.save
+
+      theirs_project = create_copy_of_project(base_project.path, "theirs")
+      theirs_project["other_group"].move(theirs_project["new_group"])
+
+      changes_to_apply = get_diff(theirs_project, base_project)
+
+      base_project.main_group.find_subpath("new_group").remove_from_project
+
+      expect {
+        described_class.apply_change_to_project(base_project, changes_to_apply)
+      }.to raise_error(Kintsugi::MergeError)
+    end
+
     it "moves a group with a group in it" do
       new_group = base_project.main_group.find_subpath("new_group", true)
       new_group.find_subpath("sub_group", true)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kintsugi
 version: !ruby/object:Gem::Version
-  version: 0.6.2
+  version: 0.6.3
 platform: ruby
 authors:
 - Ben Yohay
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-09-12 00:00:00.000000000 Z
+date: 2022-11-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: xcodeproj
@@ -168,7 +168,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.22
+rubygems_version: 3.3.26
 signing_key: 
 specification_version: 4
 summary: pbxproj files git conflicts solver