Disable logging for org.htmlunit.css

Removes 1000+ CSSValue warning, SilentCssErrorHandler doesn't cover this case https://github.com/HtmlUnit/htmlunit/blob/4.7.0/src/main/java/org/htmlunit/css/CssStyleSheet.java#L1427
quarkus-qe · Jan 6, 2025 · d6f8925 · d6f8925
1 parent afa830c
commit d6f8925
Show file tree

Hide file tree

Showing 5 changed files with 17 additions and 2 deletions.
diff --git a/...keycloak-jwt/src/test/java/io/quarkus/ts/security/keycloak/jwt/BaseOidcJwtSecurityIT.java b/...keycloak-jwt/src/test/java/io/quarkus/ts/security/keycloak/jwt/BaseOidcJwtSecurityIT.java
@@ -6,6 +6,8 @@
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.util.Optional;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 import org.apache.http.HttpStatus;
 import org.htmlunit.FailingHttpStatusCodeException;
@@ -34,6 +36,7 @@ public abstract class BaseOidcJwtSecurityIT {
     public void setup() {
         webClient = new WebClient();
         webClient.setCssErrorHandler(new SilentCssErrorHandler());
+        Logger.getLogger("org.htmlunit.css").setLevel(Level.OFF);
         webClient.getOptions().setRedirectEnabled(true);
     }
 

diff --git a/.../src/test/java/io/quarkus/ts/security/keycloak/multitenant/BaseMultiTenantSecurityIT.java b/.../src/test/java/io/quarkus/ts/security/keycloak/multitenant/BaseMultiTenantSecurityIT.java
@@ -7,6 +7,8 @@
 
 import java.net.URI;
 import java.util.Optional;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 import org.apache.http.HttpStatus;
 import org.htmlunit.SilentCssErrorHandler;
@@ -37,6 +39,7 @@ public abstract class BaseMultiTenantSecurityIT {
     public void setup() {
         webClient = new WebClient();
         webClient.setCssErrorHandler(new SilentCssErrorHandler());
+        Logger.getLogger("org.htmlunit.css").setLevel(Level.OFF);
         webClient.getOptions().setRedirectEnabled(true);
     }
 

diff --git a/...uarkus/ts/security/keycloak/oidcclient/extended/restclient/LogoutSinglePageAppFlowIT.java b/...uarkus/ts/security/keycloak/oidcclient/extended/restclient/LogoutSinglePageAppFlowIT.java
@@ -8,6 +8,8 @@
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.io.IOException;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 import org.htmlunit.SilentCssErrorHandler;
 import org.htmlunit.WebClient;
@@ -41,7 +43,6 @@ public class LogoutSinglePageAppFlowIT {
     @Test
     public void singlePageAppLogoutFlow() throws IOException {
         try (final WebClient webClient = createWebClient()) {
-            webClient.getOptions().setRedirectEnabled(true);
             HtmlPage page = webClient.getPage(app.getURI(Protocol.HTTP).toString() + "/code-flow");
 
             HtmlForm form = page.getHtmlElementById("kc-form-login");
@@ -64,6 +65,8 @@ public void singlePageAppLogoutFlow() throws IOException {
     private WebClient createWebClient() {
         WebClient webClient = new WebClient();
         webClient.setCssErrorHandler(new SilentCssErrorHandler());
+        Logger.getLogger("org.htmlunit.css").setLevel(Level.OFF);
+        webClient.getOptions().setRedirectEnabled(true);
         return webClient;
     }
 

diff --git a/.../quarkus/ts/security/keycloak/oidcclient/reactive/extended/LogoutSinglePageAppFlowIT.java b/.../quarkus/ts/security/keycloak/oidcclient/reactive/extended/LogoutSinglePageAppFlowIT.java
@@ -8,6 +8,8 @@
 
 import java.io.IOException;
 import java.util.Objects;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 import org.htmlunit.SilentCssErrorHandler;
 import org.htmlunit.WebClient;
@@ -53,7 +55,6 @@ public class LogoutSinglePageAppFlowIT {
     @Test
     public void singlePageAppLogoutFlow() throws IOException {
         try (final WebClient webClient = createWebClient()) {
-            webClient.getOptions().setRedirectEnabled(true);
             String content = makeHttpPostFormLogin(webClient, "/code-flow", "alice", "alice")
                     .getContentAsString();
 
@@ -91,6 +92,8 @@ private WebResponse makeHttpPostFormLogin(WebClient webClient, String loginPath,
     private WebClient createWebClient() {
         WebClient webClient = new WebClient();
         webClient.setCssErrorHandler(new SilentCssErrorHandler());
+        Logger.getLogger("org.htmlunit.css").setLevel(Level.OFF);
+        webClient.getOptions().setRedirectEnabled(true);
         return webClient;
     }
 

diff --git a/...oak-webapp/src/test/java/io/quarkus/ts/security/keycloak/webapp/BaseWebappSecurityIT.java b/...oak-webapp/src/test/java/io/quarkus/ts/security/keycloak/webapp/BaseWebappSecurityIT.java
@@ -9,6 +9,8 @@
 import java.net.URI;
 import java.util.Optional;
 import java.util.concurrent.TimeUnit;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 import org.apache.http.HttpStatus;
 import org.htmlunit.FailingHttpStatusCodeException;
@@ -41,6 +43,7 @@ public abstract class BaseWebappSecurityIT {
     public void setup() {
         webClient = new WebClient();
         webClient.setCssErrorHandler(new SilentCssErrorHandler());
+        Logger.getLogger("org.htmlunit.css").setLevel(Level.OFF);
         webClient.getOptions().setRedirectEnabled(true);
     }