Skip to content

Feature/ticket 6.1 testing ci#36

Merged
vincentgrobler merged 4 commits intomainfrom
feature/ticket-6.1-testing-ci
Feb 25, 2026
Merged

Feature/ticket 6.1 testing ci#36
vincentgrobler merged 4 commits intomainfrom
feature/ticket-6.1-testing-ci

Conversation

@vincentgrobler
Copy link
Copy Markdown
Collaborator

Description

feat(ticket-6.1): E2E testing, unit tests, CI pipeline

Playwright E2E: - playwright.config.ts: Chromium, auth state reuse, Vite dev server - e2e/auth.setup.ts: shared login flow - e2e/login.spec.ts: dashboard + sidebar navigation - e2e/agents.spec.ts: serial CRUD (create → edit → delete) - e2e/tasks.spec.ts: create task + verify status - e2e/marketplace.spec.ts: browse, search, tag filter, detail modal Task Runner unit tests (18 tests): - usageWriter.test.ts: detectBillingModel (9 tests) - pipelineExecutor.test.ts: buildStepPrompt (9 tests) CI pipeline: - Add task-runner-test job (tsc + vitest) - Add e2e job (manual trigger, Playwright + Chromium, artifact upload) - Exclude e2e/, playwright.config.ts from ESLint - Add Playwright artifacts to .gitignore

Related Issue

None

Type of Change

  • [ ✓ ] ✨ New feature (non-breaking change that adds functionality)

How Has This Been Tested?

Internal

  • [ x ] Unit tests
  • [ ✓ ] Manual testing
  • [ x ] Browser testing (specify browsers)

Screenshots (if applicable)

None

Checklist

  • [ ✓ ] My code follows the project's coding standards
  • [ ✓ ] I have run npm run lint and there are no errors
  • [ ✓ ] I have run npm run test and all tests pass
  • [ ✓ ] I have added tests for my changes (if applicable)
  • [ ✓ ] I have updated the documentation (if applicable)
  • [ ✓ ] My source files have AGPL licence headers
  • [ ✓ ] My changes generate no new TypeScript errors

Playwright E2E:
- playwright.config.ts: Chromium, auth state reuse, Vite dev server
- e2e/auth.setup.ts: shared login flow
- e2e/login.spec.ts: dashboard + sidebar navigation
- e2e/agents.spec.ts: serial CRUD (create → edit → delete)
- e2e/tasks.spec.ts: create task + verify status
- e2e/marketplace.spec.ts: browse, search, tag filter, detail modal

Task Runner unit tests (18 tests):
- usageWriter.test.ts: detectBillingModel (9 tests)
- pipelineExecutor.test.ts: buildStepPrompt (9 tests)

CI pipeline:
- Add task-runner-test job (tsc + vitest)
- Add e2e job (manual trigger, Playwright + Chromium, artifact upload)
- Exclude e2e/, playwright.config.ts from ESLint
- Add Playwright artifacts to .gitignore
- Add task-runner to ESLint ignores (has its own CI job)
- Add e2e/ and task-runner/ to vitest exclude (prevents vitest
  from loading Playwright specs and task-runner tests)
- Regenerate package-lock.json
Prevents vitest from walking up to root vitest.config.ts which
requires jsdom and frontend-only deps not available in task-runner CI.
@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
crewform Ready Ready Preview, Comment Feb 25, 2026 4:42pm

The required status check is 'Build, Lint & Test', not the renamed
'Frontend — Build, Lint & Test'.
@vincentgrobler vincentgrobler merged commit dac4379 into main Feb 25, 2026
7 checks passed
@vincentgrobler vincentgrobler deleted the feature/ticket-6.1-testing-ci branch March 5, 2026 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant