Resolve Pitest Issues - RedundantImportCheck

[rnveach]

Child issue of #7797 ,

checkstyle/.ci/pitest.sh

Line 90 in f9131f9

"RedundantImportCheck.java.html:<td class='covered'><pre><span class='survived'> else if (pkgName != null &#38;&#38; isFromPackage(imp.getText(), pkgName)) {</span></pre></td></tr>"

[Gaurav-Punjabi]

I'm on it.

[Gaurav-Punjabi]

Pitest report :
https://gaurav-punjabi.github.io/checkstyle/issue-7920/pit-reports/202003221458/com.puppycrawl.tools.checkstyle.checks.imports/RedundantImportCheck.java.html#org.pitest.mutationtest.report.html.SourceFile@803449a_120

Surviving mutation :
removed conditional - replaced equality check with true → SURVIVED

Hardcoded mutation :
Gaurav-Punjabi@b2a587b

Regression Report :
https://gaurav-punjabi.github.io/checkstyle/issue-7920/regression/index.html

[rnveach]

Pitest report :
https://gaurav-punjabi.github.io/checkstyle/issue-7920/pit-reports/202003221458/com.puppycrawl.tools.checkstyle.checks.imports/RedundantImportCheck.java.html#grouporg.pitest.mutationtest.report.html.SourceFile@f8857d3_120
removed conditional - replaced equality check with true → SURVIVED

Issue is not shown in pitest report. Pitest shows 100% coverage on class where our CI tells us differently. I cannot even find the line in your report that pitest is reporting, which makes me think you ran pitest on a modified branch.

Did you run pitest report on a clean checkstyle master branch?

[Gaurav-Punjabi]

Yes, I recently updated this report. Let me check.

[Gaurav-Punjabi]

Pitest report :
https://gaurav-punjabi.github.io/checkstyle/issue-7920/pit-reports/202003221458/com.puppycrawl.tools.checkstyle.checks.imports/RedundantImportCheck.java.html#grouporg.pitest.mutationtest.report.html.SourceFile@f8857d3_120
removed conditional - replaced equality check with true → SURVIVED

Issue is not shown in pitest report. Pitest shows 100% coverage on class where our CI tells us differently. I cannot even find the line in your report that pitest is reporting, which makes me think you ran pitest on a modified branch.

Did you run pitest report on a clean checkstyle master branch?

@romani Updated the report. Please check again.

[rnveach]

Pitest report and regression look good.

[strkkk]

Fix is merged