fix(coverage): lcov avoid double reporting for line hit

[grandizzy]

Motivation

The line hit is reported doubled in lcov report. That can be repro with a contract

contract AContract {
    int256 public i;

    function foo() public {
        i = 1;
    }
}

and test

contract AContractTest is Test {
    function testFoo() public {
        AContract a = new AContract();
        a.foo();
    }
}

running
forge coverage --report lcov && genhtml lcov.info -o report && firefox report/index.html will generate report below

That happens because the generated lcov report contains DA:8,1 line 2 times (DA: line number, hit count)

Solution

• avoid adding line hit count twice (as a line and also as a statement)

[DaniPopes]

Needs snapbox assertions

[grandizzy]

Needs snapbox assertions

not sure how to do that since this one is LCOV report related only (that is to make sure certain line is not written twice in lcov.info file), do you have any suggestion? thanks

[DaniPopes]

Just replace assert_eq! with assert_data_eq!, and use str![] for the string literal

[grandizzy]

Just replace assert_eq! with assert_data_eq!, and use str![] for the string literal

done ba4d759 thank you!