search

STAMP-project / test-runner

test driver to run JUnit tests isolated in a new JVM
GNU General Public License v3.0
16 stars 15 forks source link

Support multiple classes directories #103

Closed andre15silva closed 3 years ago

andre15silva commented 3 years ago

Right now, test-runner assumes only two directories are passed in the targetProjectClasses, with the first one being the source classes dir, and the second the test classes dir.

https://github.com/STAMP-project/test-runner/blob/90d7a84f119a865f6d232de49ddd8dfe0f5f7289/src/main/java/eu/stamp_project/testrunner/runner/coverage/JUnit4JacocoRunnerCoveredResultPerTestMethod.java#L45-L47

We should also support several classes directories, possibly through a different API which separates source and test classes in the arguments.

danglotb commented 3 years ago

Hi @andre15silva,

Is there any real use case for this? I do not see why one would like to have multiple classes and test-classes directories.

andre15silva commented 3 years ago

Hi @danglotb ,

https://github.com/SpoonLabs/astor supports several directories for both application and test binaries. That is why I want to add this feature, so that flacoco can be integrated in astor without any issue coming up.

I do agree that this is probably a very rare case, but supporting I'd argue that supporting it doesn't impact anything else.

tdurieux commented 3 years ago

Hi @andre15silva,

Is there any real use case for this? I do not see why one would like to have multiple classes and test-classes directories.

It is the case in multi-module projects.