Skip to content

Chad 17092 zwave sensor lazy-loading v2#2799

Open
aleclorimer wants to merge 2 commits intomainfrom
CHAD-17092-zwave-sensor-llv2
Open

Chad 17092 zwave sensor lazy-loading v2#2799
aleclorimer wants to merge 2 commits intomainfrom
CHAD-17092-zwave-sensor-llv2

Conversation

@aleclorimer
Copy link
Contributor

Description of Change

Lazy loading of zwave-sensor sub-drivers. Re-using the commits from the first PR to main, then adding in the redundant driver loading in can_handle fix.

Summary of Completed Tests

  • Ran regular driver tests locally on latest scripting-engine python3 tools/run_driver_tests.py
  • Copied and modified drivers/SmartThings/zwave-sensor/src/test/test_zwave_sensor.lua to set version.api = 10
    • Tests passed
    • Tests failed when a can_handle is removed in a subdriver

@github-actions
Copy link

@github-actions
Copy link

github-actions bot commented Feb 23, 2026

Test Results

   72 files    492 suites   0s ⏱️
2 689 tests 2 689 ✅ 0 💤 0 ❌
4 550 runs  4 550 ✅ 0 💤 0 ❌

Results for commit 1efa616.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Feb 23, 2026

File Coverage
All files 98%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-sensor/src/sensative-strip/init.lua 94%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-sensor/src/init.lua 97%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-sensor/src/lazy_load_subdriver.lua 57%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-sensor/src/configurations.lua 99%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-sensor/src/enerwave-motion-sensor/init.lua 92%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-sensor/src/homeseer-multi-sensor/init.lua 94%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-sensor/src/wakeup-no-poll/init.lua 90%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-sensor/src/apiv6_bugfix/can_handle.lua 72%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-sensor/src/aeotec-multisensor/multisensor-7/init.lua 92%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-sensor/src/aeotec-multisensor/multisensor-6/init.lua 92%

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 1efa616

@aleclorimer aleclorimer force-pushed the CHAD-17092-zwave-sensor-llv2 branch from 4c85fa0 to 1efa616 Compare March 11, 2026 16:12
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