Skip to content

Fixing zigbee-switch tests for ColorTempPhysMireds#2886

Open
aleclorimer wants to merge 2 commits intomainfrom
colorTempPhysicalMireds-zigbee-switch-test-fixes
Open

Fixing zigbee-switch tests for ColorTempPhysMireds#2886
aleclorimer wants to merge 2 commits intomainfrom
colorTempPhysicalMireds-zigbee-switch-test-fixes

Conversation

@aleclorimer
Copy link
Copy Markdown
Contributor

  • Fixing tests against dev for expected new messages for the ColorTempPhysical(min/max)Mireds attribute
  • Adding dev to jenkins-driver-tests.yml workflow file matrix

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

Channel deleted.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

Test Results

   72 files    495 suites   0s ⏱️
2 714 tests 2 702 ✅ 0 💤 12 ❌
4 613 runs  4 588 ✅ 0 💤 25 ❌

For more details on these failures, see this check.

Results for commit 799a170.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

File Coverage
All files 96%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/color_temp_range_handlers/init.lua 80%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/aqara-light/init.lua 92%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/lazy_load_subdriver.lua 57%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/preferences.lua 97%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/init.lua 92%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/inovelli/vzm30-sn/init.lua 95%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/laisiao/init.lua 96%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/configurations/init.lua 98%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/aqara/version/init.lua 94%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/frient-IO/unbind_request.lua 71%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/frient-IO/init.lua 88%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/inovelli/init.lua 98%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/aqara/init.lua 94%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/tuya-multi/can_handle.lua 90%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/inovelli/vzm32-sn/init.lua 95%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/frient/init.lua 94%

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 799a170

@aleclorimer aleclorimer force-pushed the colorTempPhysicalMireds-zigbee-switch-test-fixes branch from d87a63d to 799a170 Compare April 9, 2026 20:27
@aleclorimer
Copy link
Copy Markdown
Contributor Author

These tests are failing because the lua libs being used for 60 and the "Run driver tests" GH workflow don't yet have the filtering on api version, so this is trying to run the tests meant for api.version = 20.

@varzac
Copy link
Copy Markdown
Contributor

varzac commented Apr 10, 2026

These tests are failing because the lua libs being used for 60 and the "Run driver tests" GH workflow don't yet have the filtering on api version, so this is trying to run the tests meant for api.version = 20.

I would have expected the Jenkins runs to pass though? I'll check the failure details tomorrow morning, but I'm not sure why they would be failing

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