Import spec: Move most `#child_change` specs to `#update_status`

The new `#child_change` doesn't do a whole lot, so the specs that used
to test it don't belong any more. Update them so they test
`#update_status` instead, where this logic now lives.

1 files changed, 18 insertions, 19 deletions

diff --git a/spec/models/import_spec.rb b/spec/models/import_spec.rb
index 46fd8e8b2..08a3c81c0 100644
--- a/spec/models/import_spec.rb
+++ b/spec/models/import_spec.rb
@@ -48,8 +48,23 @@ RSpec.describe Import, type: :model do
     end
   end
 
-  # TODO: Move most of these to #update_status
   describe "#child_change" do
+    it "calls #update_status" do
+      allow(workbench_import).to receive(:update)
+
+      expect(workbench_import).to receive(:update_status)
+      workbench_import.child_change
+    end
+
+    it "calls #update_referentials" do
+      allow(workbench_import).to receive(:update)
+
+      expect(workbench_import).to receive(:update_referentials)
+      workbench_import.child_change
+    end
+  end
+
+  describe "#update_status" do
     shared_examples(
       "updates :status to failed when child status indicates failure"
     ) do |failure_status|
@@ -68,7 +83,7 @@ RSpec.describe Import, type: :model do
             status: 'failed'
           )
 
-          workbench_import.child_change
+          workbench_import.update_status
         end
       end
     end
@@ -119,7 +134,7 @@ RSpec.describe Import, type: :model do
 
         expect(workbench_import).not_to receive(:update)
 
-        workbench_import.child_change
+        workbench_import.update_status
       end
     end
 
@@ -140,22 +155,6 @@ RSpec.describe Import, type: :model do
       "canceled"
     )
 
-    it "calls #update_status" do
-      allow(workbench_import).to receive(:update)
-
-      expect(workbench_import).to receive(:update_status)
-      workbench_import.child_change
-    end
-
-    it "calls #update_referentials" do
-      allow(workbench_import).to receive(:update)
-
-      expect(workbench_import).to receive(:update_referentials)
-      workbench_import.child_change
-    end
-  end
-
-  describe "#update_status" do
     it "updates :ended_at to now when status is finished" do
       skip "Redo the `#update_status` code to make it easier to write this."
     end