Skip to content

test: cut git subprocess spawns in TestRepo::create() #367

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Ziinc merged 2 commits into from
Aug 28, 2026
+7 −8
Merged

test: cut git subprocess spawns in TestRepo::create() #367

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
8029fa3
test: cut git subprocess spawns in TestRepo::create()
Ziinc Aug 27, 2026
06d6ebd
Merge branch 'main' into claude/ci-test-repo-fewer-git-spawns
Ziinc Aug 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 7 additions & 8 deletions src-tauri/tests/e2e_test_helpers.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,8 +70,13 @@ impl TestRepo {
let temp_dir = TempDir::new().map_err(|e| format!("Failed to create temp dir: {}", e))?;
let repo_path = temp_dir.path().to_string_lossy().to_string();

// Initialize git repo
Self::run_git(&repo_path, &["init"])?;
// Initialize git repo with its initial branch already named, in one
// spawn instead of `init` + `branch -M` + `checkout -b`. Each test
// creates its own repo, and on Windows process-spawn overhead (not
// git's actual work) dominates these setup calls, so cutting the
// subprocess count here matters for CI wall time.
let default_branch = random_default_branch_name();
Self::run_git(&repo_path, &["init", "-b", &default_branch])?;

// Configure git user (required for commits)
Self::run_git(&repo_path, &["config", "user.email", "test@example.com"])?;
Expand All @@ -81,12 +86,6 @@ impl TestRepo {
// that default to core.autocrlf=true rewrite LF to CRLF on checkout.
Self::run_git(&repo_path, &["config", "core.autocrlf", "false"])?;

let default_branch = random_default_branch_name();
Self::run_git(&repo_path, &["branch", "-M", &default_branch])
.map_err(|e| format!("Failed to create default branch: {}", e))?;

Self::run_git(&repo_path, &["checkout", "-b", &default_branch])?;

// Record the default branch in local git config so get_default_branch() can
// discover it via the merged init.defaultBranch fallback, even when HEAD moves
// to a feature branch and there is no remote or main/master branch.
Expand Down
Loading