diff options
| author | Bjorn Svensson <bjorn.a.svensson@est.tech> | 2022-10-14 17:11:40 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-10-14 08:11:40 -0700 |
| commit | 3b15a04b5939fc85f5e134ed09eb5c9e1584bef1 (patch) | |
| tree | 742c529f0cdb34615ce7dd2a1085fcc34f955b6d /Makefile | |
| parent | c245df9fb92fdb51b8255dc90a60eaf03b9846a0 (diff) | |
Fixup of PR734: Coverage of hiredis.c (#1124)
Improve coverage (#734)
* Remove duplicate tests
- double covered by:
"Can parse RESP3 doubles"
- bool covered via:
"Can parse RESP3 bool"
* Make (connect) timeout in test config general
* Set error string in Unix connect with invalid timeout
Restructure testcase since redisConnectWithTimeout() and
redisConnectUnixWithTimeout() now behaves similar.
* Use quiet flag in lcov/genhtml instead of piping to /dev/null
* Fixup of redisCommandArgv test case
* Update test case to match what it covers
Use new test case info text since the previous one seemed copy&pasted.
The sought coverage was the handling of the parent-chaining
for a double object, which the test case now focuses on.
Co-authored-by: Ariel <ashtul@gmail.com>
Diffstat (limited to 'Makefile')
| -rw-r--r-- | Makefile | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -338,7 +338,8 @@ coverage: gcov make check mkdir -p tmp/lcov lcov -d . -c --exclude '/usr*' -o tmp/lcov/hiredis.info - genhtml --legend -o tmp/lcov/report tmp/lcov/hiredis.info + lcov -q -l tmp/lcov/hiredis.info + genhtml --legend -q -o tmp/lcov/report tmp/lcov/hiredis.info noopt: $(MAKE) OPTIMIZATION="" |