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