Skip to content

Matter Lock: Re-profile a Lock if it requires an unlatch embedded config#2945

Merged
hcarter-775 merged 2 commits intomainfrom
fix/update-lock-profile-if-unlatch-supported
May 4, 2026
Merged

Matter Lock: Re-profile a Lock if it requires an unlatch embedded config#2945
hcarter-775 merged 2 commits intomainfrom
fix/update-lock-profile-if-unlatch-supported

Conversation

@hcarter-775
Copy link
Copy Markdown
Contributor

Description of Change

Issue reported: #2940

In latest production release last week, a profiling bug related to a subset of Nuki devices supporting UNLATCH was introduced which led the embedded config for these devices to not be exposed, since the capabilities may not have explicitly changed, though their fingerprints did not initially include the unlatch embedded configs.

Summary of Completed Tests

New unit test added to match the Nuki Ultra Lock profiling behavior.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

Channel deleted.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

Test Results

   72 files    506 suites   0s ⏱️
2 790 tests 2 787 ✅ 0 💤 3 ❌
4 678 runs  4 675 ✅ 0 💤 3 ❌

For more details on these failures, see this check.

Results for commit 2c5714c.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

File Coverage
All files 81%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-lock/src/lazy_load_subdriver.lua 57%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-lock/src/lock_utils.lua 68%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-lock/src/init.lua 91%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-lock/src/new-matter-lock/can_handle.lua 90%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-lock/src/new-matter-lock/init.lua 79%

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 2c5714c

@ldeora
Copy link
Copy Markdown

ldeora commented May 4, 2026

Tested it with a real device (while not being at home! No risk, no fun!) and it works.

Copy link
Copy Markdown
Contributor

@ctowns ctowns left a comment

Choose a reason for hiding this comment

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

Thank you @hcarter-775 and @ldeora!

@hcarter-775 hcarter-775 force-pushed the fix/update-lock-profile-if-unlatch-supported branch from 6788222 to 2c5714c Compare May 4, 2026 20:25
@hcarter-775 hcarter-775 merged commit 7f042d1 into main May 4, 2026
19 of 22 checks passed
@hcarter-775 hcarter-775 deleted the fix/update-lock-profile-if-unlatch-supported branch May 4, 2026 21:09
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