Enhance test method names and documentation across multiple test classes for clarity and consistency

Author: brunoborges

src/test/java/com/github/copilot/sdk/AskUserTest.java

@@ -44,8 +44,14 @@ static void teardown() throws Exception {
         }
     }
 
+    /**
+     * Verifies that user input handler is invoked when model uses ask_user tool.
+     *
+     * @see Snapshot:
+     *      ask_user/should_invoke_user_input_handler_when_model_uses_ask_user_tool
+     */
     @Test
-    void testUserInputHandlerInvokedWhenModelUsesAskUserTool() throws Exception {
+    void testShouldInvokeUserInputHandlerWhenModelUsesAskUserTool() throws Exception {
         ctx.configureForTest("ask_user", "should_invoke_user_input_handler_when_model_uses_ask_user_tool");
 
         List<UserInputRequest> userInputRequests = new ArrayList<>();
@@ -82,8 +88,13 @@ void testUserInputHandlerInvokedWhenModelUsesAskUserTool() throws Exception {
         }
     }
 
+    /**
+     * Verifies that choices are received in user input requests.
+     *
+     * @see Snapshot: ask_user/should_receive_choices_in_user_input_request
+     */
     @Test
-    void testUserInputRequestWithChoices() throws Exception {
+    void testShouldReceiveChoicesInUserInputRequest() throws Exception {
         ctx.configureForTest("ask_user", "should_receive_choices_in_user_input_request");
 
         List<UserInputRequest> userInputRequests = new ArrayList<>();
@@ -116,10 +127,12 @@ void testUserInputRequestWithChoices() throws Exception {
     }
 
     /**
-     * Test for freeform user input response.
+     * Verifies that freeform user input responses are handled.
+     *
+     * @see Snapshot: ask_user/should_handle_freeform_user_input_response
      */
     @Test
-    void testFreeformUserInputResponse() throws Exception {
+    void testShouldHandleFreeformUserInputResponse() throws Exception {
         ctx.configureForTest("ask_user", "should_handle_freeform_user_input_response");
 
         final List<UserInputRequest> userInputRequests = new ArrayList<>();

src/test/java/com/github/copilot/sdk/CompactionTest.java

@@ -48,9 +48,15 @@ static void teardown() throws Exception {
         }
     }
 
+    /**
+     * Verifies that compaction is triggered with low threshold and emits events.
+     *
+     * @see Snapshot:
+     *      compaction/should_trigger_compaction_with_low_threshold_and_emit_events
+     */
     @Test
     @Timeout(value = 120, unit = TimeUnit.SECONDS)
-    void testTriggerCompactionWithLowThresholdAndEmitEvents() throws Exception {
+    void testShouldTriggerCompactionWithLowThresholdAndEmitEvents() throws Exception {
         ctx.configureForTest("compaction", "should_trigger_compaction_with_low_threshold_and_emit_events");
 
         // Create session with very low compaction thresholds to trigger compaction
@@ -114,8 +120,15 @@ void testTriggerCompactionWithLowThresholdAndEmitEvents() throws Exception {
         }
     }
 
+    /**
+     * Verifies that compaction events are not emitted when infinite sessions is
+     * disabled.
+     *
+     * @see Snapshot:
+     *      compaction/should_not_emit_compaction_events_when_infinite_sessions_disabled
+     */
     @Test
-    void testNotEmitCompactionEventsWhenInfiniteSessionsDisabled() throws Exception {
+    void testShouldNotEmitCompactionEventsWhenInfiniteSessionsDisabled() throws Exception {
         ctx.configureForTest("compaction", "should_not_emit_compaction_events_when_infinite_sessions_disabled");
 
         InfiniteSessionConfig infiniteConfig = new InfiniteSessionConfig().setEnabled(false);