Skip to content

VAPI-2714 Update the python SDK to include BRTC endpoint APIs#280

Open
smoghe-bw wants to merge 49 commits intomainfrom
brtc-python-sdk
Open

VAPI-2714 Update the python SDK to include BRTC endpoint APIs#280
smoghe-bw wants to merge 49 commits intomainfrom
brtc-python-sdk

Conversation

@smoghe-bw
Copy link
Copy Markdown

No description provided.

- Introduced `SipCredentials` model for managing SIP connection credentials.
- Added documentation for `CreateEndpointRequestBase`, `CreateEndpointResponse`, `CreateEndpointResponseObject`, `CreateWebRtcConnectionRequest`, `Device`, `DeviceStatusEnum`, `Endpoint`, `EndpointDirectionEnum`, `EndpointEvent`, `EndpointEventTypeEnum`, `EndpointResponse`, `EndpointStatusEnum`, `EndpointTypeEnum`, `Endpoints`, `EndpointsApi`, `ErrorResponse`, `ListEndpointsResponse`, `Page`, `SipConnectionMetadata`, and `SipCredentials`.
- Implemented example usage in the documentation for each model.
- Implement integration tests for creating, retrieving, listing, and deleting WebRTC endpoints in `test/smoke/test_endpoints_api.py`.
- Add unit tests for models related to endpoint requests and responses, including `CreateEndpointRequestBase`, `CreateEndpointResponse`, `CreateEndpointResponseObject`, `Device`, and their respective enums in the `test/unit/models` directory.
- Ensure tests cover various scenarios including unauthorized and forbidden access attempts.
- Validate the functionality of endpoint events and status enums with dedicated unit tests.
@smoghe-bw smoghe-bw requested review from a team as code owners January 15, 2026 21:43
@bwappsec
Copy link
Copy Markdown

bwappsec commented Jan 15, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@smoghe-bw smoghe-bw marked this pull request as draft January 15, 2026 21:44
@smoghe-bw smoghe-bw marked this pull request as ready for review January 22, 2026 00:27
@smoghe-bw smoghe-bw changed the title VAPI-2694 Update the python SDK to include BRTC endpoint APIs VAPI-2714 Update the python SDK to include BRTC endpoint APIs Jan 22, 2026
smoghe-bw and others added 12 commits March 26, 2026 15:14
- Smoke tests: Replace has_properties with individual assertions, remove
  hamcrest dependency, add assertions for links/data/errors fields and
  all properties on response objects, remove AI-generated comments,
  reuse endpoint ID from create test in delete test
- Unit API tests: Remove pagination test, required-only props test, and
  error handling test; add assertions for tag, devices, and other fields
  on mocked response data
- Unit model tests: Add missing test files for ErrorResponse, Page,
  SipConnectionMetadata, and SipCredentials models

Generated from Claude9 with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Model tests: Replace isinstance checks with value equality for timestamps
  and add missing field assertions (endpoint_response, list_endpoints_response)
- Smoke tests: Add tag assertion on listed endpoint items in listEndpoints
- test_list_endpoints_response: Add assertions for page.total_pages and
  page.page_number, plus value checks on endpoint data fields
- Unit API tests: Replace instance_of/is_in with equal_to assertions using
  expected Prism mock values for endpoint_id, type, status, tag, token, page
  fields, and error arrays

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…terns

- Remove unnecessary assertion in smoke test (listed_ids check)
- Replace all hamcrest matchers with plain assert statements in unit API tests
- Remove dead make_instance(include_optional) pattern from 13 model tests

Generated from Claude9 with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add method (enum: GET/POST/DELETE) to link schema and id, type, source
fields to error schema per the authoritative BRTC spec. Extract
LinkMethodEnum to its own file to match codebase conventions.

Generated from Claude9 with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Use assert_that with individual matchers (equal_to, instance_of, not_none,
greater_than) to match the assertion style used across all other SDK tests.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants