parsingdata / metal

A Java library for parsing binary data formats, using declarative descriptions.
Apache License 2.0
18 stars 9 forks source link

Test all constructors that do not contain generics in AutoEqualityTest. #409

Closed mvanaken closed 9 months ago

mvanaken commented 9 months ago

Ok, het mysterie is ontrafelt!

GitHub: Je build is groen! Codecov: Maar geen 100% coverage, je mist equality checks! IntelliJ: Onzin! Die worden gewoon gecheckt! 100%, behalve dan de Source interface. CodeCov: Source interface? Die wordt veelvuldig gebruikt en getest waar heb je het over? Die equalities zijn het probleem! IntelliJ: AutoEqualityTest runt gewoon volledig! Ik snap niet waar je het over hebt! Waarom denk jij dat ze missen? CodeCov: Weet ik veel! Ze missen gewoon!

que Maven

Maven: Oh, releasen?! Nee, je AutoEqualityTest faalt... Je moet een mapping toevoegen voor Map object. CodeCov: Ha! IntelliJ: Huh? Maar de build is groen?! GitHub: Laat mij erbuiten! IntelliJ: En ik heb helemaal geen Map object nodig om de volledig test te kunnen runnen. Java geeft mij geen enkele constructor terug met een Map parameter!

que Java

Java: You called? IntelliJ: Welke constructors ben je wanneer aan het geven? Java: Oh, gewoon in een volgorde die ik leuk vind! IntelliJ: ... Java: Niet goed? CodeCov: Ik heb dus gelijk dat equality checks missen! :D IntelliJ: Je kon alleen niet zeggen waarom! CodeCov: Dat is mijn werk niet! IntelliJ: ...

THE END

Let's get our 100% coverage back!

codecov[bot] commented 9 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ddf0add) 92.31% compared to head (6cc8d51) 100.00%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #409 +/- ## ============================================= + Coverage 92.31% 100.00% +7.68% - Complexity 1014 1153 +139 ============================================= Files 99 99 Lines 1536 1536 Branches 157 157 ============================================= + Hits 1418 1536 +118 + Misses 89 0 -89 + Partials 29 0 -29 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.