The parameterized test excludesLints in src/test/java/org/eolang/lints/WithoutLintsTest.java:34 builds a WithoutLints over a single fake lint named ascii-only (line 41) and asks it to exclude unit-test-missing or unit-test-without-live-file (line 42). Neither name is in the list, so the assertion at line 44 only checks that ascii-only is not equal to the excluded name. That holds whatever WithoutLints does: if its filter handed the origin back untouched, the test would still be green.
The fixture drifted over several refactors, the last one in 235cda2d swapping in LtFake("ascii-only"), while the names in @ValueSource stayed the same.
Putting new WithoutLintsTest.LtFake(lid) into the fixture next to another fake, and asserting that lid is not among the remaining names, makes the test fail when exclusion stops working.
@yegor256
The parameterized test
excludesLintsinsrc/test/java/org/eolang/lints/WithoutLintsTest.java:34builds aWithoutLintsover a single fake lint namedascii-only(line 41) and asks it to excludeunit-test-missingorunit-test-without-live-file(line 42). Neither name is in the list, so the assertion at line 44 only checks thatascii-onlyis not equal to the excluded name. That holds whateverWithoutLintsdoes: if its filter handed the origin back untouched, the test would still be green.The fixture drifted over several refactors, the last one in
235cda2dswapping inLtFake("ascii-only"), while the names in@ValueSourcestayed the same.Putting
new WithoutLintsTest.LtFake(lid)into the fixture next to another fake, and asserting thatlidis not among the remaining names, makes the test fail when exclusion stops working.@yegor256