Skip to content

Commit 98a6927

Browse files
committed
Disable AskUserTest and HooksTest until CLI 0.0.400+ runtime is available
These tests require CLI version 0.0.400+ which includes support for the ask_user tool and hooks system. The test harness uses a separate XDG_CONFIG_HOME which causes the CLI to download an older runtime version (0.0.394) that doesn't support these features. The tests are properly implemented but disabled with @disabled annotation until the CLI runtime version issue is resolved.
1 parent 6d1185d commit 98a6927

2 files changed

Lines changed: 14 additions & 9 deletions

File tree

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

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,28 +13,30 @@
1313

1414
import org.junit.jupiter.api.AfterAll;
1515
import org.junit.jupiter.api.BeforeAll;
16-
import org.junit.jupiter.api.Disabled;
1716
import org.junit.jupiter.api.Test;
1817

1918
import com.github.copilot.sdk.json.MessageOptions;
2019
import com.github.copilot.sdk.json.SessionConfig;
2120
import com.github.copilot.sdk.json.UserInputRequest;
2221
import com.github.copilot.sdk.json.UserInputResponse;
2322

23+
import org.junit.jupiter.api.Disabled;
24+
2425
/**
2526
* Tests for user input handler (ask_user) functionality.
2627
*
2728
* <p>
2829
* These tests use the shared CapiProxy infrastructure for deterministic API
2930
* response replay. Snapshots are stored in test/snapshots/ask-user/.
3031
* </p>
31-
*
3232
* <p>
33-
* <b>Note:</b> These tests require the latest test harness with ask_user
34-
* support.
33+
* <b>Note:</b> These tests are currently disabled because they require CLI
34+
* version 0.0.400+ which supports the ask_user tool. The test harness uses a
35+
* separate XDG_CONFIG_HOME which causes the CLI to download an older runtime
36+
* version (0.0.394) that doesn't support ask_user.
3537
* </p>
3638
*/
37-
@Disabled("Requires test harness update with ask_user support - see upstream PR #269")
39+
@Disabled("Requires CLI version 0.0.400+ with ask_user tool support. See class javadoc for details.")
3840
public class AskUserTest {
3941

4042
private static E2ETestContext ctx;

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

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717

1818
import org.junit.jupiter.api.AfterAll;
1919
import org.junit.jupiter.api.BeforeAll;
20-
import org.junit.jupiter.api.Disabled;
2120
import org.junit.jupiter.api.Test;
2221

2322
import com.github.copilot.sdk.json.MessageOptions;
@@ -27,19 +26,23 @@
2726
import com.github.copilot.sdk.json.SessionConfig;
2827
import com.github.copilot.sdk.json.SessionHooks;
2928

29+
import org.junit.jupiter.api.Disabled;
30+
3031
/**
3132
* Tests for hooks functionality (pre-tool-use and post-tool-use hooks).
3233
*
3334
* <p>
3435
* These tests use the shared CapiProxy infrastructure for deterministic API
3536
* response replay. Snapshots are stored in test/snapshots/hooks/.
3637
* </p>
37-
*
3838
* <p>
39-
* <b>Note:</b> These tests require the latest test harness with hooks support.
39+
* <b>Note:</b> These tests are currently disabled because they require CLI
40+
* version 0.0.400+ which supports the hooks system. The test harness uses a
41+
* separate XDG_CONFIG_HOME which causes the CLI to download an older runtime
42+
* version (0.0.394) that doesn't fully support hooks.
4043
* </p>
4144
*/
42-
@Disabled("Requires test harness update with hooks support - see upstream PR #269")
45+
@Disabled("Requires CLI version 0.0.400+ with hooks support. See class javadoc for details.")
4346
public class HooksTest {
4447

4548
private static E2ETestContext ctx;

0 commit comments

Comments
 (0)