Skip to content
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 TestClusterMultiFabric to run on chip-repl based yamltest #24925

Conversation

tehampson
Copy link
Contributor

Problem: Test is incorrect as it is sending and expecting out of range enum values. While today the accessory is sending out enum values that are out of range, after #24368 is resolve this test was going to start failing anyway. For that reason I have chosen to modify the test to not send/receive out of range enum values.

@github-actions
Copy link

github-actions bot commented Feb 8, 2023

PR #24925: Size comparison from fd3217a to d055faa

Decreases (1 build for cc32xx)
platform target config section fd3217a d055faa change % change
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20194648 20194647 -1 -0.0
Full report (4 builds for cc32xx, k32w)
platform target config section fd3217a d055faa change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 640233 640233 0 0.0
(read/write) 204084 204084 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197488 197488 0 0.0
.comment 194 194 0 0.0
.data 1476 1476 0 0.0
.debug_abbrev 928439 928439 0 0.0
.debug_aranges 87352 87352 0 0.0
.debug_frame 299840 299840 0 0.0
.debug_info 20194648 20194647 -1 -0.0
.debug_line 2649781 2649781 0 0.0
.debug_loc 2785870 2785870 0 0.0
.debug_ranges 280720 280720 0 0.0
.debug_str 3001804 3001804 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105585 105585 0 0.0
.shstrtab 232 232 0 0.0
.stab 204 204 0 0.0
.stabstr 441 441 0 0.0
.stack 2048 2048 0 0.0
.strtab 375840 375840 0 0.0
.symtab 255856 255856 0 0.0
.text 532524 532524 0 0.0
k32w contact k32w0+release (read/write) 660108 660108 0 0.0
.bss 76560 76560 0 0.0
.data 2112 2112 0 0.0
.text 562324 562324 0 0.0
light k32w0+release (read/write) 672368 672368 0 0.0
.bss 74344 74344 0 0.0
.data 2064 2064 0 0.0
.text 593232 593232 0 0.0
lock k32w0+release (read/write) 631760 631760 0 0.0
.bss 75112 75112 0 0.0
.data 2084 2084 0 0.0
.text 551836 551836 0 0.0

@bzbarsky-apple bzbarsky-apple merged commit 738d33c into project-chip:master Feb 9, 2023
lpbeliveau-silabs pushed a commit to lpbeliveau-silabs/connectedhomeip that referenced this pull request Feb 13, 2023
lecndav pushed a commit to lecndav/connectedhomeip that referenced this pull request Mar 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants