DaGeRe / peass

Tool for Performance analysis of software system
GNU Affero General Public License v3.0
10 stars 9 forks source link

Enable using coverageSelection_*.json for measurements #48

Closed stro18 closed 2 years ago

stro18 commented 2 years ago

Currently, it is not possible to use coverageSelection_*.json for measurements because it does not contain the first version.

The following exception occured when passing the file with -executionfile results/coverageSelection_tomcat.json. The version 6cf20b4a83f1e5b2ae86525b6b96389e801776b2 (mentioned in the error), was the first version

java.lang.RuntimeException: Did not find 6cf20b4a83f1e5b2ae86525b6b96389e801776b2 in given PRONTO-files! at de.dagere.peass.DependencyTestStarter.getStartVersionIndex(DependencyTestStarter.java:122) at de.dagere.peass.DependencyTestStarter.call(DependencyTestStarter.java:74) at de.dagere.peass.DependencyTestStarter.call(DependencyTestStarter.java:38) at picocli.CommandLine.executeUserObject(CommandLine.java:1953) at picocli.CommandLine.access$1300(CommandLine.java:145) at picocli.CommandLine$RunLast.executeUserObjectOfLastSubcommandWithSameParent(CommandLine.java:2358) at picocli.CommandLine$RunLast.handle(CommandLine.java:2352) at picocli.CommandLine$RunLast.handle(CommandLine.java:2314) at picocli.CommandLine$AbstractParseResultHandler.execute(CommandLine.java:2179) at picocli.CommandLine$RunLast.execute(CommandLine.java:2316) at picocli.CommandLine.execute(CommandLine.java:2078) at de.dagere.peass.PeASSMain.main(PeASSMain.java:30)

DaGeRe commented 2 years ago

Thanks for the hint. The coverage selection files where not usable for measurement for several reasons (missing initial version, missing android-info, missing url). Since https://github.com/DaGeRe/peass/commit/5a5bcb20d3c28f67779ac4198903b9214cfe40ae, this should be possible.

Can you confirm this?

stro18 commented 2 years ago

Thanks, this works now.