From d5a2c616e2f6f54303d70bb0bd788f6e19f12253 Mon Sep 17 00:00:00 2001 From: Guillaume Smet Date: Wed, 1 Mar 2023 09:37:24 +0100 Subject: [PATCH] Remove useless assertion that makes ProviderConfigInjectionWarningsTest brittle We sometimes have RESTEasy complaining about an Apache HTTP Client not being closed. --- .../test/config/ProviderConfigInjectionWarningsTest.java | 5 ----- 1 file changed, 5 deletions(-) diff --git a/extensions/resteasy-classic/resteasy/deployment/src/test/java/io/quarkus/resteasy/test/config/ProviderConfigInjectionWarningsTest.java b/extensions/resteasy-classic/resteasy/deployment/src/test/java/io/quarkus/resteasy/test/config/ProviderConfigInjectionWarningsTest.java index 568b3f45729fd0..0e2abbf4e79716 100644 --- a/extensions/resteasy-classic/resteasy/deployment/src/test/java/io/quarkus/resteasy/test/config/ProviderConfigInjectionWarningsTest.java +++ b/extensions/resteasy-classic/resteasy/deployment/src/test/java/io/quarkus/resteasy/test/config/ProviderConfigInjectionWarningsTest.java @@ -39,11 +39,6 @@ public class ProviderConfigInjectionWarningsTest { .assertLogRecords(logRecords -> { Set messages = logRecords.stream().map(LogRecord::getMessage).collect(Collectors.toSet()); - for (String message : messages) { - System.out.println("Message from ProviderConfigInjectionWarningsTest: " + message); - } - - assertEquals(4, logRecords.size()); assertTrue(messages.contains( "Directly injecting a org.eclipse.microprofile.config.Config into a jakarta.ws.rs.ext.Provider may lead to unexpected results. To ensure proper results, please change the type of the field to jakarta.enterprise.inject.Instance. Offending field is 'config' of class 'io.quarkus.resteasy.test.config.ProviderConfigInjectionWarningsTest$FooProvider'")); assertTrue(messages.contains(