coverallsapp / coverage-reporter / 31848710948

95%

push

github

feat: LuaCov support (#190)

* spec(fixtures): add LuaCov files

* spec: add LuaCov context to parser_spec

* spec: add luacov_parser_spec

* feat: add a parser to LuaCov reports

* refactor: add LuaCovParser to the list of available parsers

* lint: exclude cyclomatic complexity for LuaCov

* docs(README): include LuaCov

* fix: use a descriptive block parameter in `luacov_parser_spec`

* fix(luacov): drop separator line, fix mislabeled fixture, guard matches?

Three fixes on top of #190:

1. LuaCov's DefaultReporter writes a blank line after each file section
   (`on_end_file`), which the parser counted as a source line. Every file
   report therefore carried one spurious trailing `nil` -- a 19-line
   test.lua produced a 20-entry coverage array. Drop that separator when
   closing a section. A source file that genuinely ends in a blank line
   keeps its entry, because LuaCov emits both. The specs now also assert
   coverage.size against the real line count of the fixture, so this
   cannot silently regress.

2. spec/fixtures/luacov/second-header-content-empty/luacov.report.out was
   byte-identical to first-header-content-empty: the blanked line was
   line 2, the *first* header. The test passed without ever exercising
   the second header. Rebuilt from the valid single-file report with only
   line 25 ("Summary") blanked, so it now differs from a passing fixture
   in exactly the property under test.

3. `matches?` lacked the `rescue Exception` guard every other
   content-reading parser has (clover, cobertura, jacoco, golang,
   coveralls, coveragepy). LuaCovParser is registered ahead of those, so
   an unreadable file that used to be skipped could raise out of
   Parser#parse_file.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: James Kessler <afinetooth@gmail.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>

64 of 66 new or added lines in 2 files covered. (96.97%)

1013 of 1073 relevant lines covered (94.41%)

2.11 hits per line