Skip to content

Commit

Permalink
Clarify test
Browse files Browse the repository at this point in the history
  • Loading branch information
matthewelwell committed Jan 15, 2024
1 parent 3f87d7f commit 722f838
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion api/tests/unit/webhooks/test_unit_webhooks.py
Original file line number Diff line number Diff line change
Expand Up @@ -324,6 +324,11 @@ def test_call_webhook_with_failure_mail_after_retries_does_not_retry_if_not_usin
def test_call_integration_webhook_does_not_raise_error_on_backoff_give_up(
mocker: MockerFixture,
) -> None:
"""
This test is essentially verifying that the `raise_on_giveup` argument
passed to the backoff decorator on _call_webhook is working as we
expect it to.
"""
# Given
url = "https://test.com/webhook"
config = mocker.MagicMock(secret=None, url=url)
Expand All @@ -334,5 +339,6 @@ def test_call_integration_webhook_does_not_raise_error_on_backoff_give_up(
result = call_integration_webhook(config, data={})

# Then
# we don't get a result from the function, but no exception is raised
# we don't get a result from the function (as expected), and no exception is
# raised
assert result is None

0 comments on commit 722f838

Please sign in to comment.