Skip to content

Add e2e tests to CI - #80

Open
justanotherariel wants to merge 1 commit into
mainfrom
justanotherariel-playwright-ci
Open

Add e2e tests to CI#80
justanotherariel wants to merge 1 commit into
mainfrom
justanotherariel-playwright-ci

Conversation

@justanotherariel

@justanotherariel justanotherariel commented Sep 1, 2026

Copy link
Copy Markdown
Member

Description

Run e2e playwright tests in CI test workflow. If they are not run automatically, they are very likely to break over time since developers would have to check them manually.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactoring (no functional changes)
  • CI/CD changes

Related Issues

Relates to #1

Testing

  • Unit tests added/updated
  • E2E tests added/updated
  • Manually tested on macOS
  • Manually tested on Windows
  • Manually tested on Linux

Screenshots

Checklist

  • My code follows the project's code style
  • I have run cargo fmt and npm run lint
  • I have added tests for my changes
  • All new and existing tests pass
  • I have updated documentation if needed
  • My commits follow conventional commit format

Copilot AI balanced review requested due to automatic review settings September 1, 2026 15:48
@codecov-commenter

codecov-commenter commented Sep 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.54%. Comparing base (be76509) to head (44d0587).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #80   +/-   ##
=======================================
  Coverage   89.54%   89.54%           
=======================================
  Files          29       29           
  Lines       10910    10910           
  Branches       84       84           
=======================================
  Hits         9769     9769           
  Misses       1141     1141           
Flag Coverage Δ
backend 92.01% <ø> (ø)
frontend 84.86% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds Playwright E2E execution to CI.

Changes:

  • Runs Chromium and WebKit E2E jobs with browser caching.
  • Adds GitHub reporting and failure artifacts.
  • Increases CI workers to two.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
playwright.config.ts Configures CI parallelism and reporters.
.github/workflows/test.yml Adds matrix-based E2E workflow jobs.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.


test-e2e:
name: Test E2E (${{ matrix.browser }})
runs-on: ubuntu-latest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants