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

overlord/fdestate: mock amd64 architecture in unit tests #14999

Conversation

bboozzoo
Copy link
Contributor

Mock the architecture so that boot assets can be properly identified even when running unit tests on ARM64.

@bboozzoo bboozzoo added Simple 😃 A small PR which can be reviewed quickly FDE Manager Pull requests that target FDE manager branch labels Jan 29, 2025
@pedronis pedronis force-pushed the fde-manager-features branch from 7005c64 to 25f9119 Compare January 29, 2025 10:49
Mock the architecture so that boot assets can be properly identified
even when running unit tests on ARM64.

Signed-off-by: Maciej Borzecki <[email protected]>
@bboozzoo bboozzoo force-pushed the bboozzoo/fde-unit-test-on-arm-assets branch from 5c115c4 to 1a98f00 Compare January 29, 2025 12:28
Copy link

codecov bot commented Jan 29, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.85%. Comparing base (25f9119) to head (1a98f00).
Report is 2 commits behind head on fde-manager-features.

Additional details and impacted files
@@                   Coverage Diff                    @@
##           fde-manager-features   #14999      +/-   ##
========================================================
- Coverage                 78.20%   77.85%   -0.35%     
========================================================
  Files                      1171     1175       +4     
  Lines                    155885   155856      -29     
========================================================
- Hits                     121903   121344     -559     
- Misses                    26450    26993     +543     
+ Partials                   7532     7519      -13     
Flag Coverage Δ
unittests 77.85% <ø> (-0.35%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

github-actions bot commented Jan 29, 2025

Wed Jan 29 15:46:10 UTC 2025
The following results are from: https://github.com/canonical/snapd/actions/runs/13031318219

Failures:

Preparing:

  • openstack:debian-sid-64

Restoring:

  • openstack:debian-sid-64:tests/unit/
  • openstack:debian-12-64:tests/unit/
  • openstack:debian-sid-64:tests/unit/

@bboozzoo bboozzoo merged commit 74f68d8 into canonical:fde-manager-features Jan 30, 2025
65 of 67 checks passed
@bboozzoo bboozzoo deleted the bboozzoo/fde-unit-test-on-arm-assets branch January 30, 2025 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FDE Manager Pull requests that target FDE manager branch Simple 😃 A small PR which can be reviewed quickly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants