fix new capabilities unit test

pegor-karoglanian · pegor-karoglanian · commit 62e48ca20635 · 2025-12-22T15:11:54.000-08:00
diff --git a/drivers/SmartThings/zigbee-lock/src/test/test_zigbee_lock_new_capabilities.lua b/drivers/SmartThings/zigbee-lock/src/test/test_zigbee_lock_new_capabilities.lua
@@ -26,6 +26,7 @@ local DoorLockUserType = DoorLock.types.DrlkUserType
 
 local test_credential_index = 1
 local test_credentials = {}
+local test_users = {}
 local mock_device = test.mock_device.build_test_zigbee_device(
     {
         profile = t_utils.get_profile_definition("base-lock.yml"),
@@ -37,6 +38,7 @@ zigbee_test_utils.prepare_zigbee_env_info()
 local function test_init_new_capabilities()
     test_credential_index = 1
     test_credentials = {}
+    test_users = {}
     test.mock_device.add_test_device(mock_device)
 end
 
@@ -82,27 +84,27 @@ local function add_default_users()
             {
                 capability = capabilities.lockUsers.ID,
                 command = "addUser",
-                args = { "TestUser" .. i, "guest" }
+                args = { "Guest" .. i, "guest" }
             },
         })
         -- add to the user list that is now expected
-        user_list["user"..i] = {userIndex = i, userType = "guest", userName = "TestUser" .. i }
+        table.insert(user_list, {userIndex = i, userType = "guest", userName = "Guest" .. i })
 
         test.socket.capability:__expect_send(
             mock_device:generate_test_message(
                 "main",
                 capabilities.lockUsers.users(
                     user_list,
-                    { visibility = { displayed = false } }
+                    { visibility = { displayed = true } }
                 )
             )
         )
         test.socket.capability:__expect_send(
             mock_device:generate_test_message(
                 "main",
                 capabilities.lockUsers.commandResult(
-                    { commandName = "addUser", statusCode = "success" },
-                    { state_change = true, visibility = { displayed = false } }
+                    { commandName = "addUser", statusCode = "success", userIndex = i},
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )
@@ -150,19 +152,26 @@ test.socket.capability:__queue_receive({
             )
         }
     )
-    table.insert(test_credentials, { userIndex = tonumber(user_index), credentialIndex = test_credential_index, credentialType = "pin" })
+    table.insert(test_credentials, { userIndex = test_credential_index, credentialIndex = test_credential_index, credentialType = "pin" })
+    table.insert(test_users, {userIndex = test_credential_index, userName = "Guest" .. test_credential_index, userType = "guest"})
     test.socket.capability:__expect_send(
         mock_device:generate_test_message(
             "main",
-            capabilities.lockCredentials.credentials(test_credentials, { visibility = { displayed = false } })
+            capabilities.lockUsers.users(test_users, { state_change = true, visibility = { displayed = true } })
+        )
+    )
+    test.socket.capability:__expect_send(
+        mock_device:generate_test_message(
+            "main",
+            capabilities.lockCredentials.credentials(test_credentials, { state_change = true, visibility = { displayed = true } })
         )
     )
     test.socket.capability:__expect_send(
         mock_device:generate_test_message(
             "main",
             capabilities.lockCredentials.commandResult(
-                { commandName = "addCredential", statusCode = "success" },
-                { state_change = true, visibility = { displayed = false } }
+                { commandName = "addCredential", statusCode = "success", credentialIndex = test_credential_index, userIndex = test_credential_index },
+                { state_change = true, visibility = { displayed = true } }
             )
         )
     )
@@ -194,7 +203,7 @@ test.register_coroutine_test(
                 "main",
                 capabilities.lockUsers.commandResult(
                     { commandName = "addUser", statusCode = "resourceExhausted" },
-                    { state_change = true, visibility = { displayed = false } }
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )
@@ -219,23 +228,24 @@ test.register_coroutine_test(
             },
         })
 
-        local users = {}
-        users["user" .. 1] = { userIndex = 1, userName = "TestUser1", userType = "guest" }
-        users["user" .. 2] = { userIndex = 2, userName = "ChangeUserName", userType = "guest" }
-        users["user" .. 3] = { userIndex = 3, userName = "TestUser3", userType = "guest" }
-        users["user" .. 4] = { userIndex = 4, userName = "TestUser4", userType = "guest" }
+        local users = {
+            { userIndex = 1, userName = "Guest1", userType = "guest" },
+            { userIndex = 2, userName = "ChangeUserName", userType = "guest" },
+            { userIndex = 3, userName = "Guest3", userType = "guest" },
+            { userIndex = 4, userName = "Guest4", userType = "guest" },
+        }
         test.socket.capability:__expect_send(
             mock_device:generate_test_message(
                 "main",
-                capabilities.lockUsers.users(users, { visibility = { displayed = false } })
+                capabilities.lockUsers.users(users, { state_change=true, visibility = { displayed = true } })
             )
         )
         test.socket.capability:__expect_send(
             mock_device:generate_test_message(
                 "main",
                 capabilities.lockUsers.commandResult(
-                    { commandName = "updateUser", statusCode = "success" },
-                    { state_change = true, visibility = { displayed = false } }
+                    { commandName = "updateUser", statusCode = "success", userIndex = 2 },
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )
@@ -254,7 +264,7 @@ test.register_coroutine_test(
                 "main",
                 capabilities.lockUsers.commandResult(
                     { commandName = "updateUser", statusCode = "failure" },
-                    { state_change = true, visibility = { displayed = false } }
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )
@@ -279,23 +289,24 @@ test.register_coroutine_test(
             },
         })
         
-        local users = {}
-        users["user" .. 1] = { userIndex = 1, userName = "TestUser1", userType = "guest" }
-        users["user" .. 2] = { userIndex = 2, userName = "TestUser2", userType = "guest" }
-        users["user" .. 4] = { userIndex = 4, userName = "TestUser4", userType = "guest" }
+        local users = {
+            { userIndex = 1, userName = "Guest1", userType = "guest" },
+            { userIndex = 2, userName = "Guest2", userType = "guest" },
+            { userIndex = 4, userName = "Guest4", userType = "guest" },
+        }
 
         test.socket.capability:__expect_send(
             mock_device:generate_test_message(
                 "main",
-                capabilities.lockUsers.users(users, { visibility = { displayed = false } })
+                capabilities.lockUsers.users(users, { state_change=true, visibility = { displayed = true } })
             )
         )
         test.socket.capability:__expect_send(
             mock_device:generate_test_message(
                 "main",
                 capabilities.lockUsers.commandResult(
-                    { commandName = "deleteUser", statusCode = "success" },
-                    { state_change = true, visibility = { displayed = false } }
+                    { commandName = "deleteUser", statusCode = "success", userIndex = 3 },
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )
@@ -314,7 +325,7 @@ test.register_coroutine_test(
                 "main",
                 capabilities.lockUsers.commandResult(
                     { commandName = "deleteUser", statusCode = "failure" },
-                    { state_change = true, visibility = { displayed = false } }
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )
@@ -326,31 +337,31 @@ test.register_coroutine_test(
     "addCredential command received and commandResult is success",
     function()
         init_migration()
-        add_credential("2", "abc123")
+        add_credential(0, "abc123")
     end
 )
 
 test.register_coroutine_test(
     "updateCredential command received and commandResult is success",
     function()
         init_migration()
-        add_credential("2", "abc123")
+        add_credential(0, "abc123")
 
-        -- try to update the wrong credentialIndex first and expect a failure
+        -- try to update the wrong credentialIndex (4) first and expect a failure
         test.socket.capability:__queue_receive({
             mock_device.id,
             {
                 capability = capabilities.lockCredentials.ID,
                 command = "updateCredential",
-                args = { "2", "4", "pin", "abc123" }
+                args = { "4", "4", "pin", "abc123" }
             },
         })
         test.socket.capability:__expect_send(
             mock_device:generate_test_message(
                 "main",
                 capabilities.lockCredentials.commandResult(
                     { commandName = "updateCredential", statusCode = "failure" },
-                    { state_change = true, visibility = { displayed = false } }
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )
@@ -362,7 +373,7 @@ test.register_coroutine_test(
             {
                 capability = capabilities.lockCredentials.ID,
                 command = "updateCredential",
-                args = { "2", "1", "pin", "changedPin123" }
+                args = { "1", "1", "pin", "changedPin123" }
             },
         })
         test.socket.zigbee:__expect_send(
@@ -397,23 +408,34 @@ test.register_coroutine_test(
                 )
             }
         )
+        test.socket.capability:__expect_send(
+            mock_device:generate_test_message(
+                "main",
+                capabilities.lockUsers.users(
+                    {
+                        { userIndex = 1, userType = "guest", userName = "Guest1" }
+                    },
+                    { state_change = true, visibility = { displayed = true } }
+                )
+            )
+        )
         test.socket.capability:__expect_send(
             mock_device:generate_test_message(
                 "main",
                 capabilities.lockCredentials.credentials(
                     {
-                        { userIndex = 2, credentialIndex = 1, credentialType = "pin" }
+                        { userIndex = 1, credentialIndex = 1, credentialType = "pin" }
                     },
-                    { visibility = { displayed = false } }
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )
         test.socket.capability:__expect_send(
             mock_device:generate_test_message(
                 "main",
                 capabilities.lockCredentials.commandResult(
-                    { commandName = "updateCredential", statusCode = "success" },
-                    { state_change = true, visibility = { displayed = false } }
+                    { commandName = "updateCredential", statusCode = "success", credentialIndex = 1, userIndex = 1 },
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )
@@ -425,9 +447,9 @@ test.register_coroutine_test(
     "deleteCredential command received and commandResult is success",
     function()
         init_migration()
-        add_credential("2", "abc123")
-        add_credential("1", "test123")
-        add_credential("3", "321test")
+        add_credential(0, "abc123")
+        add_credential(0, "test123")
+        add_credential(0, "321test")
 
         -- try to delete credential with wrong index and expect a failure
         test.socket.capability:__queue_receive({
@@ -443,7 +465,7 @@ test.register_coroutine_test(
                 "main",
                 capabilities.lockCredentials.commandResult(
                     { commandName = "deleteCredential", statusCode = "failure" },
-                    { state_change = true, visibility = { displayed = false } }
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )
@@ -483,24 +505,36 @@ test.register_coroutine_test(
                 ""
             )
         })
+        test.socket.capability:__expect_send(
+            mock_device:generate_test_message(
+                "main",
+                capabilities.lockUsers.users(
+                    {
+                        { userIndex = 2, userType = "guest", userName = "Guest2" },
+                        { userIndex = 3, userType = "guest", userName = "Guest3" }
+                    },
+                    { state_change = true, visibility = { displayed = true } }
+                )
+            )
+        )
         test.socket.capability:__expect_send(
             mock_device:generate_test_message(
                 "main",
                 capabilities.lockCredentials.credentials(
                     {
-                        { userIndex = 1, credentialIndex = 2, credentialType = "pin" },
+                        { userIndex = 2, credentialIndex = 2, credentialType = "pin" },
                         { userIndex = 3, credentialIndex = 3, credentialType = "pin" }
                     },
-                    { visibility = { displayed = false } }
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )
         test.socket.capability:__expect_send(
             mock_device:generate_test_message(
                 "main",
                 capabilities.lockCredentials.commandResult(
-                    { commandName = "deleteCredential", statusCode = "success" },
-                    { state_change = true, visibility = { displayed = false } }
+                    { commandName = "deleteCredential", statusCode = "success", credentialIndex = 1, userIndex = 1 },
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )
@@ -512,9 +546,9 @@ test.register_coroutine_test(
     "deleteAllCredentials command received and commandResult is success",
     function()
         init_migration()
-        add_credential("2", "abc123")
-        add_credential("1", "test123")
-        add_credential("3", "321test")
+        add_credential(0, "abc123")
+        add_credential(0, "test123")
+        add_credential(0, "321test")
 
         test.socket.capability:__queue_receive({
             mock_device.id,
@@ -547,15 +581,27 @@ test.register_coroutine_test(
                 ""
             )
         })
+        test.socket.capability:__expect_send(
+            mock_device:generate_test_message(
+                "main",
+                capabilities.lockUsers.users(
+                    {
+                        { userIndex = 2, userType = "guest", userName = "Guest2" },
+                        { userIndex = 3, userType = "guest", userName = "Guest3" }
+                    },
+                    { state_change = true, visibility = { displayed = true } }
+                )
+            )
+        )
         test.socket.capability:__expect_send(
             mock_device:generate_test_message(
                 "main",
                 capabilities.lockCredentials.credentials(
                     {
-                        { userIndex = 1, credentialIndex = 2, credentialType = "pin" },
+                        { userIndex = 2, credentialIndex = 2, credentialType = "pin" },
                         { userIndex = 3, credentialIndex = 3, credentialType = "pin" }
                     },
-                    { visibility = { displayed = false } }
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )
@@ -564,7 +610,7 @@ test.register_coroutine_test(
                 "main",
                 capabilities.lockCredentials.commandResult(
                     { commandName = "deleteAllCredentials", statusCode = "success" },
-                    { state_change = true, visibility = { displayed = false } }
+                    { state_change = true, visibility = { displayed = true } }
                 )
             )
         )

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,7 @@ local DoorLockUserType = DoorLock.types.DrlkUserType`
`26`	`26`
`27`	`27`	`local test_credential_index = 1`
`28`	`28`	`local test_credentials = {}`
	`29`	`+local test_users = {}`
`29`	`30`	`local mock_device = test.mock_device.build_test_zigbee_device(`
`30`	`31`	`{`
`31`	`32`	`profile = t_utils.get_profile_definition("base-lock.yml"),`
`@@ -37,6 +38,7 @@ zigbee_test_utils.prepare_zigbee_env_info()`
`37`	`38`	`local function test_init_new_capabilities()`
`38`	`39`	`test_credential_index = 1`
`39`	`40`	`test_credentials = {}`
	`41`	`+ test_users = {}`
`40`	`42`	`test.mock_device.add_test_device(mock_device)`
`41`	`43`	`end`
`42`	`44`
`@@ -82,27 +84,27 @@ local function add_default_users()`
`82`	`84`	`{`
`83`	`85`	`capability = capabilities.lockUsers.ID,`
`84`	`86`	`command = "addUser",`
`85`		`- args = { "TestUser" .. i, "guest" }`
	`87`	`+ args = { "Guest" .. i, "guest" }`
`86`	`88`	`},`
`87`	`89`	`})`
`88`	`90`	`-- add to the user list that is now expected`
`89`		`- user_list["user"..i] = {userIndex = i, userType = "guest", userName = "TestUser" .. i }`
	`91`	`+ table.insert(user_list, {userIndex = i, userType = "guest", userName = "Guest" .. i })`
`90`	`92`
`91`	`93`	`test.socket.capability:__expect_send(`
`92`	`94`	`mock_device:generate_test_message(`
`93`	`95`	`"main",`
`94`	`96`	`capabilities.lockUsers.users(`
`95`	`97`	`user_list,`
`96`		`- { visibility = { displayed = false } }`
	`98`	`+ { visibility = { displayed = true } }`
`97`	`99`	`)`
`98`	`100`	`)`
`99`	`101`	`)`
`100`	`102`	`test.socket.capability:__expect_send(`
`101`	`103`	`mock_device:generate_test_message(`
`102`	`104`	`"main",`
`103`	`105`	`capabilities.lockUsers.commandResult(`
`104`		`- { commandName = "addUser", statusCode = "success" },`
`105`		`- { state_change = true, visibility = { displayed = false } }`
	`106`	`+ { commandName = "addUser", statusCode = "success", userIndex = i},`
	`107`	`+ { state_change = true, visibility = { displayed = true } }`
`106`	`108`	`)`
`107`	`109`	`)`
`108`	`110`	`)`
`@@ -150,19 +152,26 @@ test.socket.capability:__queue_receive({`
`150`	`152`	`)`
`151`	`153`	`}`
`152`	`154`	`)`
`153`		`- table.insert(test_credentials, { userIndex = tonumber(user_index), credentialIndex = test_credential_index, credentialType = "pin" })`
	`155`	`+ table.insert(test_credentials, { userIndex = test_credential_index, credentialIndex = test_credential_index, credentialType = "pin" })`
	`156`	`+ table.insert(test_users, {userIndex = test_credential_index, userName = "Guest" .. test_credential_index, userType = "guest"})`
`154`	`157`	`test.socket.capability:__expect_send(`
`155`	`158`	`mock_device:generate_test_message(`
`156`	`159`	`"main",`
`157`		`- capabilities.lockCredentials.credentials(test_credentials, { visibility = { displayed = false } })`
	`160`	`+ capabilities.lockUsers.users(test_users, { state_change = true, visibility = { displayed = true } })`
	`161`	`+ )`
	`162`	`+ )`
	`163`	`+ test.socket.capability:__expect_send(`
	`164`	`+ mock_device:generate_test_message(`
	`165`	`+ "main",`
	`166`	`+ capabilities.lockCredentials.credentials(test_credentials, { state_change = true, visibility = { displayed = true } })`
`158`	`167`	`)`
`159`	`168`	`)`
`160`	`169`	`test.socket.capability:__expect_send(`
`161`	`170`	`mock_device:generate_test_message(`
`162`	`171`	`"main",`
`163`	`172`	`capabilities.lockCredentials.commandResult(`
`164`		`- { commandName = "addCredential", statusCode = "success" },`
`165`		`- { state_change = true, visibility = { displayed = false } }`
	`173`	`+ { commandName = "addCredential", statusCode = "success", credentialIndex = test_credential_index, userIndex = test_credential_index },`
	`174`	`+ { state_change = true, visibility = { displayed = true } }`
`166`	`175`	`)`
`167`	`176`	`)`
`168`	`177`	`)`
`@@ -194,7 +203,7 @@ test.register_coroutine_test(`
`194`	`203`	`"main",`
`195`	`204`	`capabilities.lockUsers.commandResult(`
`196`	`205`	`{ commandName = "addUser", statusCode = "resourceExhausted" },`
`197`		`- { state_change = true, visibility = { displayed = false } }`
	`206`	`+ { state_change = true, visibility = { displayed = true } }`
`198`	`207`	`)`
`199`	`208`	`)`
`200`	`209`	`)`
`@@ -219,23 +228,24 @@ test.register_coroutine_test(`
`219`	`228`	`},`
`220`	`229`	`})`
`221`	`230`
`222`		`- local users = {}`
`223`		`- users["user" .. 1] = { userIndex = 1, userName = "TestUser1", userType = "guest" }`
`224`		`- users["user" .. 2] = { userIndex = 2, userName = "ChangeUserName", userType = "guest" }`
`225`		`- users["user" .. 3] = { userIndex = 3, userName = "TestUser3", userType = "guest" }`
`226`		`- users["user" .. 4] = { userIndex = 4, userName = "TestUser4", userType = "guest" }`
	`231`	`+ local users = {`
	`232`	`+ { userIndex = 1, userName = "Guest1", userType = "guest" },`
	`233`	`+ { userIndex = 2, userName = "ChangeUserName", userType = "guest" },`
	`234`	`+ { userIndex = 3, userName = "Guest3", userType = "guest" },`
	`235`	`+ { userIndex = 4, userName = "Guest4", userType = "guest" },`
	`236`	`+ }`
`227`	`237`	`test.socket.capability:__expect_send(`
`228`	`238`	`mock_device:generate_test_message(`
`229`	`239`	`"main",`
`230`		`- capabilities.lockUsers.users(users, { visibility = { displayed = false } })`
	`240`	`+ capabilities.lockUsers.users(users, { state_change=true, visibility = { displayed = true } })`
`231`	`241`	`)`
`232`	`242`	`)`
`233`	`243`	`test.socket.capability:__expect_send(`
`234`	`244`	`mock_device:generate_test_message(`
`235`	`245`	`"main",`
`236`	`246`	`capabilities.lockUsers.commandResult(`
`237`		`- { commandName = "updateUser", statusCode = "success" },`
`238`		`- { state_change = true, visibility = { displayed = false } }`
	`247`	`+ { commandName = "updateUser", statusCode = "success", userIndex = 2 },`
	`248`	`+ { state_change = true, visibility = { displayed = true } }`
`239`	`249`	`)`
`240`	`250`	`)`
`241`	`251`	`)`
`@@ -254,7 +264,7 @@ test.register_coroutine_test(`
`254`	`264`	`"main",`
`255`	`265`	`capabilities.lockUsers.commandResult(`
`256`	`266`	`{ commandName = "updateUser", statusCode = "failure" },`
`257`		`- { state_change = true, visibility = { displayed = false } }`
	`267`	`+ { state_change = true, visibility = { displayed = true } }`
`258`	`268`	`)`
`259`	`269`	`)`
`260`	`270`	`)`
`@@ -279,23 +289,24 @@ test.register_coroutine_test(`
`279`	`289`	`},`
`280`	`290`	`})`
`281`	`291`
`282`		`- local users = {}`
`283`		`- users["user" .. 1] = { userIndex = 1, userName = "TestUser1", userType = "guest" }`
`284`		`- users["user" .. 2] = { userIndex = 2, userName = "TestUser2", userType = "guest" }`
`285`		`- users["user" .. 4] = { userIndex = 4, userName = "TestUser4", userType = "guest" }`
	`292`	`+ local users = {`
	`293`	`+ { userIndex = 1, userName = "Guest1", userType = "guest" },`
	`294`	`+ { userIndex = 2, userName = "Guest2", userType = "guest" },`
	`295`	`+ { userIndex = 4, userName = "Guest4", userType = "guest" },`
	`296`	`+ }`
`286`	`297`
`287`	`298`	`test.socket.capability:__expect_send(`
`288`	`299`	`mock_device:generate_test_message(`
`289`	`300`	`"main",`
`290`		`- capabilities.lockUsers.users(users, { visibility = { displayed = false } })`
	`301`	`+ capabilities.lockUsers.users(users, { state_change=true, visibility = { displayed = true } })`
`291`	`302`	`)`
`292`	`303`	`)`
`293`	`304`	`test.socket.capability:__expect_send(`
`294`	`305`	`mock_device:generate_test_message(`
`295`	`306`	`"main",`
`296`	`307`	`capabilities.lockUsers.commandResult(`
`297`		`- { commandName = "deleteUser", statusCode = "success" },`
`298`		`- { state_change = true, visibility = { displayed = false } }`
	`308`	`+ { commandName = "deleteUser", statusCode = "success", userIndex = 3 },`
	`309`	`+ { state_change = true, visibility = { displayed = true } }`
`299`	`310`	`)`
`300`	`311`	`)`
`301`	`312`	`)`
`@@ -314,7 +325,7 @@ test.register_coroutine_test(`
`314`	`325`	`"main",`
`315`	`326`	`capabilities.lockUsers.commandResult(`
`316`	`327`	`{ commandName = "deleteUser", statusCode = "failure" },`
`317`		`- { state_change = true, visibility = { displayed = false } }`
	`328`	`+ { state_change = true, visibility = { displayed = true } }`
`318`	`329`	`)`
`319`	`330`	`)`
`320`	`331`	`)`
`@@ -326,31 +337,31 @@ test.register_coroutine_test(`
`326`	`337`	`"addCredential command received and commandResult is success",`
`327`	`338`	`function()`
`328`	`339`	`init_migration()`
`329`		`- add_credential("2", "abc123")`
	`340`	`+ add_credential(0, "abc123")`
`330`	`341`	`end`
`331`	`342`	`)`
`332`	`343`
`333`	`344`	`test.register_coroutine_test(`
`334`	`345`	`"updateCredential command received and commandResult is success",`
`335`	`346`	`function()`
`336`	`347`	`init_migration()`
`337`		`- add_credential("2", "abc123")`
	`348`	`+ add_credential(0, "abc123")`
`338`	`349`
`339`		`- -- try to update the wrong credentialIndex first and expect a failure`
	`350`	`+ -- try to update the wrong credentialIndex (4) first and expect a failure`
`340`	`351`	`test.socket.capability:__queue_receive({`
`341`	`352`	`mock_device.id,`
`342`	`353`	`{`
`343`	`354`	`capability = capabilities.lockCredentials.ID,`
`344`	`355`	`command = "updateCredential",`
`345`		`- args = { "2", "4", "pin", "abc123" }`
	`356`	`+ args = { "4", "4", "pin", "abc123" }`
`346`	`357`	`},`
`347`	`358`	`})`
`348`	`359`	`test.socket.capability:__expect_send(`
`349`	`360`	`mock_device:generate_test_message(`
`350`	`361`	`"main",`
`351`	`362`	`capabilities.lockCredentials.commandResult(`
`352`	`363`	`{ commandName = "updateCredential", statusCode = "failure" },`
`353`		`- { state_change = true, visibility = { displayed = false } }`
	`364`	`+ { state_change = true, visibility = { displayed = true } }`
`354`	`365`	`)`
`355`	`366`	`)`
`356`	`367`	`)`
`@@ -362,7 +373,7 @@ test.register_coroutine_test(`
`362`	`373`	`{`
`363`	`374`	`capability = capabilities.lockCredentials.ID,`
`364`	`375`	`command = "updateCredential",`
`365`		`- args = { "2", "1", "pin", "changedPin123" }`
	`376`	`+ args = { "1", "1", "pin", "changedPin123" }`
`366`	`377`	`},`
`367`	`378`	`})`
`368`	`379`	`test.socket.zigbee:__expect_send(`
`@@ -397,23 +408,34 @@ test.register_coroutine_test(`
`397`	`408`	`)`
`398`	`409`	`}`
`399`	`410`	`)`
	`411`	`+ test.socket.capability:__expect_send(`
	`412`	`+ mock_device:generate_test_message(`
	`413`	`+ "main",`
	`414`	`+ capabilities.lockUsers.users(`
	`415`	`+ {`
	`416`	`+ { userIndex = 1, userType = "guest", userName = "Guest1" }`
	`417`	`+ },`
	`418`	`+ { state_change = true, visibility = { displayed = true } }`
	`419`	`+ )`
	`420`	`+ )`
	`421`	`+ )`
`400`	`422`	`test.socket.capability:__expect_send(`
`401`	`423`	`mock_device:generate_test_message(`
`402`	`424`	`"main",`
`403`	`425`	`capabilities.lockCredentials.credentials(`
`404`	`426`	`{`
`405`		`- { userIndex = 2, credentialIndex = 1, credentialType = "pin" }`
	`427`	`+ { userIndex = 1, credentialIndex = 1, credentialType = "pin" }`
`406`	`428`	`},`
`407`		`- { visibility = { displayed = false } }`
	`429`	`+ { state_change = true, visibility = { displayed = true } }`
`408`	`430`	`)`
`409`	`431`	`)`
`410`	`432`	`)`
`411`	`433`	`test.socket.capability:__expect_send(`
`412`	`434`	`mock_device:generate_test_message(`
`413`	`435`	`"main",`
`414`	`436`	`capabilities.lockCredentials.commandResult(`
`415`		`- { commandName = "updateCredential", statusCode = "success" },`
`416`		`- { state_change = true, visibility = { displayed = false } }`
	`437`	`+ { commandName = "updateCredential", statusCode = "success", credentialIndex = 1, userIndex = 1 },`
	`438`	`+ { state_change = true, visibility = { displayed = true } }`
`417`	`439`	`)`
`418`	`440`	`)`
`419`	`441`	`)`
`@@ -425,9 +447,9 @@ test.register_coroutine_test(`
`425`	`447`	`"deleteCredential command received and commandResult is success",`
`426`	`448`	`function()`
`427`	`449`	`init_migration()`
`428`		`- add_credential("2", "abc123")`
`429`		`- add_credential("1", "test123")`
`430`		`- add_credential("3", "321test")`
	`450`	`+ add_credential(0, "abc123")`
	`451`	`+ add_credential(0, "test123")`
	`452`	`+ add_credential(0, "321test")`
`431`	`453`
`432`	`454`	`-- try to delete credential with wrong index and expect a failure`
`433`	`455`	`test.socket.capability:__queue_receive({`
`@@ -443,7 +465,7 @@ test.register_coroutine_test(`
`443`	`465`	`"main",`
`444`	`466`	`capabilities.lockCredentials.commandResult(`
`445`	`467`	`{ commandName = "deleteCredential", statusCode = "failure" },`
`446`		`- { state_change = true, visibility = { displayed = false } }`
	`468`	`+ { state_change = true, visibility = { displayed = true } }`
`447`	`469`	`)`
`448`	`470`	`)`
`449`	`471`	`)`
`@@ -483,24 +505,36 @@ test.register_coroutine_test(`
`483`	`505`	`""`
`484`	`506`	`)`
`485`	`507`	`})`
	`508`	`+ test.socket.capability:__expect_send(`
	`509`	`+ mock_device:generate_test_message(`
	`510`	`+ "main",`
	`511`	`+ capabilities.lockUsers.users(`
	`512`	`+ {`
	`513`	`+ { userIndex = 2, userType = "guest", userName = "Guest2" },`
	`514`	`+ { userIndex = 3, userType = "guest", userName = "Guest3" }`
	`515`	`+ },`
	`516`	`+ { state_change = true, visibility = { displayed = true } }`
	`517`	`+ )`
	`518`	`+ )`
	`519`	`+ )`
`486`	`520`	`test.socket.capability:__expect_send(`
`487`	`521`	`mock_device:generate_test_message(`
`488`	`522`	`"main",`
`489`	`523`	`capabilities.lockCredentials.credentials(`
`490`	`524`	`{`
`491`		`- { userIndex = 1, credentialIndex = 2, credentialType = "pin" },`
	`525`	`+ { userIndex = 2, credentialIndex = 2, credentialType = "pin" },`
`492`	`526`	`{ userIndex = 3, credentialIndex = 3, credentialType = "pin" }`
`493`	`527`	`},`
`494`		`- { visibility = { displayed = false } }`
	`528`	`+ { state_change = true, visibility = { displayed = true } }`
`495`	`529`	`)`
`496`	`530`	`)`
`497`	`531`	`)`
`498`	`532`	`test.socket.capability:__expect_send(`
`499`	`533`	`mock_device:generate_test_message(`
`500`	`534`	`"main",`
`501`	`535`	`capabilities.lockCredentials.commandResult(`
`502`		`- { commandName = "deleteCredential", statusCode = "success" },`
`503`		`- { state_change = true, visibility = { displayed = false } }`
	`536`	`+ { commandName = "deleteCredential", statusCode = "success", credentialIndex = 1, userIndex = 1 },`
	`537`	`+ { state_change = true, visibility = { displayed = true } }`
`504`	`538`	`)`
`505`	`539`	`)`
`506`	`540`	`)`
`@@ -512,9 +546,9 @@ test.register_coroutine_test(`
`512`	`546`	`"deleteAllCredentials command received and commandResult is success",`
`513`	`547`	`function()`
`514`	`548`	`init_migration()`
`515`		`- add_credential("2", "abc123")`
`516`		`- add_credential("1", "test123")`
`517`		`- add_credential("3", "321test")`
	`549`	`+ add_credential(0, "abc123")`
	`550`	`+ add_credential(0, "test123")`
	`551`	`+ add_credential(0, "321test")`
`518`	`552`
`519`	`553`	`test.socket.capability:__queue_receive({`
`520`	`554`	`mock_device.id,`
`@@ -547,15 +581,27 @@ test.register_coroutine_test(`
`547`	`581`	`""`
`548`	`582`	`)`
`549`	`583`	`})`
	`584`	`+ test.socket.capability:__expect_send(`
	`585`	`+ mock_device:generate_test_message(`
	`586`	`+ "main",`
	`587`	`+ capabilities.lockUsers.users(`
	`588`	`+ {`
	`589`	`+ { userIndex = 2, userType = "guest", userName = "Guest2" },`
	`590`	`+ { userIndex = 3, userType = "guest", userName = "Guest3" }`
	`591`	`+ },`
	`592`	`+ { state_change = true, visibility = { displayed = true } }`
	`593`	`+ )`
	`594`	`+ )`
	`595`	`+ )`
`550`	`596`	`test.socket.capability:__expect_send(`
`551`	`597`	`mock_device:generate_test_message(`
`552`	`598`	`"main",`
`553`	`599`	`capabilities.lockCredentials.credentials(`
`554`	`600`	`{`
`555`		`- { userIndex = 1, credentialIndex = 2, credentialType = "pin" },`
	`601`	`+ { userIndex = 2, credentialIndex = 2, credentialType = "pin" },`
`556`	`602`	`{ userIndex = 3, credentialIndex = 3, credentialType = "pin" }`
`557`	`603`	`},`
`558`		`- { visibility = { displayed = false } }`
	`604`	`+ { state_change = true, visibility = { displayed = true } }`
`559`	`605`	`)`
`560`	`606`	`)`
`561`	`607`	`)`
`@@ -564,7 +610,7 @@ test.register_coroutine_test(`
`564`	`610`	`"main",`
`565`	`611`	`capabilities.lockCredentials.commandResult(`
`566`	`612`	`{ commandName = "deleteAllCredentials", statusCode = "success" },`
`567`		`- { state_change = true, visibility = { displayed = false } }`
	`613`	`+ { state_change = true, visibility = { displayed = true } }`
`568`	`614`	`)`
`569`	`615`	`)`
`570`	`616`	`)`