Skip to content

Commit 9550e41

Browse files
committed
Fix Javadoc for execute(LauncherExecutionRequest) plus polishing
1 parent af47043 commit 9550e41

File tree

1 file changed

+9
-11
lines changed
  • junit-platform-launcher/src/main/java/org/junit/platform/launcher

1 file changed

+9
-11
lines changed

junit-platform-launcher/src/main/java/org/junit/platform/launcher/Launcher.java

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -81,12 +81,11 @@ public interface Launcher {
8181
* {@link #execute(TestPlan, TestExecutionListener...)} for execution at
8282
* most once.
8383
*
84-
* @param launcherDiscoveryRequest the launcher discovery request; never
85-
* {@code null}
84+
* @param discoveryRequest the launcher discovery request; never {@code null}
8685
* @return an unmodifiable {@code TestPlan} that contains all resolved
8786
* {@linkplain TestIdentifier identifiers} from all registered engines
8887
*/
89-
TestPlan discover(LauncherDiscoveryRequest launcherDiscoveryRequest);
88+
TestPlan discover(LauncherDiscoveryRequest discoveryRequest);
9089

9190
/**
9291
* Execute a {@link TestPlan} which is built according to the supplied
@@ -106,14 +105,14 @@ public interface Launcher {
106105
* performance degradation (e.g., classpath scanning) of running test
107106
* discovery twice.
108107
*
109-
* @param launcherDiscoveryRequest the launcher discovery request; never {@code null}
108+
* @param discoveryRequest the launcher discovery request; never {@code null}
110109
* @param listeners additional test execution listeners; never {@code null}
111110
* @deprecated Please use {@link #execute(LauncherExecutionRequest)} instead.
112111
*/
113112
@Deprecated(since = "6.0")
114113
@API(status = DEPRECATED, since = "6.0")
115-
default void execute(LauncherDiscoveryRequest launcherDiscoveryRequest, TestExecutionListener... listeners) {
116-
var executionRequest = LauncherExecutionRequestBuilder.request(launcherDiscoveryRequest) //
114+
default void execute(LauncherDiscoveryRequest discoveryRequest, TestExecutionListener... listeners) {
115+
var executionRequest = LauncherExecutionRequestBuilder.request(discoveryRequest) //
117116
.listeners(listeners) //
118117
.build();
119118
execute(executionRequest);
@@ -146,8 +145,8 @@ default void execute(TestPlan testPlan, TestExecutionListener... listeners) {
146145
}
147146

148147
/**
149-
* Execute tests according to the supplied {@link LauncherExecutionRequest}
150-
* {@linkplain #registerTestExecutionListeners registered listeners} about
148+
* Execute tests according to the supplied {@link LauncherExecutionRequest} and
149+
* notify {@linkplain #registerTestExecutionListeners registered listeners} about
151150
* the progress and results of the execution.
152151
*
153152
* <p>Test execution listeners supplied
@@ -167,11 +166,10 @@ default void execute(TestPlan testPlan, TestExecutionListener... listeners) {
167166
* supplied execution request to avoid the potential performance degradation
168167
* (e.g., classpath scanning) of running test discovery twice.
169168
*
170-
* @param launcherExecutionRequest the launcher execution request; never
171-
* {@code null}
169+
* @param executionRequest the launcher execution request; never {@code null}
172170
* @since 6.0
173171
*/
174172
@API(status = MAINTAINED, since = "6.0")
175-
void execute(LauncherExecutionRequest launcherExecutionRequest);
173+
void execute(LauncherExecutionRequest executionRequest);
176174

177175
}

0 commit comments

Comments
 (0)