Skip to content

Commit

Permalink
Merge pull request #6556 from samvera/updates_changes_required_notifi…
Browse files Browse the repository at this point in the history
…cation_spec

Adds work class swapping to `spec/services/hyrax/workflow/changes_required_notification_spec.rb`.
  • Loading branch information
dlpierce authored Jan 3, 2024
2 parents 7160e0e + b85fda3 commit 7617d1f
Showing 1 changed file with 14 additions and 7 deletions.
21 changes: 14 additions & 7 deletions spec/services/hyrax/workflow/changes_required_notification_spec.rb
Original file line number Diff line number Diff line change
@@ -1,20 +1,27 @@
# frozen_string_literal: true
RSpec.describe Hyrax::Workflow::ChangesRequiredNotification do
let(:approver) { FactoryBot.create(:user) }
let(:depositor) { FactoryBot.create(:user) }
let(:to_user) { FactoryBot.create(:user) }
let(:cc_user) { FactoryBot.create(:user) }
let(:work) { FactoryBot.create(:work, user: depositor) }
let(:entity) { FactoryBot.create(:sipity_entity, proxy_for_global_id: work.to_global_id) }
let(:approver) { create(:user) }
let(:depositor) { create(:user) }
let(:to_user) { create(:user) }
let(:cc_user) { create(:user) }
let(:work) do
if Hyrax.config.disable_wings
valkyrie_create(:monograph, title: ["Test title"], depositor: depositor.user_key)
else
create(:work, user: depositor)
end
end
let(:entity) { create(:sipity_entity, proxy_for_global_id: Hyrax::GlobalID(work).to_s) }
let(:comment) { double("comment", comment: 'A pleasant read') }
let(:recipients) { { 'to' => [to_user], 'cc' => [cc_user] } }
let(:expected_klass) { Hyrax.config.disable_wings ? 'monographs' : 'generic_works' }

describe ".send_notification" do
it 'sends a message to all users' do # rubocop:disable RSpec/ExampleLength
expect(approver)
.to receive(:send_message)
.with(anything,
"Test title (<a href=\"/concern/generic_works/#{work.id}\">#{work.id}</a>) " \
"Test title (<a href=\"/concern/#{expected_klass}/#{work.id}\">#{work.id}</a>) " \
"requires additional changes before approval.\n 'A pleasant read'",
anything)
.exactly(3)
Expand Down

0 comments on commit 7617d1f

Please sign in to comment.