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

Cherry-pick to master [rom_ext_e2e] Add an ownership transfer test #25668

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

cfrantz
Copy link
Contributor

@cfrantz cfrantz commented Dec 16, 2024

This is a manual cherry-pick of #24419.

  • Create a library of helper functions for facilitating ownership transfer tests.
  • Create a basic ownership transfer test that transfers chip ownership from the default fake test owner to a dummy owner.

1. Enhance the boot-svc commands to retrieve and display the response.
2. Emit any boot-svc response `status` as hex.

Signed-off-by: Chris Frantz <[email protected]>
(cherry picked from commit 4c593f5)
(cherry picked from commit fea62b3)
The `dummy` owner is a secondary owner used for testing ownership
transfer flows.

Signed-off-by: Chris Frantz <[email protected]>
(cherry picked from commit e5c2899)
(cherry picked from commit ba2766f)
@cfrantz cfrantz requested a review from a team as a code owner December 16, 2024 18:36
@cfrantz cfrantz removed the request for review from a team December 16, 2024 18:36
Signed-off-by: Chris Frantz <[email protected]>
(cherry picked from commit 5ecb3fb)
(cherry picked from commit 3837b32)
1. Create a library of helper functions for facilitating ownership
   transfer tests.
2. Create a basic ownership transfer test that transfers chip ownership
   from the default `fake` test owner to a `dummy` owner.

Signed-off-by: Chris Frantz <[email protected]>
(cherry picked from commit 0ad99c9)
(cherry picked from commit 71cc612)
@cfrantz cfrantz force-pushed the port-24911-to-master branch from 9eb3f60 to 784047f Compare December 16, 2024 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants