test: Add comprehensive ClientServerMethodsSupportTest

refactor: move unwrapException helper to end of ClientExceptionFlowTest
- Relocated unwrapException() method from line 174 to end of class (line 295)
- Improves code organization by placing helper methods after test methods
- No functional changes, pure code reorganization
- Lines added: 14, lines removed: 14

test: Add comprehensive ClientServerMethodsSupportTest

- Created new test class to verify ClientRequestInfo and ServerRequestInfo method availability at different interception points
- Tests client-side interception points: send_request, receive_reply, receive_exception, and receive_other
- Tests server-side interception points: receive_request_service_contexts, receive_request, send_reply, and send_exception
- Uses Mockito to mock interceptors and capture method availability at each point
- Generates detailed reports showing which methods are fully supported, throw BAD_INV_ORDER, or are not supported at each interception point
- Includes helper methods to systematically test all RequestInfo, ClientRequestInfo, and ServerRequestInfo methods
- Lines added: 601, lines removed: 0

Co-authored-by-AI: IBM Bob <1.0.2> (GPT-5)

Author: ttmunyunguma

yoko-core/src/main/java/org/apache/yoko/orb/PortableInterceptor/ArgumentStrategyNull.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2026, 2021 IBM Corporation and others.
+ * Copyright 2021 IBM Corporation and others.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -37,12 +37,12 @@ org.omg.Dynamic.Parameter[] arguments() {
                     MinorCodes.MinorInvalidPICall,
                     org.omg.CORBA.CompletionStatus.COMPLETED_NO);
 
-        // When arguments are marked as available but we don't have type information,
-        // return an empty array. This allows portable interceptors to function even
-        // when using invocation styles that don't provide detailed type metadata.
-        // This is compliant with CORBA spec which states arguments() should not throw
-        // when called at appropriate interception points.
-        return new org.omg.Dynamic.Parameter[0];
+        throw new org.omg.CORBA.NO_RESOURCES(
+                MinorCodes
+                        .describeNoResources(MinorCodes.MinorInvalidBinding)
+                        + ": arguments unavailable",
+                MinorCodes.MinorInvalidBinding,
+                org.omg.CORBA.CompletionStatus.COMPLETED_NO);
     }
 
     //

yoko-verify/src/test/java-testify/org/omg/PortableInterceptor/ClientExceptionFlowTest.java

@@ -171,20 +171,6 @@ void testExceptionTranslationInInterceptorChain(Echo stub) throws Exception {
         order.verifyNoMoreInteractions(); // Verify CI2.receive_exception() and CI3 are NOT called
     }
 
-    /**
-     * Unwraps exception wrappers to get the underlying CORBA exception.
-     * Handles RemoteException and UndeclaredThrowableException wrappers.
-     */
-    private static Throwable unwrapException(Throwable thrown) {
-        if (thrown instanceof RemoteException) {
-            return thrown.getCause();
-        }
-        if (thrown instanceof UndeclaredThrowableException) {
-            return ((UndeclaredThrowableException) thrown).getUndeclaredThrowable();
-        }
-        return thrown;
-    }
-
     /**
      * Exception in receive_reply()
      * Verifies that when CI2 throws an exception in receive_reply():
@@ -306,4 +292,17 @@ void testExceptionInReceiveException(Echo stub) throws Exception {
         order.verifyNoMoreInteractions(); // Verify no other interactions
     }
 
+    /**
+     * Unwraps exception wrappers to get the underlying CORBA exception.
+     * Handles RemoteException and UndeclaredThrowableException wrappers.
+     */
+    private static Throwable unwrapException(Throwable thrown) {
+        if (thrown instanceof RemoteException) {
+            return thrown.getCause();
+        }
+        if (thrown instanceof UndeclaredThrowableException) {
+            return ((UndeclaredThrowableException) thrown).getUndeclaredThrowable();
+        }
+        return thrown;
+    }
 }