Skip to content
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

Classloader test detection #82

Merged

Conversation

andre15silva
Copy link
Member

@andre15silva andre15silva commented Aug 12, 2021

This PR fixes #80, #71, and partly fixes #74.

I took inspiration from the test detection in nopol and adapted it to our needs. This means it now scans methods and not only classes, and also supports JUnit5.

There are still two tiny improvements that will be finished soon.

This also aims at fixing test detection in lang63.

Closes #80
Closes #71

@andre15silva andre15silva marked this pull request as ready for review August 13, 2021 09:11
@andre15silva
Copy link
Member Author

Ready for review

@andre15silva andre15silva changed the title Test detection by scanning classes Classloader test detection Aug 13, 2021
@andre15silva
Copy link
Member Author

Running flacoco on math70 with #82 reduces test detection time from ~60s to ~1s (~99% reduction)

@andre15silva andre15silva mentioned this pull request Aug 13, 2021
@martinezmatias
Copy link
Collaborator

Running flacoco on math70 with #82 reduces test detection time from ~60s to ~1s (~99% reduction)

Wowwww Amazing @andre15silva

@martinezmatias martinezmatias merged commit 2f2b59b into ASSERT-KTH:master Aug 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Optimize flacoco Test detection missing tests Detecting lang63 tests
2 participants