Skip to content

Feature/rdkemw 12540#383

Open
hgfell683 wants to merge 31 commits into
developfrom
feature/RDKEMW-12540
Open

Feature/rdkemw 12540#383
hgfell683 wants to merge 31 commits into
developfrom
feature/RDKEMW-12540

Conversation

@hgfell683

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings February 23, 2026 15:43
@hgfell683
hgfell683 requested a review from a team as a code owner February 23, 2026 15:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds comprehensive L2 (Level 2) integration tests for the HdcpProfile plugin, enabling automated testing of both COM-RPC and JSON-RPC interfaces. The tests validate HDCP status queries, notification mechanisms, and event handling through mocked device services layer APIs.

Changes:

  • Adds HdcpProfile_L2Test.cpp with 870 lines covering COM-RPC interface tests, JSON-RPC method tests, and event notification validation
  • Updates CMakeLists.txt to conditionally compile HdcpProfile tests when PLUGIN_HDCPPROFILE is enabled
  • Modifies GitHub Actions workflow to include HdcpProfile in the L2 test builds and temporarily references feature branches

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 10 comments.

File Description
Tests/L2Tests/tests/HdcpProfile_L2Test.cpp New comprehensive test suite covering GetHDCPStatus, GetSettopHDCPSupport, Register/Unregister, and notification events using both COM-RPC and JSON-RPC interfaces
Tests/L2Tests/CMakeLists.txt Adds conditional compilation for HdcpProfile tests when PLUGIN_HDCPPROFILE flag is enabled
.github/workflows/L2-tests.yml Enables HdcpProfile plugin in build configuration and temporarily references feature/RDKEMW-12540 branch for testing

Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
Comment thread .github/workflows/L2-tests.yml Outdated
Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
Copilot AI review requested due to automatic review settings February 26, 2026 19:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
Comment thread Tests/L2Tests/tests/HdcpProfile_L2Test.cpp
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.

2 participants