Skip to content

Commit e54ecac

Browse files
committed
test(cli): make probe timeout regression deterministic
1 parent 072dd5e commit e54ecac

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

src/test/java/io/github/easy4j/codex/cli/CodexCliExecutorTest.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
import static org.junit.jupiter.api.Assertions.assertNotNull;
2424
import static org.junit.jupiter.api.Assertions.assertTrue;
2525

26+
import java.nio.file.Paths;
2627
import java.util.concurrent.TimeUnit;
2728

2829
/**
@@ -38,6 +39,9 @@
3839
*/
3940
class CodexCliExecutorTest {
4041

42+
private static final String SLOW_CODEX_SCRIPT =
43+
Paths.get("src", "test", "resources", "slow-codex.sh").toAbsolutePath().toString();
44+
4145
private CodexClientConfig configFor(String executable) {
4246
CodexClientConfig config = new CodexClientConfig();
4347
config.setLocalExecutable(executable);
@@ -162,7 +166,7 @@ void shouldReportFailureFromProbeWhenExecutableMissing() {
162166

163167
@Test
164168
void shouldUseDedicatedProbeTimeoutWithoutChangingNormalTimeout() {
165-
CodexClientConfig config = configFor("/bin/sh -c \"sleep 4; exit 0\"");
169+
CodexClientConfig config = configFor("/bin/sh " + SLOW_CODEX_SCRIPT);
166170
config.setLocalProbeTimeoutSeconds(1);
167171
config.setLocalTimeoutSeconds(8);
168172
CodexCliExecutor executor = new CodexCliExecutor(config);

0 commit comments

Comments
 (0)