Skip to content

Commit 992cf3e

Browse files
committed
🤖 test: skip non-core runtime suites for DockerRuntime
Keeps the docker runtime integration coverage focused on the core Runtime contract to avoid the integration job hitting its time limit.
1 parent f058993 commit 992cf3e

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

tests/runtime/runtime.test.ts

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,10 @@ describeIntegration("Runtime integration tests", () => {
9090
}
9191
return os.tmpdir();
9292
};
93+
94+
// DockerRuntime is slower than local/ssh, and the integration job has a hard
95+
// time budget. Keep the Docker coverage focused on the core Runtime contract.
96+
const describeNonDocker = type === "docker" ? describe.skip : describe;
9397
const createRuntime = (): Runtime =>
9498
createTestRuntime(type, getBaseWorkdir(), sshConfig, dockerConfig);
9599

@@ -547,7 +551,7 @@ describeIntegration("Runtime integration tests", () => {
547551
});
548552
});
549553

550-
describe("Edge cases", () => {
554+
describeNonDocker("Edge cases", () => {
551555
test.concurrent(
552556
"handles large files efficiently",
553557
async () => {
@@ -605,7 +609,7 @@ describeIntegration("Runtime integration tests", () => {
605609
});
606610
});
607611

608-
describe("Git operations", () => {
612+
describeNonDocker("Git operations", () => {
609613
test.concurrent("can initialize a git repository", async () => {
610614
const runtime = createRuntime();
611615
await using workspace = await TestWorkspace.create(runtime, type);
@@ -712,7 +716,7 @@ describeIntegration("Runtime integration tests", () => {
712716
});
713717
});
714718

715-
describe("Environment and shell behavior", () => {
719+
describeNonDocker("Environment and shell behavior", () => {
716720
test.concurrent("preserves multi-line output formatting", async () => {
717721
const runtime = createRuntime();
718722
await using workspace = await TestWorkspace.create(runtime, type);
@@ -784,7 +788,7 @@ describeIntegration("Runtime integration tests", () => {
784788
});
785789
});
786790

787-
describe("Error handling", () => {
791+
describeNonDocker("Error handling", () => {
788792
test.concurrent("handles command not found", async () => {
789793
const runtime = createRuntime();
790794
await using workspace = await TestWorkspace.create(runtime, type);

0 commit comments

Comments
 (0)