Skip to content
This repository was archived by the owner on May 15, 2025. It is now read-only.

Commit 3bac291

Browse files
refactor(aider): remove use_screen parameter from tests and set it to false where applicable for consistency with screen being default now in module
1 parent 4b5b29c commit 3bac291

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

aider/main.test.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ describe("aider", async () => {
1717
it("installs aider with default settings", async () => {
1818
const state = await runTerraformApply(import.meta.dir, {
1919
agent_id: "foo",
20-
use_screen: true, // Need to set this since default is false
2120
});
2221

2322
// Execute the script with mock setup first
@@ -41,6 +40,7 @@ describe("aider", async () => {
4140
const state = await runTerraformApply(import.meta.dir, {
4241
agent_id: "foo",
4342
use_tmux: true,
43+
use_screen: false,
4444
});
4545

4646
// Execute the script with mock setup first
@@ -63,7 +63,6 @@ describe("aider", async () => {
6363
const state = await runTerraformApply(import.meta.dir, {
6464
agent_id: "foo",
6565
experiment_report_tasks: true,
66-
use_screen: true, // Set explicitly since default is false
6766
});
6867

6968
// Set up mocks including coder command
@@ -92,7 +91,6 @@ describe("aider", async () => {
9291
agent_id: "foo",
9392
experiment_pre_install_script: "echo 'Pre-install script executed'",
9493
experiment_post_install_script: "echo 'Post-install script executed'",
95-
use_screen: true, // Set explicitly since default is false
9694
});
9795

9896
// Execute the script with basic mocks

0 commit comments

Comments
 (0)