Skip to content

Comments

feat(video): Add the list connections response#610

Merged
dragonmantank merged 4 commits intomainfrom
devx-10296-list-video-connections
Feb 20, 2026
Merged

feat(video): Add the list connections response#610
dragonmantank merged 4 commits intomainfrom
devx-10296-list-video-connections

Conversation

@dragonmantank
Copy link
Member

See title

Contribution Checklist

@codecov-commenter
Copy link

codecov-commenter commented Feb 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.93%. Comparing base (c5c0999) to head (d0b8b37).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #610   +/-   ##
=========================================
  Coverage     98.93%   98.93%           
- Complexity     3610     3622   +12     
=========================================
  Files           542      545    +3     
  Lines          7987     8004   +17     
  Branches        443      443           
=========================================
+ Hits           7902     7919   +17     
  Misses           64       64           
  Partials         21       21           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for listing connections in Vonage Video sessions. The implementation follows the established patterns in the video package and includes comprehensive test coverage.

Changes:

  • Added listConnections() method to VideoClient to retrieve all connections from a video session
  • Created Connection class to represent individual connection data
  • Created ConnectionState enum to represent connection states (CONNECTING, CONNECTED, DISCONNECTED)
  • Created ListConnectionsResponse class to wrap the API response
  • Added comprehensive test coverage including unit tests and endpoint tests

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/main/java/com/vonage/client/video/VideoClient.java Added listConnections endpoint and public method
src/main/java/com/vonage/client/video/ListConnectionsResponse.java Response model extending ListResourceResponse with projectId and sessionId fields
src/main/java/com/vonage/client/video/Connection.java Entity class representing a connection with id, state, and creation timestamp
src/main/java/com/vonage/client/video/ConnectionState.java Enum defining connection states with custom capitalization pattern
src/test/java/com/vonage/client/video/VideoClientTest.java Added test methods and loadJsonResource helper for testing list connections functionality
src/test/java/com/vonage/client/video/ListConnectionsResponseTest.java Comprehensive tests for ListConnectionsResponse deserialization
src/test/resources/com/vonage/client/video/list-connections-response.json Test fixture with sample connection data
src/test/resources/com/vonage/client/video/list-connections-response-full.json Test fixture with placeholders for parameterized testing
CHANGELOG.md Updated with new feature entry

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI commented Feb 20, 2026

@dragonmantank I've opened a new pull request, #611, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 3 commits February 20, 2026 15:23
* Initial plan

* Use Jsonable.fromString() for ConnectionState consistency

Co-authored-by: dragonmantank <108948+dragonmantank@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: dragonmantank <108948+dragonmantank@users.noreply.github.com>
…sponse.json

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@dragonmantank dragonmantank merged commit c0f3497 into main Feb 20, 2026
16 checks passed
@dragonmantank dragonmantank deleted the devx-10296-list-video-connections branch February 20, 2026 20:28
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