Skip to content

Matter Switch: Update Parent/Child Switch Device Profiling #2150

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

Closed
wants to merge 2 commits into from

Conversation

hcarter-775
Copy link
Contributor

Description of Change

This PR attempts to do 3 separate tasks:

  1. Re-work the find_default_endpoint function to take a specific cluster type and find the default endpoint per cluster. This is an extension of the current handling where the OnOff cluster and sometimes the Switch cluster is handled as the input cluster.
  2. Re-work the switch profiling logic to attempt a profile switch on a parent device, in case the device has chosen a sub-optimal profile.
  3. Re-work the switch child profiling logic to support profile switches to energy reporting profiles.

Summary of Completed Tests

Copy link

github-actions bot commented May 27, 2025

Channel deleted.

Copy link

github-actions bot commented May 27, 2025

Test Results

   67 files    440 suites   0s ⏱️
2 250 tests 2 250 ✅ 0 💤 0 ❌
3 842 runs  3 842 ✅ 0 💤 0 ❌

Results for commit 9314a30.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 27, 2025

File Coverage
All files 91%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/third-reality-mk1/init.lua 95%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/embedded-cluster-utils.lua 38%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/init.lua 92%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/aqara-cube/init.lua 96%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/eve-energy/init.lua 90%

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 9314a30

@hcarter-775
Copy link
Contributor Author

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.

1 participant