From 1ceb3854756144d20caa66c9b686772660c06c09 Mon Sep 17 00:00:00 2001
From: Brandon Straley <straleyb@onid.oregonstate.edu>
Date: Mon, 27 Jun 2016 08:46:53 -0700
Subject: [PATCH] Add extra test with 2 ids

---
 .../actors/apply_order_actor.rb               |  2 --
 .../apply_order_actor_spec.rb                 | 31 +++++++++++++++++++
 2 files changed, 31 insertions(+), 2 deletions(-)

diff --git a/app/actors/curation_concerns/actors/apply_order_actor.rb b/app/actors/curation_concerns/actors/apply_order_actor.rb
index fff98ebe8..35f3f8f02 100644
--- a/app/actors/curation_concerns/actors/apply_order_actor.rb
+++ b/app/actors/curation_concerns/actors/apply_order_actor.rb
@@ -16,13 +16,11 @@ def sync_members(ordered_member_ids)
             work = ::ActiveFedora::Base.find(old_id)
             curation_concern.ordered_members.delete(work)
           end
-          curation_concern.save
 
           (ordered_member_ids - existing_members_ids).each do |work_id|
             work = ::ActiveFedora::Base.find(work_id)
             curation_concern.ordered_members << work
           end
-          curation_concern.save
           true
         end
 
diff --git a/spec/actors/curation_concerns/apply_order_actor_spec.rb b/spec/actors/curation_concerns/apply_order_actor_spec.rb
index 8e2e417d0..56c454c47 100644
--- a/spec/actors/curation_concerns/apply_order_actor_spec.rb
+++ b/spec/actors/curation_concerns/apply_order_actor_spec.rb
@@ -31,4 +31,35 @@
       end
     end
   end
+
+  describe '#update' do
+    let(:curation_concern) { create(:work_with_two_children, user: user) }
+
+    let(:user) { create(:user) }
+
+    before do
+      allow(described_class).to receive(:update).with(attributes)
+    end
+
+    subject do
+      CurationConcerns::Actors::ActorStack.new(curation_concern,
+                                               user,
+                                               [described_class,
+                                                CurationConcerns::Actors::GenericWorkActor])
+    end
+
+    context 'with in_works_ids' do
+      let(:attributes) { { ordered_member_ids: ["Blah", "Blah2"] } }
+      let(:root_actor) { double }
+      before do
+        allow(CurationConcerns::Actors::RootActor).to receive(:new).and_return(root_actor)
+        allow(root_actor).to receive(:update).with({}).and_return(true)
+        curation_concern.apply_depositor_metadata(user.user_key)
+        curation_concern.save!
+      end
+      it "attaches the parent" do
+        expect(subject.update(attributes)).to be true
+      end
+    end
+  end
 end