Skip to content

rest_api: Add tests for outbound websocket. #85

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gtjoseph
Copy link
Member

  • Added AriServerFactory and AriServerProtocol to lib/python/asterisk/ari.py.
  • Created tests/rest_api/websocket.
  • Moved existing tests/rest_api/websocket_requests/* tests to
    tests/rest_api/websocket/rest_over_websocket.
  • Updated the rest_over_websocket tests to use reactor.run() properly.
  • Added test outbound_websocket/persistent.
  • Added test outbound_websocket/per_call.
  • Added test outbound_websocket/local_ari_user_read_only.

@gtjoseph
Copy link
Member Author

cherry-pick-to: 20
cherry-pick-to: 21
cherry-pick-to: 22

Copy link

Workflow PRCheck completed successfully

Copy link

Workflow PRCheck completed successfully

@github-actions github-actions bot added cherry-pick-testing-in-progress Cherry-Pick tests in progress cherry-pick-checks-passed Cherry-Pick checks passed and removed cherry-pick-test cherry-pick-testing-in-progress Cherry-Pick tests in progress labels Apr 24, 2025
Copy link

Workflow PRCPCheck completed successfully

Copy link

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 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:

Copy link

Workflow PRCheck completed successfully

Copy link

Workflow PRCheck completed successfully

Copy link

Workflow PRCheck completed successfully

* Added AriServerFactory and AriServerProtocol to lib/python/asterisk/ari.py.
* Created tests/rest_api/websocket.
* Moved existing tests/rest_api/websocket_requests/* tests to
  tests/rest_api/websocket/rest_over_websocket.
* Updated the rest_over_websocket tests to use reactor.run() properly.
* Added test outbound_websocket/persistent.
* Added test outbound_websocket/per_call.
* Added test outbound_websocket/local_ari_user_read_only.
* Updated rest_api/asterisk/modules/get_modules to load res_sorcery_config.so
* Updated rest_api/asterisk/modules/get_modules to load res_websocket_client.so
@gtjoseph gtjoseph force-pushed the master-outbound-websockets branch from ea644c4 to aa08392 Compare May 15, 2025 13:13
Copy link

Workflow PRCheck completed successfully

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant