tarantool / luajit / 29917445400

93%

tarantool/master: 93%

push

github

test: add tests for the leb128 encode/decode

This patch is a follow-up to the commit
7058e9215 ("utils: introduce leb128
reader and writer").

The leb128 encoding is used inside our profilers (memprof/sysprof). But
there are no tests for this module, so its coverage is low, since only
`lj_utils_write_uleb128()` is used. Also, there is no correctness check
for this implementation.

This patch adapts the LuaVela tests [1] to cover this module. Since the
initial version of the module wasn't adapted as is, the guarded version
of the reader functions has different behaviour for zero-sized requested
reads. Hence, this case is removed for now. The readers with a requested
size are unused in our code, so this is not crucial at the moment.

[1]: https://github.com/luavela/luavela/blob/288e8a952/tests/impl/uJIT-tests-C/suite/test_leb128.c

5737 of 6058 branches covered (94.7%)

Branch coverage included in aggregate %.

21927 of 23556 relevant lines covered (93.08%)

3972606.19 hits per line