Skip to content

Commit

Permalink
Rename GenericFilter<T> to Filter<T>
Browse files Browse the repository at this point in the history
  • Loading branch information
marcphilipp committed Jan 22, 2016
1 parent 08bf642 commit abd3f38
Show file tree
Hide file tree
Showing 6 changed files with 13 additions and 11 deletions.
Expand Up @@ -26,7 +26,7 @@
* @see EngineDiscoveryRequest * @see EngineDiscoveryRequest
* @see TestEngine * @see TestEngine
*/ */
public interface DiscoveryFilter<T> extends GenericFilter<T> { public interface DiscoveryFilter<T> extends Filter<T> {


/** /**
* Combines an array of {@code DiscoveryFilters} into a new filter that will * Combines an array of {@code DiscoveryFilters} into a new filter that will
Expand Down
Expand Up @@ -20,7 +20,7 @@
* @see DiscoveryFilter * @see DiscoveryFilter
*/ */
@FunctionalInterface @FunctionalInterface
public interface GenericFilter<T> { public interface Filter<T> {


FilterResult filter(T object); FilterResult filter(T object);


Expand Down
Expand Up @@ -16,7 +16,7 @@
import org.junit.gen5.commons.util.ToStringBuilder; import org.junit.gen5.commons.util.ToStringBuilder;


/** /**
* The result of evaluating a {@link GenericFilter}. * The result of evaluating a {@link Filter}.
* *
* @since 5.0 * @since 5.0
*/ */
Expand Down
Expand Up @@ -12,8 +12,8 @@


import static org.junit.gen5.engine.FilterResult.includedIf; import static org.junit.gen5.engine.FilterResult.includedIf;


import org.junit.gen5.engine.Filter;
import org.junit.gen5.engine.FilterResult; import org.junit.gen5.engine.FilterResult;
import org.junit.gen5.engine.GenericFilter;
import org.junit.gen5.engine.TestEngine; import org.junit.gen5.engine.TestEngine;


/** /**
Expand All @@ -24,7 +24,7 @@
* @since 5.0 * @since 5.0
* @see TestDiscoveryRequest * @see TestDiscoveryRequest
*/ */
public class EngineIdFilter implements GenericFilter<String> { public class EngineIdFilter implements Filter<String> {


public static EngineIdFilter byEngineId(String engineId) { public static EngineIdFilter byEngineId(String engineId) {
return new EngineIdFilter(engineId); return new EngineIdFilter(engineId);
Expand Down
Expand Up @@ -10,7 +10,7 @@


package org.junit.gen5.launcher; package org.junit.gen5.launcher;


import org.junit.gen5.engine.GenericFilter; import org.junit.gen5.engine.Filter;
import org.junit.gen5.engine.TestDescriptor; import org.junit.gen5.engine.TestDescriptor;
import org.junit.gen5.engine.TestEngine; import org.junit.gen5.engine.TestEngine;


Expand All @@ -23,5 +23,5 @@
* @see TestDiscoveryRequest * @see TestDiscoveryRequest
* @see TestEngine * @see TestEngine
*/ */
public interface PostDiscoveryFilter extends GenericFilter<TestDescriptor> { public interface PostDiscoveryFilter extends Filter<TestDescriptor> {
} }
Expand Up @@ -16,8 +16,10 @@


import org.junit.gen5.engine.DiscoveryFilter; import org.junit.gen5.engine.DiscoveryFilter;
import org.junit.gen5.engine.DiscoverySelector; import org.junit.gen5.engine.DiscoverySelector;
import org.junit.gen5.engine.GenericFilter; import org.junit.gen5.engine.Filter;
import org.junit.gen5.launcher.*; import org.junit.gen5.launcher.EngineIdFilter;
import org.junit.gen5.launcher.PostDiscoveryFilter;
import org.junit.gen5.launcher.TestDiscoveryRequest;


/** /**
* The {@code DiscoveryRequestBuilder} provides a light-weight DSL for * The {@code DiscoveryRequestBuilder} provides a light-weight DSL for
Expand Down Expand Up @@ -73,14 +75,14 @@ public TestDiscoveryRequestBuilder select(List<DiscoverySelector> elements) {
return this; return this;
} }


public TestDiscoveryRequestBuilder filter(GenericFilter<?>... filters) { public TestDiscoveryRequestBuilder filter(Filter<?>... filters) {
if (filters != null) { if (filters != null) {
Arrays.stream(filters).forEach(this::storeFilter); Arrays.stream(filters).forEach(this::storeFilter);
} }
return this; return this;
} }


private void storeFilter(GenericFilter<?> filter) { private void storeFilter(Filter<?> filter) {
if (filter instanceof EngineIdFilter) { if (filter instanceof EngineIdFilter) {
this.engineIdFilters.add((EngineIdFilter) filter); this.engineIdFilters.add((EngineIdFilter) filter);
} }
Expand Down

0 comments on commit abd3f38

Please sign in to comment.