another arch test

Author: rbri

src/main/java/org/htmlunit/websocket/JettyWebSocketAdapter.java

@@ -27,7 +27,9 @@
 import org.htmlunit.jetty.websocket.client.WebSocketClient;
 
 /**
- * Jetty based impl of the WebSocketAdapter.
+ * Jetty9 based impl of the WebSocketAdapter.
+ * To avoid conflicts with other jetty versions used by projects, we use
+ * our own shaded version of jetty9 (https://github.com/HtmlUnit/htmlunit-websocket-client).
  *
  * @author Ronald Brill
  */

src/test/java/org/htmlunit/archunit/ArchitectureTest.java

@@ -90,6 +90,19 @@ public class ArchitectureTest {
             .and().resideOutsideOfPackage("org.htmlunit.jetty..")
         .should().dependOnClassesThat().resideInAnyPackage("java.awt..");
 
+    /**
+     * The jetty websocket stuff is only used by one class.
+     */
+    @ArchTest
+    public static final ArchRule webSocketPackageRule = noClasses()
+            .that()
+                .resideOutsideOfPackage("org.htmlunit.jetty..")
+                .and().doNotHaveFullyQualifiedName("org.htmlunit.websocket.JettyWebSocketAdapter")
+                .and().doNotHaveFullyQualifiedName("org.htmlunit.websocket.JettyWebSocketAdapter$JettyWebSocketAdapterFactory")
+                .and().doNotHaveFullyQualifiedName("org.htmlunit.websocket.JettyWebSocketAdapter$JettyWebSocketAdapterImpl")
+            .should()
+                .dependOnClassesThat().resideInAnyPackage("org.htmlunit.jetty..");
+
     /**
      * JsxClasses are always in the javascript package.
      */