Override child profiles for lights and plugs only

[nickolas-deboom]

Type of Change

• WWST Certification Request
  • If this is your first time contributing code:
    • I have reviewed the README.md file
    • I have reviewed the CODE_OF_CONDUCT.md file
    • I have signed the CLA
  • I plan on entering a WWST Certification Request or have entered a request through the WWST Certification console at developer.smartthings.com
• Bug fix
• New feature
• Refactor

Checklist

• I have performed a self-review of my code
• I have commented my code in hard-to-understand areas
• I have verified my changes by testing with a device or have communicated a plan for testing
• I am adding new behavior, such as adding a sub-driver, and have added and run new unit tests to cover the new behavior

Description of Change

Devices that utilize the child_device_profile_overrides table may contain other endpoints that should not have their profile overridden when the child device is created. This PR ensures that the profile is only overridden for light and plug child device types. See the discussion from PR 1872.

Note that GENERIC_SWITCH_ID was removed from device_type_profile_map. This is because it is never used in this table and also to make it clear that device types in this table may have their child device profile overridden (buttons do not use parent-child).

Summary of Completed Tests

[github-actions]

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

[github-actions]

Test Results

   64 files    406 suites   0s ⏱️
2 013 tests 2 013 ✅ 0 💤 0 ❌
3 489 runs  3 489 ✅ 0 💤 0 ❌

Results for commit 56da529.

♻️ This comment has been updated with latest results.

[github-actions]

File	Coverage
All files	94%	✅
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/init.lua	96%	✅
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/embedded-cluster-utils.lua	38%	❌
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/eve-energy/init.lua	91%	✅
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/aqara-cube/init.lua	96%	✅

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 56da529

[hcarter-775]

can we make a unit test or two for this? seems like a worthwhile thing to make sure it doesn't break

[nickolas-deboom]

can we make a unit test or two for this? seems like a worthwhile thing to make sure it doesn't break

Yeah good call - I updated a mock device in test_multi_switch_parent_child_lights.lua to use an overridden profile for one of its child devices. Also note that I made a mistake in the initial commit, now it only overrides the profile if the profile is going to be "plug-binary", rather than any profile in device_type_profile_map. This is the only situation that we'd want to do this at this point.