-
Notifications
You must be signed in to change notification settings - Fork 348
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
Migrating to the new API #1283
Comments
Yes, the interface as changed a lot and the ClassInfo class has been largely deprecated. You should have all the data you need to determine the number of covered lines for the Set<ClassLine> getCoveredLines(ClassName clazz); Method. How were you using ClassInfo? |
@hcoles thanks for the answer. That's the main part https://github.com/pitest/pitclipse/blob/0b1f2926d97fb0b92f3c79dd31f4667823d287c5/bundles/org.pitest.pitclipse.runner/src/org/pitest/pitclipse/runner/results/summary/SummaryResultListener.java#L63 where we are using Concerning We are also using |
Hi
In Pitclipse, https://github.com/pitest/pitclipse, we need to update to a more recent version of pit to support Java 21 (pitest/pitclipse#221). We were using 1.6.8 and moving to 1.15.3 leads to compilation errors. I seem to understand this is due to 95e8102 which changes the API of
CoverageDatabase
(we were using its methodsgetNumberOfCoveredLines
andgetClassInfo
, which now are gone), which we used to retrieve fromListenerArguments.getCoverage
, which now returns aReportCoverage
.Is there any suggested way to migrate to the new API?
thanks in advance
The text was updated successfully, but these errors were encountered: