search

rnpgp / ruby-rnp

Ruby bindings for librnp
https://www.rnpgp.org
MIT License
7 stars 3 forks source link

Update tests according to more relaxed userid validation rules. #72

Closed ni4 closed 3 years ago

ni4 commented 3 years ago

Related RNP PR : https://github.com/rnpgp/rnp/pull/1531

codecov[bot] commented 3 years ago

Codecov Report

Merging #72 (0a1a1ab) into master (fedd03d) will decrease coverage by 0.33%. The diff coverage is 66.66%.

:exclamation: Current head 0a1a1ab differs from pull request most recent head 3a09749. Consider uploading reports for the commit 3a09749 to get more accurate results Impacted file tree graph

@@            Coverage Diff             @@
##           master      #72      +/-   ##
==========================================
- Coverage   98.94%   98.60%   -0.34%     
==========================================
  Files          41       41              
  Lines        3327     3305      -22     
==========================================
- Hits         3292     3259      -33     
- Misses         35       46      +11
Impacted Files Coverage Δ
lib/rnp/misc.rb 100.00% <ø> (ø)
spec/key/properties/4BE147BB22DF1E60_spec.rb 97.29% <66.66%> (-2.71%) :arrow_down:
lib/rnp/input.rb 79.24% <0.00%> (-11.67%) :arrow_down:
lib/rnp/ffi/librnp.rb 95.55% <0.00%> (-2.24%) :arrow_down:
lib/rnp/error.rb 95.23% <0.00%> (-0.22%) :arrow_down:
lib/rnp/signature.rb 97.82% <0.00%> (-0.10%) :arrow_down:
lib/rnp/output.rb 94.91% <0.00%> (-0.09%) :arrow_down:
lib/rnp/key.rb 97.60% <0.00%> (-0.06%) :arrow_down:
lib/rnp/op/verify.rb 98.36% <0.00%> (-0.03%) :arrow_down:
lib/rnp/rnp.rb 98.41% <0.00%> (-0.03%) :arrow_down:
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update fedd03d...3a09749. Read the comment docs.

ni4 commented 3 years ago

@ronaldtse codecov patch will be corrected once PR https://github.com/rnpgp/rnp/pull/1531 is merged into master.

ni4 commented 3 years ago

@ronaldtse Thanks! So, merging with one approval.