Skip to content

Commit b4c426d

Browse files
committed
Improve LauncherFactoryTests
1 parent fa081f9 commit b4c426d

File tree

2 files changed

+11
-12
lines changed

2 files changed

+11
-12
lines changed

platform-tests/src/test/java/org/junit/platform/launcher/TestLauncherSessionListener.java

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -10,21 +10,19 @@
1010

1111
package org.junit.platform.launcher;
1212

13+
import org.jspecify.annotations.Nullable;
14+
1315
public class TestLauncherSessionListener implements LauncherSessionListener {
1416

17+
public static @Nullable LauncherSession session;
18+
1519
@Override
16-
public boolean equals(Object obj) {
17-
if (this == obj) {
18-
return true;
19-
}
20-
if (obj == null) {
21-
return false;
22-
}
23-
return getClass() == obj.getClass();
20+
public void launcherSessionOpened(LauncherSession session) {
21+
TestLauncherSessionListener.session = session;
2422
}
2523

2624
@Override
27-
public int hashCode() {
28-
return 1;
25+
public void launcherSessionClosed(LauncherSession session) {
26+
TestLauncherSessionListener.session = null;
2927
}
3028
}

platform-tests/src/test/java/org/junit/platform/launcher/core/LauncherFactoryTests.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -250,9 +250,10 @@ void doesNotDiscoverLauncherSessionListenerViaServiceApiWhenDisabled() {
250250
@Test
251251
void discoversLauncherSessionListenerViaServiceApiByDefault() {
252252
withTestServices(() -> {
253-
try (var session = (DefaultLauncherSession) LauncherFactory.openSession()) {
254-
assertThat(session.getListener()).isEqualTo(new TestLauncherSessionListener());
253+
try (var session = LauncherFactory.openSession()) {
254+
assertThat(TestLauncherSessionListener.session).isSameAs(session);
255255
}
256+
assertThat(TestLauncherSessionListener.session).isNull();
256257
});
257258
}
258259

0 commit comments

Comments
 (0)