Skip to content

Commit 1d13082

Browse files
committed
Merge resource predicates
1 parent 75506f1 commit 1d13082

File tree

1 file changed

+5
-8
lines changed

1 file changed

+5
-8
lines changed

junit-platform-engine/src/main/java/org/junit/platform/engine/support/discovery/ResourceContainerSelectorResolver.java

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -35,32 +35,29 @@
3535
* @since 1.12
3636
*/
3737
class ResourceContainerSelectorResolver implements SelectorResolver {
38-
private final Predicate<Resource> packageFilter;
3938
private final Predicate<Resource> resourceFilter;
4039

41-
ResourceContainerSelectorResolver(Predicate<Resource> resourceFilter, Predicate<String> resourcePackageFilter) {
42-
this.packageFilter = packageName(resourcePackageFilter);
43-
this.resourceFilter = resourceFilter;
40+
ResourceContainerSelectorResolver(Predicate<Resource> resourceFilter, Predicate<String> packageFilter) {
41+
this.resourceFilter = packageName(packageFilter).and(resourceFilter);
4442
}
4543

4644
@Override
4745
public Resolution resolve(ClasspathRootSelector selector, Context context) {
48-
return resourceSelectors(findAllResourcesInClasspathRoot(selector.getClasspathRoot(), packageFilter));
46+
return resourceSelectors(findAllResourcesInClasspathRoot(selector.getClasspathRoot(), resourceFilter));
4947
}
5048

5149
@Override
5250
public Resolution resolve(ModuleSelector selector, Context context) {
53-
return resourceSelectors(findAllResourcesInModule(selector.getModuleName(), packageFilter));
51+
return resourceSelectors(findAllResourcesInModule(selector.getModuleName(), resourceFilter));
5452
}
5553

5654
@Override
5755
public Resolution resolve(PackageSelector selector, Context context) {
58-
return resourceSelectors(findAllResourcesInPackage(selector.getPackageName(), packageFilter));
56+
return resourceSelectors(findAllResourcesInPackage(selector.getPackageName(), resourceFilter));
5957
}
6058

6159
private Resolution resourceSelectors(List<Resource> resources) {
6260
Set<ClasspathResourceSelector> selectors = resources.stream() //
63-
.filter(resourceFilter) //
6461
.collect(groupingBy(Resource::getName)) //
6562
.values() //
6663
.stream() //

0 commit comments

Comments
 (0)