Skip to content

Commit 8aac329

Browse files
committed
fix: properly reset ConfigurationServiceProvider after test
1 parent 41c0647 commit 8aac329

File tree

1 file changed

+15
-10
lines changed

1 file changed

+15
-10
lines changed

operator-framework-core/src/test/java/io/javaoperatorsdk/operator/processing/event/source/informer/InformerEventSourceTest.java

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -245,16 +245,21 @@ void filtersOnDeleteEvents() {
245245

246246
@Test
247247
void informerStoppedHandlerShouldBeCalledWhenInformerStops() {
248-
final var exception = new RuntimeException("Informer stopped exceptionally!");
249-
final var informerStoppedHandler = mock(InformerStoppedHandler.class);
250-
ConfigurationServiceProvider
251-
.overrideCurrent(overrider -> overrider.withInformerStoppedHandler(informerStoppedHandler));
252-
informerEventSource = new InformerEventSource<>(informerConfiguration,
253-
MockKubernetesClient.client(Deployment.class, unused -> {
254-
throw exception;
255-
}));
256-
informerEventSource.start();
257-
verify(informerStoppedHandler, atLeastOnce()).onStop(any(), eq(exception));
248+
try {
249+
final var exception = new RuntimeException("Informer stopped exceptionally!");
250+
final var informerStoppedHandler = mock(InformerStoppedHandler.class);
251+
ConfigurationServiceProvider
252+
.overrideCurrent(
253+
overrider -> overrider.withInformerStoppedHandler(informerStoppedHandler));
254+
informerEventSource = new InformerEventSource<>(informerConfiguration,
255+
MockKubernetesClient.client(Deployment.class, unused -> {
256+
throw exception;
257+
}));
258+
informerEventSource.start();
259+
verify(informerStoppedHandler, atLeastOnce()).onStop(any(), eq(exception));
260+
} finally {
261+
ConfigurationServiceProvider.reset();
262+
}
258263
}
259264

260265
Deployment testDeployment() {

0 commit comments

Comments
 (0)