Stanko Krtalić
eceb6c9b43
* Add self-service account deletion * Disable access to cancelled accounts & implement Stripe interactions * Add tests * Fix failing tests * Remove cancelled accounts from lists * Fix incorrect redirect * Use _path instead of _url for consistency * Don't track how far the inicieration job got We still want the step tracking so that the job can be interrupted. But, since the scope is idempotent, we don't need to track how far it got. * Specify the exact time when the data will be deleted * Fix crash due to unadvancable cursor * Rename up_for_incineration to due_for_incineration * Regenrate the schema * Fix incorrect path check * Migrate the SQLite schema * Only show the cancel button on cancellable accounts * Check that a subscirption method exists before calling it * Ignore job failures due to missing records when an account gets deleted * Skip sending notifications on cancelled accounts * Use collbacks to integrate * Add a blank line between queue_as and discard_on * Break checks into methods * Inline methods * Rename Account::IncinerateJob * Run migrations * Migrate SQLite
59 lines
1.5 KiB
Ruby
59 lines
1.5 KiB
Ruby
require "test_helper"
|
|
|
|
class Webhook::TriggerableTest < ActiveSupport::TestCase
|
|
setup do
|
|
@account = accounts(:"37s")
|
|
@board = boards(:writebook)
|
|
@card = @board.cards.first
|
|
@webhook = @board.webhooks.create!(
|
|
name: "Test Webhook",
|
|
url: "https://example.com/webhook",
|
|
subscribed_actions: [ "card_published" ]
|
|
)
|
|
# Create a test event
|
|
@event = @board.events.create!(
|
|
creator: users(:david),
|
|
eventable: @card,
|
|
action: "card_published"
|
|
)
|
|
@user = users(:david)
|
|
end
|
|
|
|
test "trigger creates delivery for active accounts" do
|
|
assert_difference -> { Webhook::Delivery.count }, 1 do
|
|
@webhook.trigger(@event)
|
|
end
|
|
|
|
delivery = Webhook::Delivery.last
|
|
assert_equal @event, delivery.event
|
|
assert_equal @webhook, delivery.webhook
|
|
end
|
|
|
|
test "trigger skips cancelled accounts" do
|
|
@account.cancel(initiated_by: @user)
|
|
|
|
assert_no_difference -> { Webhook::Delivery.count } do
|
|
@webhook.trigger(@event)
|
|
end
|
|
end
|
|
|
|
test "triggered_by scope finds webhooks for event" do
|
|
other_webhook = @board.webhooks.create!(
|
|
name: "Other Webhook",
|
|
url: "https://example.com/other",
|
|
subscribed_actions: [ "card_closed" ]
|
|
)
|
|
|
|
matching_webhooks = Webhook.triggered_by(@event)
|
|
|
|
assert_includes matching_webhooks, @webhook
|
|
assert_not_includes matching_webhooks, other_webhook
|
|
end
|
|
|
|
test "active scope only returns active webhooks" do
|
|
@webhook.update!(active: false)
|
|
|
|
assert_not_includes Webhook.active, @webhook
|
|
end
|
|
end
|