Limit the lenght of User Name

[HunsupJung]

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]

Channel deleted.

[github-actions]

Test Results

   72 files    492 suites   0s ⏱️
2 689 tests 2 689 ✅ 0 💤 0 ❌
4 550 runs  4 550 ✅ 0 💤 0 ❌

Results for commit bf23f18.

♻️ This comment has been updated with latest results.

[github-actions]

File	Coverage
All files	72%	❌
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-lock/src/new-matter-lock/can_handle.lua	90%	✅
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-lock/src/new-matter-lock/init.lua	64%	❌
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-lock/src/lock_utils.lua	98%	✅
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-lock/src/lazy_load_subdriver.lua	57%	❌
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-lock/src/init.lua	92%	✅

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against bf23f18

[hcarter-775]

Suggested change

	local userNameMatter = userName
	if #userNameMatter > NAME_MAX_L then
	userNameMatter = string.sub(userNameMatter, 1, NAME_MAX_L)
	end
	local userName = string.sub(command.args.userName, 1, NAME_MAX_L)

the sub will only limit the userName input if it is too long, so no need for the extra check, or the extra variable

[HunsupJung]

userNameMatter needs to be created for Matter. userName is used for lockUsers capability and the limitation of length is different.

[hcarter-775]

Approving but unit tests must be fixed before merging