Skip to content

Setup Playwright e2e testing#63

Merged
AlexJSully merged 4 commits into
masterfrom
alexjsully-251019-setup-playwright
Oct 19, 2025
Merged

Setup Playwright e2e testing#63
AlexJSully merged 4 commits into
masterfrom
alexjsully-251019-setup-playwright

Conversation

@AlexJSully
Copy link
Copy Markdown
Owner

No description provided.

@AlexJSully AlexJSully requested a review from Copilot October 19, 2025 18:21
@AlexJSully AlexJSully self-assigned this Oct 19, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR sets up Playwright end-to-end testing infrastructure for the SciGrade application, including accessibility testing capabilities using axe-playwright. The changes integrate Playwright tests into the validation workflow and improve HTML semantic structure for better accessibility.

  • Adds comprehensive Playwright configuration with multi-browser and device testing
  • Implements accessibility testing utilities with axe-playwright integration
  • Creates initial homepage e2e tests with automated accessibility checks
  • Updates build pipeline to include Playwright tests in validation

Reviewed Changes

Copilot reviewed 11 out of 16 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
playwright.config.js Comprehensive test configuration with multi-browser/device support and webServer setup
tests/playwright/utils/base.js Base test utilities for extended Playwright functionality
tests/playwright/utils/accessibility.js Accessibility testing helper using axe-playwright
tests/playwright/homepage.spec.js Homepage e2e tests with accessibility validation
package.json Added Playwright dependencies and test scripts
jest.config.mjs Updated test paths and patterns for better separation
eslint.config.js Added Playwright-specific ESLint configuration
core/systemrun.html Improved semantic HTML structure with proper navigation elements
core/scripts/login.js Added accessibility label to gene dropdown
core/scripts/crispr_scripts.js Variable naming consistency and accessibility improvements
.github/workflows/code-qa.yaml Integrated Playwright tests into CI pipeline
Comments suppressed due to low confidence (1)

core/scripts/crispr_scripts.js:1

  • Corrected spelling of 'aira-label' to 'aria-label'.
/* eslint-disable indent */

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread tests/playwright/utils/base.js Outdated
Comment thread core/systemrun.html Outdated
Comment thread core/scripts/crispr_scripts.js Outdated
@AlexJSully AlexJSully merged commit 12911e4 into master Oct 19, 2025
5 checks passed
@AlexJSully AlexJSully deleted the alexjsully-251019-setup-playwright branch October 19, 2025 18:29
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.

2 participants