-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Fix various typos in various tests #24493
Fix various typos in various tests #24493
Conversation
PR #24493: Size comparison from 7c933bd to 2871d2d Increases (6 builds for cc13x2_26x2, esp32, psoc6, telink)
Decreases (12 builds for bl602, bl702, cc13x2_26x2, nrfconnect, psoc6, qpg, telink)
Full report (54 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
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.
Looks good, but needs a regen; looks like this enables some test coverage that used to not exist!
Yes ! This is the third time I found such occurrences of typos that were not handled and results into more coverage when updated. |
154f50f
to
fb7ab3e
Compare
PR #24493: Size comparison from a722c18 to fb7ab3e Increases (10 builds for bl602, bl702, cc13x2_26x2, linux, psoc6, telink)
Decreases (3 builds for cc13x2_26x2, telink)
Full report (49 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
fb7ab3e
to
bdb7031
Compare
PR #24493: Size comparison from e48fe6a to bdb7031 Increases (3 builds for esp32, linux)
Decreases (8 builds for bl702, cyw30739, psoc6, telink)
Full report (54 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
* Fix various typos in various tests * Update generated tests Co-authored-by: Andrei Litvin <[email protected]>
Problem
Using the
matter_yamltests
framework I found various typos in various tests.Some are just uppercase/lowercase confusion but some others are non-existing arguments or
constraints
put at the wrong place.