Hi @kpbode,

Thanks for the PR.

Supporting filtering by inclusion is definitely a feature that is missing (see #177).

However it would be better if we could support combining inclusion and exclusion.
A test would be executed if:

• it matches any of the inclusion pattern or if no inclusion patterns are defined or if
• it do not match any of the defined exclusion patterns

For instance user may want to configure like this:

include=com\.mycompany\.mypackage1\.*
include=com\.mycompany\.mypackage2\.*
exclude=.*IT
exclude=.*SlowTest

This would execute only tests contained in packages com.searchy.mypackage1 or com.searchy.mypackage2 but excluding the tests ending with IT or SlowTest.

RegexFileFilter should be mdified handle to handle those two new special line prefixes when parsing the infinitest.filters file.
If a line starts with include= the remaining line content should be considered as an inclusion pattern.
If a line starts with exclude= the remaining line content should be considered as an exclusion pattern.
For backward compatibility reason lines starting with unknown prefix will still be considered as exclusion pattern.

What do you think?

Hey @sarod,

I just updated my pull request. A filter can now contain "include" and "exclude" entries. Non-prefixed lines will result in exclusions to provide compatibility with legacy files.

Best,

Karl

Thanks @kpbode!

I did another review.
Things look good apart the 2 minor comments.

Best,
-Seb

@sarod, I integrated your remarks last night.

Thanks! @kpbode