Use strings instead of symbols for workflow notification recipients

Fixes #2943
samvera-deprecated · Nov 29, 2016 · 8e2a218 · 8e2a218
1 parent 5d58988
commit 8e2a218
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/app/services/sufia/workflow/abstract_notification.rb b/app/services/sufia/workflow/abstract_notification.rb
@@ -32,7 +32,7 @@ def message
       private
 
         def users_to_notify
-          recipients[:to] + recipients[:cc]
+          recipients['to'] + recipients['cc']
         end
     end
   end

diff --git a/spec/services/sufia/workflow/complete_notification_spec.rb b/spec/services/sufia/workflow/complete_notification_spec.rb
@@ -8,7 +8,7 @@
   let(:work) { create(:generic_work, user: depositor) }
   let(:entity) { create(:sipity_entity, proxy_for_global_id: work.to_global_id.to_s) }
   let(:comment) { double("comment", comment: 'A pleasant read') }
-  let(:recipients) { { to: [to_user], cc: [cc_user] } }
+  let(:recipients) { { 'to' => [to_user], 'cc' => [cc_user] } }
 
   describe ".send_notification" do
     it 'sends a message to all users' do

diff --git a/spec/services/sufia/workflow/pending_review_notification_spec.rb b/spec/services/sufia/workflow/pending_review_notification_spec.rb
@@ -7,7 +7,7 @@
   let(:work) { create(:generic_work, user: depositor) }
   let(:entity) { create(:sipity_entity, proxy_for_global_id: work.to_global_id.to_s) }
   let(:comment) { double("comment", comment: 'A pleasant read') }
-  let(:recipients) { { to: [to_user], cc: [cc_user] } }
+  let(:recipients) { { 'to' => [to_user], 'cc' => [cc_user] } }
 
   describe ".send_notification" do
     it 'sends a message to all users except depositor' do