openwallet-foundation / acapy-vc-authn-oidc / 16503292227

89%

push

github

Connection based verification (#802)

* Initial POC of connection based verification

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* Applied black formatting

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* Removed assertion to ensure we didn't use oob

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* Added problem reports on failed connections

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* Add connection verification testing

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

1. Updated timestamps in models to use UTC for consistency across the application.
2. Added comprehensive tests for ACA-Py webhook handler, covering various scenarios including connection status changes, presentation request handling, and error reporting.
3. Introduced new dependencies in the `pyproject.toml` for HTTP handling in tests.
4. Improved mocking strategies in test cases for better isolation and reliability.

These changes ensure better adherence to time standards and robustness in handling connection-based verification's and related testing.

* Improve test coverage

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* Use multi_use keyword rather than ephemeral and only delete multi_use auth_sessions

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* Remove NEW comments and removed depreciated create_invitation

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

- Removed commented-out code for the depreciated create_invitation method
- Changed comments in the acapy_handler and oidc router modules to remove "NEW" indicators for clarification.

* Fix: removed "pending-" prefix and error reporting with invi_msg_id

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

- Removed the hardcoded "pending-" prefix from the session ID lookup when searching by `pres_exch_id` in `acapy_handler.py`, allowing for more flexible ID matching.
- ... (continued)

164 of 207 new or added lines in 7 files covered. (79.23%)

1017 of 1131 relevant lines covered (89.92%)

0.9 hits per line