Skip to content

Commit

Permalink
Remove useless assertion that makes ProviderConfigInjectionWarningsTe…
Browse files Browse the repository at this point in the history
…st brittle

We sometimes have RESTEasy complaining about an Apache HTTP Client not
being closed.
  • Loading branch information
gsmet committed Mar 1, 2023
1 parent cfbb0a4 commit d5a2c61
Showing 1 changed file with 0 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -39,11 +39,6 @@ public class ProviderConfigInjectionWarningsTest {
.assertLogRecords(logRecords -> {
Set<String> 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<org.eclipse.microprofile.config.Config>. Offending field is 'config' of class 'io.quarkus.resteasy.test.config.ProviderConfigInjectionWarningsTest$FooProvider'"));
assertTrue(messages.contains(
Expand Down

0 comments on commit d5a2c61

Please sign in to comment.