SmartThingsCommunity / SmartThingsEdgeDrivers

Apache License 2.0
259 stars 446 forks source link

CHAD-13655 Add emulated reset implementation #1550

Closed greens closed 1 month ago

greens commented 1 month ago

This emulated reset behavior is defined in the defaults, but some sub-drivers overwrote the defaults without including it.

Check all that apply

Type of Change

Checklist

Description of Change

Adds default emulated reset behavior to sub-drivers that chose to overwrite it.

Summary of Completed Tests

Existing unit tests were not affected by this change

github-actions[bot] commented 1 month ago

Invitation URL: https://bestow-regional.api.smartthings.com/invite/gV2qY1qrG429

github-actions[bot] commented 1 month ago

Test Results

   60 files  ±0    375 suites  ±0   0s :stopwatch: ±0s 1 820 tests ±0  1 820 :white_check_mark: ±0  0 :zzz: ±0  0 :x: ±0  3 169 runs  +1  3 169 :white_check_mark: +1  0 :zzz: ±0  0 :x: ±0 

Results for commit 7542e4b4. ± Comparison against base commit c18a6de6.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 month ago

zigbee-power-meter_coverage.xml

File Coverage
All files 95% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-power-meter/src/shinasystems/init.lua 96% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-power-meter/src/frient/init.lua 76% :x:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-power-meter/src/init.lua 97% :white_check_mark:

zigbee-switch_coverage.xml

File Coverage
All files 97% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-power-meter/src/shinasystems/init.lua 96% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-power-meter/src/frient/init.lua 76% :x:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-power-meter/src/init.lua 97% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/init.lua 98% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/preferences.lua 97% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/aqara/multi-switch/init.lua 86% :x:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/zigbee-dimmer-power-energy/init.lua 73% :x:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/aqara/init.lua 95% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/zigbee-switch-power/vimar/init.lua 70% :x:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/ge-link-bulb/init.lua 95% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/wallhero/init.lua 97% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/zigbee-switch-power/aurora-relay/init.lua 80% :x:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/zigbee-dimmer-power-energy/enbrighten-metering-dimmer/init.lua 91% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/aqara-light/init.lua 93% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/aqara/version/init.lua 94% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/jasco/init.lua 94% :white_check_mark:

Minimum allowed coverage is 90%

Generated by :monkey: cobertura-action against 7542e4b46209f57f337e79b716e58929f41e2bb1

inasail commented 1 month ago

@greens These devices (lumi.plug.maeu01 , lumi.plug.macn01) are using https://github.com/SmartThingsCommunity/SmartThingsEdgeDrivers/blob/main/drivers/SmartThings/zigbee-switch/src/aqara/version/init.lua#L25

and do you think we need to support reset function for all devices that have energyMeter capability including Light/Switch category device? (Let me check other people's opinion.) I listed all the related driver in the table. https://smartthings.atlassian.net/browse/CHAD-13655

greens commented 1 month ago

@inasail added two more that I missed, robb and the aqara version sub-driver