STAMP-project/pitmp-maven-plugin

-1 in codeline

martinch-kth opened this issue · 1 comments

martinch-kth commented

Set the codeline to -1 of what is now.

Example:
https://github.com/martinch-kth/commons-codec/blob/trunk/src/main/java/org/apache/commons/codec/language/DoubleMetaphone.java#L782

781 private boolean conditionC0(final String value, final int index) {
782 if (contains(value, index, 4, "CHIA")) {

So the links should go to the line of the method signature.

Characteristics

  • Issue Type: [bug, feature, test report]
  • Reproducibility: [always, sometimes, random]
  • Severity: [feature, minor, major, crash, block]
  • Tool/Service/Component: [name, version]
  • Execution Environment: [platform, OS, etc]
  • Reporter: [name, mail]

Description

Steps to reproduce

Other files and URLs

Relationships

Help on issue template

Preview to follow the link or open file .github/ISSUE_DOC.md

oscarlvp commented

This is related to the way PIT handles the mutation information internally and ow it is reported. During mutation, the actual location expands to all the lines in the method, that is, instead of the first line we actually tell PIT that all the lines in the method has been mutated. However, PIT reports only the first line involved in the mutation. I prefer to highlight all the lines inside the method instead of only the signature. It might be the case that the signature expands to several lines, so -1 will not work for all cases. We have been searching for a while now, how to improve the report without breaking the compatibility with PIT.

  • 👍1