Skip to content

Classloader test detection#82

Merged
martinezmatias merged 3 commits intoASSERT-KTH:masterfrom
andre15silva:classloader-test-detection
Aug 13, 2021
Merged

Classloader test detection#82
martinezmatias merged 3 commits intoASSERT-KTH:masterfrom
andre15silva:classloader-test-detection

Conversation

@andre15silva
Copy link
Copy Markdown
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

Signed-off-by: André Silva <andre15andre@hotmail.com>
Signed-off-by: André Silva <andre15andre@hotmail.com>
Signed-off-by: André Silva <andre15andre@hotmail.com>
@andre15silva andre15silva marked this pull request as ready for review August 13, 2021 09:11
@andre15silva
Copy link
Copy Markdown
Member Author

Ready for review

@andre15silva andre15silva changed the title Test detection by scanning classes Classloader test detection Aug 13, 2021
@andre15silva
Copy link
Copy Markdown
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
Copy Markdown
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Optimize flacoco Test detection missing tests Detecting lang63 tests

2 participants