Rosa Gutierrez
92cb749e16
Clear assignee's existing notifications in setup since Notifier now uses create_or_find_by instead of create, and reload the association to avoid caching after destroy_all. Update fixture references after rebase: use `logo_assignment_kevin` as base notification and `logo_mentioned_david` for mention tests. Update source to `logo_published` in tests that need generic card events. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
98 lines
2.7 KiB
Ruby
98 lines
2.7 KiB
Ruby
require "test_helper"
|
|
|
|
class Notification::PushableTest < ActiveSupport::TestCase
|
|
setup do
|
|
@user = users(:david)
|
|
@notification = notifications(:logo_mentioned_david)
|
|
end
|
|
|
|
test "push_later enqueues Notification::PushJob" do
|
|
assert_enqueued_with(job: Notification::PushJob, args: [ @notification ]) do
|
|
@notification.push_later
|
|
end
|
|
end
|
|
|
|
test "push calls process on all registered targets" do
|
|
target_class = mock("push_target_class")
|
|
target_class.expects(:process).with(@notification)
|
|
|
|
original_targets = Notification.push_targets
|
|
Notification.push_targets = [ target_class ]
|
|
|
|
@notification.push
|
|
ensure
|
|
Notification.push_targets = original_targets
|
|
end
|
|
|
|
test "push_later is called after notification is created" do
|
|
assert_enqueued_with(job: Notification::PushJob) do
|
|
@user.notifications.create!(
|
|
source: events(:layout_published),
|
|
creator: users(:jason)
|
|
)
|
|
end
|
|
end
|
|
|
|
test "push_later is called when notification source changes" do
|
|
assert_enqueued_with(job: Notification::PushJob) do
|
|
@notification.update!(source: events(:logo_published))
|
|
end
|
|
end
|
|
|
|
test "push_later is not called for other updates" do
|
|
assert_no_enqueued_jobs only: Notification::PushJob do
|
|
@notification.update!(unread_count: 5)
|
|
end
|
|
end
|
|
|
|
test "register_push_target accepts symbols" do
|
|
original_targets = Notification.push_targets.dup
|
|
|
|
Notification.register_push_target(:web)
|
|
|
|
assert_includes Notification.push_targets, Notification::PushTarget::Web
|
|
ensure
|
|
Notification.push_targets = original_targets
|
|
end
|
|
|
|
test "push processes targets for normal notifications" do
|
|
target_class = mock("push_target_class")
|
|
target_class.expects(:process).with(@notification)
|
|
|
|
original_targets = Notification.push_targets
|
|
Notification.push_targets = [ target_class ]
|
|
|
|
@notification.push
|
|
ensure
|
|
Notification.push_targets = original_targets
|
|
end
|
|
|
|
test "push skips targets when creator is system user" do
|
|
@notification.update!(creator: users(:system))
|
|
|
|
target_class = mock("push_target_class")
|
|
target_class.expects(:process).never
|
|
|
|
original_targets = Notification.push_targets
|
|
Notification.push_targets = [ target_class ]
|
|
|
|
@notification.push
|
|
ensure
|
|
Notification.push_targets = original_targets
|
|
end
|
|
|
|
test "push skips targets for cancelled accounts" do
|
|
@user.account.cancel(initiated_by: @user)
|
|
|
|
target_class = mock("push_target_class")
|
|
target_class.expects(:process).never
|
|
|
|
original_targets = Notification.push_targets
|
|
Notification.push_targets = [ target_class ]
|
|
|
|
@notification.push
|
|
ensure
|
|
Notification.push_targets = original_targets
|
|
end
|
|
end
|