add firstled-io_M4S4BAC by thinkaName · Pull Request #2971 · SmartThingsCommunity/SmartThingsEdgeDrivers

thinkaName · 2026-05-14T10:24:23Z

Check all that apply

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

Summary of Completed Tests

github-actions · 2026-05-14T10:24:50Z

Duplicate profile check: Passed - no duplicate profiles detected.

github-actions · 2026-05-14T10:25:35Z

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

github-actions · 2026-05-14T10:29:36Z

Test Results

73 files 513 suites 0s ⏱️
2 947 tests 2 947 ✅ 0 💤 0 ❌
4 819 runs 4 819 ✅ 0 💤 0 ❌

Results for commit 90c7984.

♻️ This comment has been updated with latest results.

github-actions · 2026-05-14T10:29:38Z

File	Coverage
All files	`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/init.lua	`94%`	✅
/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/inovelli/init.lua	`98%`	✅
/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/color_temp_range_handlers/init.lua	`71%`	❌
/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/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/lazy_load_subdriver.lua	`57%`	❌
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-switch/src/frient/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/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/laisiao/init.lua	`96%`	✅
/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/firstled-io/init.lua	`88%`	❌

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 90c7984

cjswedes

Test failures will need to be resolved prior to merging.

cjswedes · 2026-05-18T21:49:49Z

+      { visibility = { displayed = false } }))
+
+    device:send(cluster_base.write_manufacturer_specific_attribute(device,
+      PRIVATE_CLUSTER_ID, PRIVATE_ATTRIBUTE_ID, MFG_CODE, data_types.Uint8, 0x01)) -- private


I dont really know what this attribute write does, but is it something that should be done in device init? The difference being it would be written anytime the hub restarts or the driver is updated

Looks like this was moved to infoChanged to handle preference changes. It seems like it might still be needed in the added handler so that the initial preference default value is written.

When the device is entering the pairing process, it will restore the data to the default values, and both ends will have the same default values.

nand-nor · 2026-05-20T13:19:43Z

Can code coverage be improved in firstled-io/init.lua? Device init tests, preference changes, scenes cluster handling, profile switching, that kind of thing

thinkaName · 2026-05-21T09:59:48Z

Can code coverage be improved in firstled-io/init.lua? Device init tests, preference changes, scenes cluster handling, profile switching, that kind of thing

The maximum improvement can only reach 88%. What suggestions do you have?

cjswedes · 2026-05-21T15:19:02Z

Why is this test file modified? It seems like this device is unrelated to the changes.

cjswedes · 2026-05-21T15:40:42Z

The maximum improvement can only reach 88%. What suggestions do you have?

You can run coverage locally and get the report yourself. When I do it I see that none of the child button behavior is tested.

To run it locally first run your test file with coveraged enabled (note you need luacov installed)

## run test with coverage enabled from zigbee-switch/src
lua -lluacov test/test_firstled_switch.lua
## create the report from the stats output
luacov -c ./luacov.stats.out --config ~/SmartThingsEdgeDrivers/tools/config.luacov

This will create a luacov.report.out with counts next to each line in each file that was required during the lua script. Here is what I was seeing relating to the firstled-io/init.lua file:

       local function get_child_profile_name(device)
*****0   for _, fingerprint in ipairs(FINGERPRINTS) do
*****0     if device:get_model() == fingerprint.model then
*****0       return fingerprint.child_profile
           end
         end
       end

       local function find_child(parent, ep_id)
    23   return parent:get_child_by_parent_assigned_key(string.format("%02X", ep_id))
       end

       local function device_added(driver, device)
         -- Only create children for the actual Zigbee device and not the children
     8   if device.network_type == device_lib.NETWORK_TYPE_ZIGBEE then
     1     local children_amount = get_children_amount(device)
     1     if children_amount >= 2 then
     4       for i = 2, children_amount, 1 do
     3         if find_child(device, i) == nil then
*****0           local name = string.format("%s%d", string.sub(device.label, 0, -2), i)
*****0           local child_profile = get_child_profile_name(device)
*****0           local metadata = {
                   type = "EDGE_CHILD",
                   label = name,
                   profile = child_profile,
                   parent_device_id = device.id,
                   parent_assigned_child_key = string.format("%02X", i),
*****0             vendor_provided_label = name
                 }
*****0           driver:try_create_device(metadata)
               end
             end
           end

         -- Create Button if necessary
     1     local button_amount = get_button_amount(device)
     1     if button_amount >= 1 then
     5       for i = children_amount + 1, children_amount + button_amount, 1 do
     4         if find_child(device, i) == nil then
*****0           local name = string.format("%s%d", string.sub(device.label, 0, -2), i)
*****0           local metadata = {
                   type = "EDGE_CHILD",
                   label = name,
                   profile = "button",
                   parent_device_id = device.id,
                   parent_assigned_child_key = string.format("%02X", i),
                   vendor_provided_label = name,
                 }
*****0           driver:try_create_device(metadata)
               end
             end
           end

cjswedes · 2026-05-22T14:53:26Z

+      for i = children_amount + 1, children_amount + button_amount, 1 do
+        if find_child(device, i) == nil then
+          local name = string.format("%s%d", string.sub(device.label, 0, -2), i)
+          local metadata = {
+            type = "EDGE_CHILD",
+            label = name,
+            profile = "button",
+            parent_device_id = device.id,
+            parent_assigned_child_key = string.format("%02X", i),
+            vendor_provided_label = name,
+          }
+          driver:try_create_device(metadata)


please add tests that test this part of device added.

thinkaName force-pushed the firstled-io_M4S4BAC branch 2 times, most recently from d0d3e68 to 09fa419 Compare May 18, 2026 05:48

cbaumler requested review from aleclorimer, cjswedes and nand-nor May 18, 2026 20:59

cjswedes reviewed May 18, 2026

View reviewed changes

thinkaName force-pushed the firstled-io_M4S4BAC branch 2 times, most recently from 2b92e15 to 29ea878 Compare May 19, 2026 08:58

nand-nor reviewed May 20, 2026

View reviewed changes

Comment thread drivers/SmartThings/zigbee-switch/src/firstled-io/init.lua Outdated

thinkaName force-pushed the firstled-io_M4S4BAC branch 5 times, most recently from 541f830 to 59e191b Compare May 21, 2026 09:56

cjswedes reviewed May 21, 2026

View reviewed changes

thinkaName force-pushed the firstled-io_M4S4BAC branch 4 times, most recently from ab102dd to 927cc97 Compare May 22, 2026 08:31

add firstled-io_M4S4BAC

90c7984

thinkaName force-pushed the firstled-io_M4S4BAC branch from 927cc97 to 90c7984 Compare May 22, 2026 10:01

cjswedes reviewed May 22, 2026

View reviewed changes

Conversation

thinkaName commented May 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Type of Change

Checklist

Description of Change

Summary of Completed Tests

Uh oh!

github-actions Bot commented May 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented May 14, 2026

Uh oh!

github-actions Bot commented May 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results

Uh oh!

github-actions Bot commented May 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cjswedes left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cjswedes May 18, 2026

Choose a reason for hiding this comment

Uh oh!

cjswedes May 19, 2026

Choose a reason for hiding this comment

Uh oh!

thinkaName May 20, 2026

Choose a reason for hiding this comment

Uh oh!

nand-nor commented May 20, 2026

Uh oh!

Uh oh!

thinkaName commented May 21, 2026

Uh oh!

cjswedes May 21, 2026

Choose a reason for hiding this comment

Uh oh!

cjswedes commented May 21, 2026

Uh oh!

cjswedes May 22, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

thinkaName commented May 14, 2026 •

edited

Loading

github-actions Bot commented May 14, 2026 •

edited

Loading

github-actions Bot commented May 14, 2026 •

edited

Loading

github-actions Bot commented May 14, 2026 •

edited

Loading