Skip to content

Conversation

@nickolas-deboom
Copy link
Contributor

In the matter camera subdriver, the webrtc capability should be included if the WebRTCTransportProvider clusters is implemented as SERVER and if WebRTCTransportRequestor is implemented as CLIENT; this commit adds a check for the former which is currently missing from the subdriver.

In the matter camera subdriver, the webrtc capability should be included
if the `WebRTCTransportProvider` clusters is implemented as `SERVER` and
if `WebRTCTransportRequestor` is implemented as `CLIENT`; this commit
adds a check for the former which is currently missing from the subdriver.
@github-actions
Copy link

github-actions bot commented Nov 19, 2025

Channel deleted.

@github-actions
Copy link

Test Results

   71 files    469 suites   0s ⏱️
2 457 tests 2 457 ✅ 0 💤 0 ❌
4 177 runs  4 177 ✅ 0 💤 0 ❌

Results for commit 695c035.

@github-actions
Copy link

File Coverage
All files 92%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/init.lua 96%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/sub_drivers/camera/camera_utils/device_configuration.lua 97%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/sub_drivers/camera/camera_utils/utils.lua 95%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/sub_drivers/camera/camera_handlers/attribute_handlers.lua 95%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/sub_drivers/camera/camera_handlers/capability_handlers.lua 78%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/sub_drivers/third_reality_mk1/init.lua 95%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/switch_utils/fields.lua 99%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/switch_utils/utils.lua 91%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/switch_utils/device_configuration.lua 97%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/switch_utils/embedded_cluster_utils.lua 38%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/sub_drivers/camera/init.lua 98%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/sub_drivers/aqara_cube/init.lua 96%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/switch_handlers/capability_handlers.lua 88%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/switch_handlers/attribute_handlers.lua 83%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/switch_handlers/event_handlers.lua 97%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/sub_drivers/eve_energy/init.lua 92%

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 695c035

@nickolas-deboom nickolas-deboom merged commit 1395f67 into main Nov 19, 2025
11 checks passed
@nickolas-deboom nickolas-deboom deleted the matter-camera-fix-webrtc-capability-inclusion branch November 19, 2025 23:03
hcarter-775 pushed a commit that referenced this pull request Nov 20, 2025
In the matter camera subdriver, the webrtc capability should be included
if the `WebRTCTransportProvider` clusters is implemented as `SERVER` and
if `WebRTCTransportRequestor` is implemented as `CLIENT`; this commit
adds a check for the former which is currently missing from the subdriver.
hcarter-775 added a commit that referenced this pull request Nov 20, 2025
Co-authored-by: nickolas-deboom <158304111+nickolas-deboom@users.noreply.github.com>
hcarter-775 pushed a commit that referenced this pull request Nov 20, 2025
In the matter camera subdriver, the webrtc capability should be included
if the `WebRTCTransportProvider` clusters is implemented as `SERVER` and
if `WebRTCTransportRequestor` is implemented as `CLIENT`; this commit
adds a check for the former which is currently missing from the subdriver.
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.

4 participants