Nitrokey / pynitrokey

Python client for Nitrokey devices
Apache License 2.0
94 stars 28 forks source link

nkpk test: do not skip the status test #505

Closed sosthene-nitrokey closed 4 months ago

sosthene-nitrokey commented 4 months ago

niitropy nkpk test skipped the status test when it shouldn't

Changes

Checklist

Make sure to run make check and make fix before creating a PR, otherwise the CI will fail.

Test Environment and Execution

Relevant Output Example

found 1 Nitrokey Passkey device(s):
- Nitrokey Passkey at /dev/hidraw8

Running tests for Nitrokey Passkey at /dev/hidraw8

[1/4]   uuid        UUID query                  SUCCESS     850E04AF076D2AA60000000000000000
[2/4]   version     Firmware version query      SUCCESS     v1.0.0
[3/4]   status      Device status               SKIPPED     
Please press the touch button on the device ...
Please press the touch button on the device ...
[4/4]   fido2       FIDO2                       SUCCESS     

4 tests, 3 successful, 1 skipped, 0 failed

Fixes https://github.com/Nitrokey/pynitrokey/issues/504