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

Improve Darwin OTA tests. #25923

Merged

Conversation

bzbarsky-apple
Copy link
Contributor

  • Make it possible to hook all the delegate callbacks in the test.
  • Add a test for responding BUSY and then being able to handle the retry, disabled for now until we make it possible for it to run quickly.

* Make it possible to hook all the delegate callbacks in the test.
* Add a test for responding BUSY and then being able to handle the retry,
  disabled for now until we make it possible for it to run quickly.
@github-actions
Copy link

PR #25923: Size comparison from 12072b3 to 2e9383d

Full report (1 build for cc32xx)
platform target config section 12072b3 2e9383d change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 645825 645825 0 0.0
(read/write) 203848 203848 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197248 197248 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 933102 933102 0 0.0
.debug_aranges 87704 87704 0 0.0
.debug_frame 301596 301596 0 0.0
.debug_info 20310331 20310331 0 0.0
.debug_line 2681125 2681125 0 0.0
.debug_loc 2827568 2827568 0 0.0
.debug_ranges 286448 286448 0 0.0
.debug_str 3041144 3041144 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105953 105953 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 380543 380543 0 0.0
.symtab 257472 257472 0 0.0
.text 537752 537752 0 0.0

@bzbarsky-apple
Copy link
Contributor Author

Fast-tracking platform-specific change with platform owner review.

@bzbarsky-apple bzbarsky-apple merged commit 5b29e02 into project-chip:master Mar 31, 2023
@bzbarsky-apple bzbarsky-apple deleted the improve-darwin-ota-ci branch March 31, 2023 18:34
@MrLiuYunPing
Copy link

why is android not supported?

@MrLiuYunPing
Copy link

I want to use Android app to OTA the device, but when I announce the device with the controller noideId and Set OTA delegate, but uint16_t ep = emberAfFindClusterServerEndpointIndex(endpoint, OtaSoftwareUpdateProvider::Id); ep is always 65535, the kEmberInvalidEndpointIndex. I do't know why? iOS is OK.

@bzbarsky-apple
Copy link
Contributor Author

@MrLiuYunPing What does that have to do with a pull request that adds unit tests for the Darwin Matter.framework?

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