Unify ClientSupport and StubRest client APIs

    - New api is a wrapper above jersey client api to simplify test-specific configuration and validation
      (jersey api is still available). The request builder unifies all possible configurations in one place.
    - New common base class TestClient (and TestRestClient for rest clients, adding rest-specific methods)
    - ClientSupport is a TestClient, but also could provide 3 special clients: appClient(), adminClient(), restClient()
      (restClient() is the same as StubRest client)
    - New sub clients could be created by applying additional path segments:
        client.subClient("/sub/path/)
    - External api client could be created with support.customClient("som external url")
    - New client rest api based on real method calls: restClient(RestClass.class).method(mock -> mock.restMethod(args)).invoke()
      (target path and method type resolved from annotations, arguments used for request configuration)
    - Helper api for testing multipart requests: restClient(..).multipartMethod(..)
      (simplifies multipart method arguments creation)
    - Defaults mechanism: it is possible to declare default headers, cookies, etc.
      on the client to be applied for all requests (evolution of StubRest client ideas)
    - Add PATCH method shortcuts
    - Add builder-style response assertions like: client.do_request.assertHeader("Name", val)
      This allows checking response headers, cookies, status code, etc. in a chained calls style without additional variables
    - Add connector switching api to ClientSupport: apacheClient(), urlconnectorClient()
      This allows using different connectors within one test
      (note that apache connector is required for PATCH calls and urlconnector better handles multipart requests)
    - (BREAKING) previous target(String... path) methods replaced with string format: target(String, Object...args)
    - (BREAKING) deprecated targetMain(): replaced with targetApp()
    - (BREAKING) StubRest default status declaration removed as not useful
      (required status could be declared now with the new request builder)

Author: xvik

CHANGELOG.md

@@ -5,8 +5,6 @@
     - Add `useApacheClient` (shortcut) configuration into `@TestGuiceyApp` and `@TestDropwizardApp`
         to simplify usage of ApacheTestClientFactory with annotations
     - Add `useApacheClient()` (shortcut) method into extension and generic builders
-* @StubRest RestClient:
-    - Add patch method shortcuts
 * Shared state:
     - State objects, implementing AutoClosable, now would be closed on application shutdown
     - Add SharedConfigurationState.lookupOrCreate method to simplify static state usage
@@ -18,6 +16,31 @@
       automatically mapped into url path and query params in this case)
     - General utility for resource methods analysis: ResourceAnalyzer (could be used in 
       various api, based on resource (stub) call)
+* Unify ClientSupport and StubRest client APIs
+    - New api is a wrapper above jersey client api to simplify test-specific configuration and validation
+      (jersey api is still available). The request builder unifies all possible configurations in one place.  
+    - New common base class TestClient (and TestRestClient for rest clients, adding rest-specific methods)
+    - ClientSupport is a TestClient, but also could provide 3 special clients: appClient(), adminClient(), restClient()
+      (restClient() is the same as StubRest client)
+    - New sub clients could be created by applying additional path segments:
+        client.subClient("/sub/path/)
+    - External api client could be created with support.customClient("som external url")
+    - New client rest api based on real method calls: restClient(RestClass.class).method(mock -> mock.restMethod(args)).invoke()
+      (target path and method type resolved from annotations, arguments used for request configuration)
+    - Helper api for testing multipart requests: restClient(..).multipartMethod(..)
+      (simplifies multipart method arguments creation)
+    - Defaults mechanism: it is possible to declare default headers, cookies, etc.
+      on the client to be applied for all requests (evolution of StubRest client ideas)
+    - Add PATCH method shortcuts 
+    - Add builder-style response assertions like: client.do_request.assertHeader("Name", val)
+      This allows checking response headers, cookies, status code, etc. in a chained calls style without additional variables
+    - Add connector switching api to ClientSupport: apacheClient(), urlconnectorClient()
+      This allows using different connectors within one test 
+      (note that apache connector is required for PATCH calls and urlconnector better handles multipart requests)
+    - (BREAKING) previous target(String... path) methods replaced with string format: target(String, Object...args)
+    - (BREAKING) deprecated targetMain(): replaced with targetApp()
+    - (BREAKING) StubRest default status declaration removed as not useful
+      (required status could be declared now with the new request builder)
 
 ### 7.2.1 (2025-05-12)
 * Fix NoClassDefFoundError on guicey startup due to junit classes leak into core (#428)

dropwizard-guicey/src/doc/docs/guide/test/general/client.md

@@ -18,7 +18,7 @@ Example usage:
 client.targetRest("some").request().buildGet().invoke()
 
 // GET {main context path}/servlet
-client.targetMain("servlet").request().buildGet().invoke()
+client.targetApp("servlet").request().buildGet().invoke()
 
 // GET {admin context path}/adminServlet
 client.targetAdmin("adminServlet").request().buildGet().invoke()
@@ -50,7 +50,7 @@ Also, if you want to use other client, client object can simply provide required
 client.getPort()        // app port (8080)
 client.getAdminPort()   // app admin port (8081)
 client.basePathRoot()   // root server path (http://localhost:8080/)
-client.basePathMain()   // main context path (http://localhost:8080/)
+client.basePathApp()   // main context path (http://localhost:8080/)
 client.basePathAdmin()  // admin context path (http://localhost:8081/)
 client.basePathRest()   // rest context path (http://localhost:8080/)
 ```

dropwizard-guicey/src/doc/docs/guide/test/junit5/client.md

@@ -27,7 +27,7 @@ Example usages:
 client.targetRest("some").request().buildGet().invoke()
 
 // GET {main context path}/servlet
-client.targetMain("servlet").request().buildGet().invoke()
+client.targetApp("servlet").request().buildGet().invoke()
 
 // GET {admin context path}/adminServlet
 client.targetAdmin("adminServlet").request().buildGet().invoke()
@@ -59,7 +59,7 @@ Also, if you want to use other client, client object can simply provide required
 client.getPort()        // app port (8080)
 client.getAdminPort()   // app admin port (8081)
 client.basePathRoot()   // root server path (http://localhost:8080/)
-client.basePathMain()   // main context path (http://localhost:8080/)
+client.basePathApp()   // main context path (http://localhost:8080/)
 client.basePathAdmin()  // admin context path (http://localhost:8081/)
 client.basePathRest()   // rest context path (http://localhost:8080/)
 ```

dropwizard-guicey/src/doc/docs/guide/test/junit5/nested.md

@@ -104,7 +104,7 @@ public class ClientSupportDwTest {
         // test to apply for multiple environments
         @Test
         default void callClient(ClientSupport client) {
-            Assertions.assertEquals("main", client.targetMain("servlet")
+            Assertions.assertEquals("main", client.targetApp("servlet")
                     .request().buildGet().invoke().readEntity(String.class));
         }
     }
@@ -115,7 +115,7 @@ public class ClientSupportDwTest {
 
         @Test
         void testClient(ClientSupport client) {
-            Assertions.assertEquals("http://localhost:8080/", client.basePathMain());
+            Assertions.assertEquals("http://localhost:8080/", client.basePathApp());
         }
     }
 
@@ -128,7 +128,7 @@ public class ClientSupportDwTest {
 
         @Test
         void testClient(ClientSupport client) {
-            Assertions.assertEquals("http://localhost:8080/app/", client.basePathMain());
+            Assertions.assertEquals("http://localhost:8080/app/", client.basePathApp());
         }
     }
 }

dropwizard-guicey/src/doc/docs/guide/test/junit5/run.md

@@ -126,7 +126,7 @@ class WebModuleTest {
     public void checkWebBindings(ClientSupport client) {
 
         Assertions.assertEquals("Sample filter and service called", 
-            client.targetMain("servlet").request().buildGet().invoke().readEntity(String.class));
+            client.targetApp("servlet").request().buildGet().invoke().readEntity(String.class));
 
         Assertions.assertTrur(service.isCalled());
 ```

dropwizard-guicey/src/doc/docs/guide/test/spock.md

@@ -210,7 +210,7 @@ Example usages:
 client.targetRest("some").request().buildGet().invoke()
 
 // GET {main context path}/servlet
-client.targetMain("servlet").request().buildGet().invoke()
+client.targetApp("servlet").request().buildGet().invoke()
 
 // GET {admin context path}/adminServlet
 client.targetAdmin("adminServlet").request().buildGet().invoke()
@@ -240,7 +240,7 @@ Also, if you want to use other client, client object can simply provide required
 ```groovy
 client.getPort()        // app port (8080)
 client.getAdminPort()   // app admin port (8081)
-client.basePathMain()   // main context path (http://localhost:8080/)
+client.basePathApp()   // main context path (http://localhost:8080/)
 client.basePathAdmin()  // admin context path (http://localhost:8081/)
 client.basePathRest()   // rest context path (http://localhost:8080/)
 ```

dropwizard-guicey/src/main/java/ru/vyarus/dropwizard/guice/module/installer/util/StackUtils.java

@@ -35,11 +35,14 @@ private StackUtils() {
      * @param skip classes to skip in stack
      * @return caller stack frame
      */
+    @SuppressWarnings("checkstyle:BooleanExpressionComplexity")
     public static Optional<StackWalker.StackFrame> getCaller(final List<Class<?>> skip) {
         return WALKER.walk(stream ->
                 stream.dropWhile(frame -> {
                     final Class<?> type = frame.getDeclaringClass();
                     return type.equals(StackUtils.class)
+                            || type.getPackageName().startsWith("java.")
+                            || type.getPackageName().startsWith("jakarta.")
                             || type.getPackage().getName().startsWith("org.codehaus.groovy.vmplugin")
                             || skip.contains(type)
                             || skip.contains(type.getEnclosingClass());