Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

survived line in pitest.sh pitest-java-ast-visitor #10755

Closed
romani opened this issue Aug 29, 2021 · 1 comment · Fixed by #10761
Closed

survived line in pitest.sh pitest-java-ast-visitor #10755

romani opened this issue Aug 29, 2021 · 1 comment · Fixed by #10761
Labels
approved miscellaneous
Milestone
9.0.1

Comments

@romani
Copy link
Member

romani commented Aug 29, 2021

pitest failure reproduced on local and in PR #10753 (comment) :

$ ./.ci/pitest.sh pitest-java-ast-visitor
....
Actual:
JavaAstVisitor.java.html:<td class='covered'><pre><span  class='survived'>        while (firstExpression instanceof JavaLanguageParser.BinOpContext) {</span></pre></td></tr>
Ignore:

Diff:
--- target/ignored.txt	2021-08-29 07:44:37.558409722 -0700
+++ target/actual.txt	2021-08-29 07:44:37.558409722 -0700
@@ -0,0 +1 @@
+JavaAstVisitor.java.html:<td class='covered'><pre>
<span  class='survived'>        while (firstExpression instanceof JavaLanguageParser.BinOpContext) {</span></pre></td></tr>
Difference between 'Actual' and 'Ignore' lists is detected, lists should be equal.
build will be failed.
@romani
Copy link
Member Author

romani commented Aug 29, 2021

@nmancus1 , please take a look if you have time

romani added a commit to romani/checkstyle that referenced this issue Aug 29, 2021
@romani
infra: disable pitest pitest-report-java-ast-visitor until checkstyle…
5232fa9 
…#10755
@romani romani mentioned this issue Aug 29, 2021
Merged
romani added a commit that referenced this issue Aug 29, 2021
@romani
infra: disable pitest pitest-report-java-ast-visitor until #10755
7c5900d
nrmancuso added a commit to nrmancuso/checkstyle that referenced this issue Aug 30, 2021
@nrmancuso
Issue checkstyle#10755: survived line in pitest.sh pitest-java-ast-vi…
f18b001 
…sitor
@nrmancuso nrmancuso mentioned this issue Aug 30, 2021
Merged
romani pushed a commit that referenced this issue Sep 1, 2021
@nrmancuso @romani
Issue #10755: survived line in pitest.sh pitest-java-ast-visitor
9801c73
@romani romani added this to the 9.1 milestone Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
approved miscellaneous
Projects
None yet
Milestone
9.0.1
Development

Successfully merging a pull request may close this issue.

Issue #10755: survived line in pitest.sh pitest-java-ast-visitor nrmancuso/checkstyle
1 participant
@romani