refactor: update Playwright config and worktree integration tests

- Adjusted Playwright configuration to set workers to undefined for improved test execution.
- Updated comments in worktree integration tests to clarify branch creation logic and ensure accurate assertions regarding branch and worktree paths.
This commit is contained in:
Cody Seibert
2025-12-18 00:22:37 -05:00
parent a192eaa20f
commit 20d7fb1949
2 changed files with 9 additions and 6 deletions

View File

@@ -3,14 +3,15 @@ import { defineConfig, devices } from "@playwright/test";
const port = process.env.TEST_PORT || 3007;
const serverPort = process.env.TEST_SERVER_PORT || 3008;
const reuseServer = process.env.TEST_REUSE_SERVER === "true";
const mockAgent = process.env.CI === "true" || process.env.AUTOMAKER_MOCK_AGENT === "true";
const mockAgent =
process.env.CI === "true" || process.env.AUTOMAKER_MOCK_AGENT === "true";
export default defineConfig({
testDir: "./tests",
fullyParallel: true,
forbidOnly: !!process.env.CI,
retries: process.env.CI ? 2 : 0,
workers: process.env.CI ? 1 : undefined,
workers: undefined,
reporter: "html",
timeout: 30000,
use: {

View File

@@ -2441,9 +2441,9 @@ test.describe("Worktree Integration Tests", () => {
// Verify worktree was NOT created during editing (worktrees are created at execution time)
expect(fs.existsSync(expectedWorktreePath)).toBe(false);
// Verify the branch was created (if branch creation is part of the autocomplete flow)
// Verify branch was NOT created (created at execution time)
const branches = await listBranches(testRepo.path);
expect(branches).toContain(newBranchName);
expect(branches).not.toContain(newBranchName);
// Verify feature was updated with correct branchName only
// Note: worktreePath is no longer stored - worktrees are created server-side at execution time
@@ -2593,7 +2593,8 @@ test.describe("Worktree Integration Tests", () => {
);
expect(matchingWorktrees.length).toBe(1);
// Verify feature was updated with the correct worktreePath
// Verify feature was updated with the correct branchName
// Note: worktreePath is no longer stored - worktrees are created server-side at execution time
const featuresDir = path.join(testRepo.path, ".automaker", "features");
const featureDirs = fs.readdirSync(featuresDir);
const featureDir = featureDirs.find((dir) => {
@@ -2608,6 +2609,7 @@ test.describe("Worktree Integration Tests", () => {
const featureFilePath = path.join(featuresDir, featureDir!, "feature.json");
const featureData = JSON.parse(fs.readFileSync(featureFilePath, "utf-8"));
expect(featureData.branchName).toBe(existingBranch);
expect(featureData.worktreePath).toBe(existingWorktreePath);
// worktreePath should not exist in the feature data (worktrees are created at execution time)
expect(featureData.worktreePath).toBeUndefined();
});
});