fixing the issue with Using ignore_order and group_by simultaneously

Author: seperman

deepdiff/diff.py

@@ -231,6 +231,7 @@ def __init__(self,
             self.progress_logger = progress_logger
             self.cache_size = cache_size
             _parameters = self.__dict__.copy()
+            _parameters['group_by'] = None  # overwriting since these parameters will be passed on to other passes.
 
         # Non-Root
         if _shared_parameters:

tests/test_ignore_order.py

@@ -749,7 +749,7 @@ def test_cutoff_distance_for_pairs(self):
 
         assert expected == diff_with_dist
 
-    def test_ignore_order_and_group_by(self):
+    def test_ignore_order_and_group_by1(self):
         t1 = [
             {'id': 'AA', 'name': 'Joe', 'ate': ['Nothing']},
             {'id': 'BB', 'name': 'James', 'ate': ['Chips', 'Cheese']},
@@ -781,6 +781,13 @@ def test_ignore_order_and_group_by(self):
         expected2 = {'iterable_item_added': {"root['BB']['ate'][1]": 'Brownies'}}
         assert expected2 == diff2
 
+    def test_ignore_order_and_group_by2(self):
+        t1_data = [{'id': '1', 'codes': ['1', '2', '3']}]
+        t2_data = [{'id': '1', 'codes': ['1', '2', '4']}]
+        diff = DeepDiff(t1_data, t2_data, group_by='id', ignore_order=True)
+        expected = {'values_changed': {"root['1']['codes'][2]": {'new_value': '4', 'old_value': '3'}}}
+        assert expected == diff
+
 
 class TestCompareFuncIgnoreOrder: