Add actual but not expected items in the contains exactly matcher

Author: brunsa2

lib/super_diff/rspec/operation_tree_builders/collection_containing_exactly.rb

@@ -17,10 +17,16 @@ def initialize(actual:, **)
         def unary_operations
           operations = []
 
-          (0...actual.length).each do |index|
+          (0...actual.length).reject do |index|
+            indexes_in_actual_but_not_in_expected.include?(index)
+          end.each do |index|
             add_noop_to(operations, index)
           end
 
+          indexes_in_actual_but_not_in_expected.each do |index|
+            add_insert_to(operations, index)
+          end
+
           indexes_in_expected_but_not_in_actual.each do |index|
             add_delete_to(operations, index)
           end
@@ -84,6 +90,11 @@ def values_in_expected_but_not_in_actual
           end
         end
 
+        def indexes_in_actual_but_not_in_expected
+          @indexes_in_actual_but_not_in_expected ||=
+            pairings_maximizer_best_solution.unmatched_actual_indexes
+        end
+
         def indexes_in_expected_but_not_in_actual
           pairings_maximizer_best_solution.unmatched_expected_indexes
         end

spec/integration/rspec/contain_exactly_matcher_spec.rb

@@ -31,8 +31,8 @@
           diff: proc {
             plain_line %|  [|
             plain_line %|    "Marty",|
-            plain_line %|    "Jennifer",|
-            plain_line %|    "Doc",|
+            beta_line  %|+   "Jennifer",|
+            beta_line  %|+   "Doc",|
             alpha_line %|-   "Einie"|
             plain_line %|  ]|
           },
@@ -129,8 +129,8 @@
               plain_line %|  [|
               plain_line %|    "Marty McFly",|
               plain_line %|    "Doc Brown",|
-              plain_line %|    "Einie",|
               plain_line %|    "Lorraine McFly",|
+              beta_line  %|+   "Einie",|
               alpha_line %|-   "Biff Tannen",|
               alpha_line %|-   "George McFly"|
               plain_line %|  ]|
@@ -239,8 +239,8 @@
               plain_line %|  [|
               plain_line %|    "Marty McFly",|
               plain_line %|    "Doc Brown",|
-              plain_line %|    "Einie",|
-              plain_line %|    "Lorraine McFly",|
+              beta_line  %|+   "Einie",|
+              beta_line  %|+   "Lorraine McFly",|
               alpha_line %|-   "Biff Tannen",|
               alpha_line %|-   /Georg McFly/,|
               alpha_line %|-   /Lorrain McFly/|
@@ -348,9 +348,9 @@
               plain_line %|      foo: "bar"|
               plain_line %|    },|
               plain_line %|    #<Double (anonymous)>,|
-              plain_line %|    {|
-              plain_line %|      blargh: "riddle"|
-              plain_line %|    },|
+              beta_line  %|+   {|
+              beta_line  %|+     blargh: "riddle"|
+              beta_line  %|+   },|
               alpha_line %|-   #<a collection containing exactly (|
               alpha_line %|-     "zing"|
               alpha_line %|-   )>|

spec/integration/rspec/match_array_matcher_spec.rb

@@ -31,8 +31,8 @@
           diff: proc {
             plain_line %|  [|
             plain_line %|    "Marty",|
-            plain_line %|    "Jennifer",|
-            plain_line %|    "Doc",|
+            beta_line  %|+   "Jennifer",|
+            beta_line  %|+   "Doc",|
             alpha_line %|-   "Einie"|
             plain_line %|  ]|
           },
@@ -129,8 +129,8 @@
               plain_line %|  [|
               plain_line %|    "Marty McFly",|
               plain_line %|    "Doc Brown",|
-              plain_line %|    "Einie",|
               plain_line %|    "Lorraine McFly",|
+              beta_line  %|+   "Einie",|
               alpha_line %|-   "Biff Tannen",|
               alpha_line %|-   "George McFly"|
               plain_line %|  ]|
@@ -239,8 +239,8 @@
               plain_line %|  [|
               plain_line %|    "Marty McFly",|
               plain_line %|    "Doc Brown",|
-              plain_line %|    "Einie",|
-              plain_line %|    "Lorraine McFly",|
+              beta_line  %|+   "Einie",|
+              beta_line  %|+   "Lorraine McFly",|
               alpha_line %|-   "Biff Tannen",|
               alpha_line %|-   /Georg McFly/,|
               alpha_line %|-   /Lorrain McFly/|
@@ -352,9 +352,9 @@
               plain_line %|      foo: "bar"|
               plain_line %|    },|
               plain_line %|    #<Double (anonymous)>,|
-              plain_line %|    {|
-              plain_line %|      blargh: "riddle"|
-              plain_line %|    },|
+              beta_line  %|+   {|
+              beta_line  %|+     blargh: "riddle"|
+              beta_line  %|+   },|
               alpha_line %|-   #<a collection containing exactly (|
               alpha_line %|-     "zing"|
               alpha_line %|-   )>|

spec/integration/rspec/match_matcher_spec.rb

@@ -445,11 +445,11 @@
             diff: proc {
               plain_line %|  [|
               plain_line %|    "milk",|
-              plain_line %|    "toast",|
-              plain_line %|    "eggs",|
-              plain_line %|    "cheese",|
-              # plain_line %|    "English muffins",|  # FIXME
-              plain_line %|    "English muffins"|
+              beta_line  %|+   "toast",|
+              beta_line  %|+   "eggs",|
+              beta_line  %|+   "cheese",|
+              # beta_line  %|+   "English muffins",|  # FIXME
+              beta_line  %|+   "English muffins"|
               alpha_line %|-   "bread"|
               plain_line %|  ]|
             },
@@ -537,9 +537,9 @@
               plain_line %|    name: "shopping list",|
               plain_line %|    contents: [|
               plain_line %|      "milk",|
-              plain_line %|      "toast",|
-              # plain_line %|      "eggs",|     # FIXME
-              plain_line %|      "eggs"|
+              beta_line  %|+     "toast",|
+              # beta_line  %|+     "eggs",|     # FIXME
+              beta_line  %|+     "eggs"|
               alpha_line %|-     "bread"|
               plain_line %|    ]|
               plain_line %|  }|
@@ -981,7 +981,7 @@
             },
             diff: proc {
               plain_line %|  [|
-              plain_line %|    "b",|
+              beta_line  %|+   "b",|
               alpha_line %|-   "a"|
               plain_line %|  ]|
             },