SONARJAVA-5534 Remove SE engine and unused duplicated check test sources #5176
Conversation
its/plugin/tests/src/test/java/com/sonar/it/java/suite/PackageInfoTest.java
Outdated
Show resolved
Hide resolved
alban-auzeill
force-pushed
the
alban-remix/remove-unused-test-sources
branch
from
2cc820d to
eff2529
Compare
alban-auzeill
changed the title
tomasz-tylenda-sonarsource
left a comment
tomasz-tylenda-sonarsource
left a comment
There was a problem hiding this comment.
LGTM with a minor comments about unnecessary changes to new lines at the end of a few files.
We discussed the change offline. I verified that everything removed looks related to symbolic execution.
| "hasTruePositives": true, | ||
| "falseNegatives": 7, | ||
| "falsePositives": 0 | ||
| } |
There was a problem hiding this comment.
The only change here is a new line at the end of the file. We should avoid such changes as part of this PR (they are okay if there are other changes to the file).
| "hasTruePositives": true, | ||
| "falseNegatives": 4, | ||
| "falsePositives": 0 | ||
| } |
There was a problem hiding this comment.
No change.
| "hasTruePositives": false, | ||
| "falseNegatives": 0, | ||
| "falsePositives": 0 | ||
| } |
There was a problem hiding this comment.
No change
| "hasTruePositives": false, | ||
| "falseNegatives": 0, | ||
| "falsePositives": 0 | ||
| } |
There was a problem hiding this comment.
No change
| "hasTruePositives": false, | ||
| "falseNegatives": 0, | ||
| "falsePositives": 0 | ||
| } |
There was a problem hiding this comment.
No change
|
3 participants
No description provided.