PrivateStorageio / ZKAPAuthorizer

a Tahoe-LAFS storage-system plugin which authorizes storage operations based on privacy-respecting tokens
10 stars 7 forks source link

Integration test against tahoe lafs #444

Closed exarkun closed 1 year ago

exarkun commented 2 years ago

Also fix all of the storage protocol integration methods.

Fixes #433

codecov[bot] commented 2 years ago

Codecov Report

Merging #444 (6eee088) into main (58995b9) will decrease coverage by 0.17%. The diff coverage is 91.57%.

:exclamation: Current head 6eee088 differs from pull request most recent head 13e2881. Consider uploading reports for the commit 13e2881 to get more accurate results

@@            Coverage Diff             @@
##             main     #444      +/-   ##
==========================================
- Coverage   95.39%   95.23%   -0.17%     
==========================================
  Files          62       64       +2     
  Lines        8021     8344     +323     
  Branches     1035     1077      +42     
==========================================
+ Hits         7652     7946     +294     
- Misses        266      284      +18     
- Partials      103      114      +11     
Impacted Files Coverage Δ
src/_zkapauthorizer/tests/fixtures.py 95.40% <ø> (ø)
src/_zkapauthorizer/_storage_server.py 93.86% <66.66%> (+0.45%) :arrow_up:
src/_zkapauthorizer/tests/strategies.py 95.93% <80.00%> (-1.17%) :arrow_down:
src/_zkapauthorizer/_storage_client.py 84.77% <87.50%> (+1.18%) :arrow_up:
src/_zkapauthorizer/tests/resources.py 90.18% <88.05%> (-5.27%) :arrow_down:
...kapauthorizer/tests/test_tahoe_lafs_integration.py 92.50% <92.50%> (ø)
src/_zkapauthorizer/_types.py 92.00% <100.00%> (+6.28%) :arrow_up:
src/_zkapauthorizer/tests/issuer.py 100.00% <100.00%> (ø)
src/_zkapauthorizer/tests/test_plugin.py 99.71% <100.00%> (ø)
src/_zkapauthorizer/tests/test_storage_protocol.py 99.24% <100.00%> (ø)

... and 4 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

exarkun commented 1 year ago

Coverage reporting is broken but CI is otherwise green. I'm satisfied that the new integration tests are exercising what they should now.