Skip to content

Fixing zigbee-switch tests for ColorTempPhysMireds#2886

Merged
aleclorimer merged 1 commit intomainfrom
colorTempPhysicalMireds-zigbee-switch-test-fixes
Apr 14, 2026
Merged

Fixing zigbee-switch tests for ColorTempPhysMireds#2886
aleclorimer merged 1 commit intomainfrom
colorTempPhysicalMireds-zigbee-switch-test-fixes

Conversation

@aleclorimer
Copy link
Copy Markdown
Contributor

@aleclorimer aleclorimer commented Apr 9, 2026

  • 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    502 suites   0s ⏱️
2 758 tests 2 758 ✅ 0 💤 0 ❌
4 655 runs  4 655 ✅ 0 💤 0 ❌

Results for commit 8c3a810.

♻️ 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/aqara-light/init.lua 92%
/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/tuya-multi/can_handle.lua 90%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/aqara/init.lua 94%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/preferences.lua 97%
/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/init.lua 92%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/laisiao/init.lua 96%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/inovelli/init.lua 98%
/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/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/configurations/init.lua 98%
/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/frient/init.lua 94%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/aqara/version/init.lua 94%

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 8c3a810

@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

@aleclorimer aleclorimer requested a review from varzac April 10, 2026 16:28
@aleclorimer aleclorimer force-pushed the colorTempPhysicalMireds-zigbee-switch-test-fixes branch from 799a170 to 952d413 Compare April 14, 2026 14:33
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 14, 2026

Channel deleted.

@aleclorimer aleclorimer force-pushed the colorTempPhysicalMireds-zigbee-switch-test-fixes branch 3 times, most recently from 07bff8d to 6317a6c Compare April 14, 2026 18:36
- Splitting tests based off api_version
@aleclorimer aleclorimer force-pushed the colorTempPhysicalMireds-zigbee-switch-test-fixes branch from 6317a6c to 8c3a810 Compare April 14, 2026 18:37
@aleclorimer aleclorimer merged commit 98114d8 into main Apr 14, 2026
13 checks passed
@aleclorimer aleclorimer deleted the colorTempPhysicalMireds-zigbee-switch-test-fixes branch April 14, 2026 20:19
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