-
Notifications
You must be signed in to change notification settings - Fork 507
WWSTCERT-7636/7639 Frient humidity sensor (hmszb 120/hmszb 110) add support #2356
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WWSTCERT-7636/7639 Frient humidity sensor (hmszb 120/hmszb 110) add support #2356
Conversation
…(HMSZB-120/HMSZB-110)---add-support
Duplicate profile check: Passed - no duplicate profiles detected. |
Test Results 69 files 449 suites 0s ⏱️ Results for commit b8c288c. ♻️ This comment has been updated with latest results. |
Minimum allowed coverage is Generated by 🐒 cobertura-action against b8c288c |
device:configure() | ||
device.thread:call_with_delay(5, function() | ||
device:refresh() | ||
end) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
whitespace
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
3600, | ||
humiditySensitivity | ||
) | ||
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd also expect a refresh after 5s here, right? Can you incorporate that into the test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added.
…(HMSZB-120/HMSZB-110)---add-support
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because of updates to the lua libraries, you'll need to add this expected message to tests where you handle the temperature measurement cluster in order for tests to pass
…(HMSZB-120/HMSZB-110)---add-support
Check all that apply
Type of Change
Checklist
Description of Change
Summary of Completed Tests