pitest / pitclipse

Mutation testing for Java in Eclipse IDE. Based on PIT (Pitest).
https://pitest.org
Apache License 2.0
59 stars 17 forks source link

Prevent the dependencies of users' projects from messing up with Pitest #108

Closed echebbi closed 4 years ago

echebbi commented 4 years ago

Closes #74 by putting Pitest's JARs first in the classpath, and project's dependencies second. I don't know exactly why it solves the issue, but according to my tests it works as expected.

Important: I'm going to merge this PR & release a new version so that users can test whether the fix works for them as well. If it does, we really need to add a new test where Pitest is run on a project that depends on ASM to prevent future regressions.

Note: this solution has been inspired by this issue in Pitest's repository

codecov[bot] commented 4 years ago

Codecov Report

Merging #108 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #108   +/-   ##
=========================================
  Coverage     84.86%   84.86%           
  Complexity      298      298           
=========================================
  Files            45       45           
  Lines          1242     1242           
  Branches         88       88           
=========================================
  Hits           1054     1054           
  Misses          127      127           
  Partials         61       61

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update cc79b0f...90951d7. Read the comment docs.

sonarcloud[bot] commented 4 years ago

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication