Add test coverage measurement with 95% threshold to CI#8
Merged
thomaspatzke merged 5 commits intomainfrom Feb 27, 2026
Merged
Conversation
Co-authored-by: thomaspatzke <1845601+thomaspatzke@users.noreply.github.com>
Add tests for uncovered lines across condition.rs, types.rs, parser.rs (via lib.rs public API), and pipeline.rs: - condition.rs: trailing tokens, reserved words as identifiers, all of them, error after of, unexpected token/end-of-expression errors - types.rs: Display impls for Status, Level, SigmaString, SigmaValue, ConditionExpression; has_special_parts, as_plain, From<String>; serde deserialization for float and bool - parser.rs (via lib.rs): get_string_list single string, value_as_string for numbers, relation types, correlation types, sigma string wildcards and escapes, float detection values, expand modifier, search identifier errors, condition list errors, related entry errors, all known keys, alias errors, correlation condition errors, correlation rule fields - pipeline.rs: unknown transformation type, MapList variants for all transformations, change_logsource service, set_state, empty var error, logsource service condition Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: thomaspatzke <1845601+thomaspatzke@users.noreply.github.com>
Co-authored-by: thomaspatzke <1845601+thomaspatzke@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add test coverage measurement in GitHub Actions CI
Add test coverage measurement with 95% threshold to CI
Feb 26, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Integrates
cargo-tarpaulininto the GitHub Actions CI pipeline, failing builds when line coverage drops below 95%.CI workflow
cargo install cargo-tarpaulinandcargo tarpaulin --fail-under 95steps to.github/workflows/test.ymlTest coverage (77.4% → 95.3%)
Added 73 tests to reach the threshold:
condition.rs— Edge cases: reserved words as identifiers, trailing tokens,all of them, error pathstypes.rs—Displayimpls,SigmaStringmethods,SigmaValueserde round-tripslib.rs— Parser coverage via public API: all relation/correlation types, wildcard/escape parsing, expand modifier, all field accessors, error pathspipeline.rs—MapListvariants across all transformations, service conditions, placeholder edge casesNo production code was modified.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.