quic-go / qpack

a (minimal) QPACK (RFC 9204) implementation in Go
MIT License
29 stars 8 forks source link

don't reuse the encoder in the integration tests #18

Closed marten-seemann closed 4 years ago

marten-seemann commented 4 years ago

On first use, the encoder emits the required insert count. When comparing the lengths between a value encoded with and without using the static table, we need to make sure that both of them include the same overhead.

codecov-commenter commented 4 years ago

Codecov Report

Merging #18 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #18   +/-   ##
=======================================
  Coverage   91.42%   91.42%           
=======================================
  Files           5        5           
  Lines         233      233           
=======================================
  Hits          213      213           
  Misses         11       11           
  Partials        9        9           

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 f89b93a...ae9d209. Read the comment docs.