[Bug 2036406] Re: [SRU] backport mkeficapsule to jammy

[Heinrich Schuchardt]

Hello Aristo,

in your mkeficapsule.log the following tests are skipped:

test/py/tests/test_efi_capsule/test_capsule_firmware.py
test/py/tests/test_efi_capsule/test_capsule_firmware_signed.py

I guess test_capsule_firmware.py is skipped due to missing local tools.

By adding -ra as pytest argument you should see the reason for skipping:
./test/py/test.py -ra --bd sandbox --build 

See this line in test/py/tests/test_efi_capsule/conftest.py:
pytest.skip('Setup failed: %s' % exception.cmd)

When I run on my system
'make tests' passes test/py/tests/test_efi_capsule/test_capsule_firmware.py 

With CONFIG_EFI_CAPSULE_AUTHENTICATE=y added to sandbox_defconfig the package also passes
test/py/tests/test_efi_capsule/test_capsule_firmware_signed.py.

I would suggest to add the following information to debian/changelog:

"mkeficapsule with the patches applied matches U-Boot v2022.04."

Best regards

Heinrich

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to u-boot in Ubuntu.
https://bugs.launchpad.net/bugs/2036406

Title:
  [SRU] backport mkeficapsule to jammy

Status in u-boot package in Ubuntu:
  New

Bug description:
  [Impact]

  * mkeficapsule command is used to generate capsule file for updating
  firmware in u-boot

  [Test case]

  Test case 1:
  1. use mkeficapsule command to generate test_new.cap and test_old.cap, both contain U-Boot built at different time
  2. try to update the capsule file(both test_new.cap and test_old.cap)
  3. check if the U-Boot build stamp is different from previous

  Test case 2:
  1. Follow the command here(https://u-boot.readthedocs.io/en/latest/develop/testing.html#pytest-suite) to test with U-Boot sandbox, and the test_efi_capsule related files were skipped by default before or ater the code changes, the test result can be found in comment

  [Where problems could occur]

  * There is no mkeficapsule command in Jammy yet, so the regression
  risk should be low

  [Other Info]
  N/A

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/u-boot/+bug/2036406/+subscriptions