polhenarejos / pico-fido

Transforming a Raspberry Pico into a FIDO key
GNU General Public License v3.0
206 stars 24 forks source link

Tests not working from build #45

Open AlyssaRogers opened 1 month ago

AlyssaRogers commented 1 month ago

We downloaded the uf2 file from the release page onto our pico, and it seems to function as a key properly when registering the key with google. However, when running the tests, the key fails a lot of them. Are the tests all meant to pass when using the release image? Are the tests testing special features that need to be enabled?

  1. What are the tests testing for?
  2. Do they normally fail with the code built for a non-w pico?
  3. Some of the tests are encountering errors with the fido2 errors or errors with setting up the tests. Is there a dependency we are missing or a flag that needs to be set? image
polhenarejos commented 1 month ago
  1. Tests are designed to test FIDO and other funcionalities as well to ensure proper CD/CI.
  2. Tests should not fail (except for phy u2f, which might fail due to OS timing aspects).
  3. Tests shall be executed sequentially. Do not execute specific tests because prior tests might be designed to reset the memory onto a clean state, like seems happening in the screenshot. Avoid using -k flag and let start from the beginning.