stasis_broadcast: Add ARI tests for StasisBroadcast#118
Open
danieldonoghue wants to merge 1 commit intoasterisk:masterfrom
Open
stasis_broadcast: Add ARI tests for StasisBroadcast#118danieldonoghue wants to merge 1 commit intoasterisk:masterfrom
danieldonoghue wants to merge 1 commit intoasterisk:masterfrom
Conversation
|
|
github-actions
bot
added
pr-submit-testing-in-progress
pr-submit-tests-passed
and removed
pr-submit-testing-in-progress
labels
|
Workflow PRCheck completed successfully |
Author
|
cherry-pick-to: 23 |
gtjoseph
dismissed
github-actions[bot]’s stale review
Pull Request Checklist Complete
danieldonoghue
force-pushed
the
master-stasis-broadcast-test
branch
from
496d8d6 to
a258f4f
Compare
github-actions
bot
added
pr-submit-testing-in-progress
pr-submit-tests-passed
and removed
pr-submit-tests-passed
pr-submit-testing-in-progress
labels
|
Workflow PRCheck completed successfully |
Author
|
asterisk-test-pr: 1617 |
Adds tests for the res_stasis_broadcast and app_stasis_broadcast modules, covering the four core behaviours of StasisBroadcast(): claim_success: originates a channel via StasisBroadcast(), verifies the CallBroadcast event is delivered to connected ARI applications, claims the channel via POST /ari/events/claim, and confirms the channel enters Stasis in the claiming application with a StasisStart event. claim_already_taken: verifies that when two applications race to claim the same channel, the first receives 204 No Content and the second receives 409 Conflict. claim_filter: verifies that the app_filter regex parameter correctly limits CallBroadcast delivery to matching application names only. claim_timeout: verifies that when no application claims the channel within the configured timeout, STASISSTATUS is set to TIMEOUT and control returns to the dialplan. Development was assisted by Claude (Anthropic). All generated code has been reviewed, tested, and is understood by the author. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
danieldonoghue
force-pushed
the
master-stasis-broadcast-test
branch
from
a258f4f to
d6d6b53
Compare
github-actions
bot
added
pr-submit-testing-in-progress
pr-submit-tests-passed
and removed
pr-submit-tests-passed
pr-submit-testing-in-progress
labels
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Attention! This pull request may contain issues that could prevent it from being accepted. Please review the checklist below and take the recommended action. If you believe any of these are not applicable, just add a comment and let us know.
- The PR title does not match the commit title. This can cause confusion for reviewers and future maintainers. GitHub doesn't automatically update the PR title when you update the commit message so if you've updated the commit with a force-push, please update the PR title to match the new commit message body.
- The PR description does not match the commit message body. This can cause confusion for reviewers and future maintainers. GitHub doesn't automatically update the PR description when you update the commit message so if you've updated the commit with a force-push, please update the PR description to match the new commit message body.
Documentation:
|
Workflow PRCheck completed successfully |
danieldonoghue
changed the title
Author
|
I believe the GitHub-actions request related to the PR title and description should now be satisfied |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds tests for the res_stasis_broadcast and app_stasis_broadcast modules,
covering the four core behaviours of StasisBroadcast():
claim_success: originates a channel via StasisBroadcast(), verifies the
CallBroadcast event is delivered to connected ARI applications, claims
the channel via POST /ari/events/claim, and confirms the channel enters
Stasis in the claiming application with a StasisStart event.
claim_already_taken: verifies that when two applications race to claim
the same channel, the first receives 204 No Content and the second
receives 409 Conflict.
claim_filter: verifies that the app_filter regex parameter correctly
limits CallBroadcast delivery to matching application names only.
claim_timeout: verifies that when no application claims the channel
within the configured timeout, STASISSTATUS is set to TIMEOUT and
control returns to the dialplan.
Development was assisted by Claude (Anthropic). All generated code has
been reviewed, tested, and is understood by the author.
Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com