Issue #13999: Resolve pitest suppression for skipHtmlComment() method for TagParser

AmitKumarDeoghoria · romani · commit d4025b26159e · 2025-03-02T05:29:50.000-08:00
diff --git a/config/pitest-suppressions/pitest-javadoc-suppressions.xml b/config/pitest-suppressions/pitest-javadoc-suppressions.xml
@@ -98,13 +98,4 @@
     <description>removed conditional - replaced equality check with true</description>
     <lineContent>return astType == TokenTypes.VARIABLE_DEF</lineContent>
   </mutation>
-
-  <mutation unstable="false">
-    <sourceFile>TagParser.java</sourceFile>
-    <mutatedClass>com.puppycrawl.tools.checkstyle.checks.javadoc.TagParser</mutatedClass>
-    <mutatedMethod>skipHtmlComment</mutatedMethod>
-    <mutator>org.pitest.mutationtest.engine.gregor.mutators.experimental.ArgumentPropagationMutator</mutator>
-    <description>replaced call to com/puppycrawl/tools/checkstyle/checks/javadoc/TagParser::findChar with argument</description>
-    <lineContent>toPoint = findChar(text, &apos;&gt;&apos;, getNextPoint(text, toPoint));</lineContent>
-  </mutation>
 </suppressedMutations>
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/javadoc/TagParser.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/javadoc/TagParser.java
@@ -214,7 +214,7 @@ private static Point skipHtmlComment(String[] text, Point fromPoint) {
         Point toPoint = fromPoint;
         while (toPoint.getLineNo() < text.length && !text[toPoint.getLineNo()]
                 .substring(0, toPoint.getColumnNo() + 1).endsWith("-->")) {
-            toPoint = findChar(text, '>', getNextPoint(text, toPoint));
+            toPoint = getNextPoint(text, toPoint);
         }
         return toPoint;
     }

Original file line number	Diff line number	Diff line change
`@@ -214,7 +214,7 @@ private static Point skipHtmlComment(String[] text, Point fromPoint) {`
`214`	`214`	`Point toPoint = fromPoint;`
`215`	`215`	`while (toPoint.getLineNo() < text.length && !text[toPoint.getLineNo()]`
`216`	`216`	`.substring(0, toPoint.getColumnNo() + 1).endsWith("-->")) {`
`217`		`- toPoint = findChar(text, '>', getNextPoint(text, toPoint));`
	`217`	`+ toPoint = getNextPoint(text, toPoint);`
`218`	`218`	`}`
`219`	`219`	`return toPoint;`
`220`	`220`	`}`