Skip to content

V13 QA updated pipeline for acceptance to avoid issue when installing playwright#19140

Merged
andr317c merged 3 commits intov13/devfrom
v13/QA/update-pipeline
Apr 29, 2025
Merged

V13 QA updated pipeline for acceptance to avoid issue when installing playwright#19140
andr317c merged 3 commits intov13/devfrom
v13/QA/update-pipeline

Conversation

@andr317c
Copy link
Contributor

We were having issues installing some packages when using the npx playwright install --with-deps command on linux. We now only install the chromium browser as it is the only one we currently use.

The test artifacts were also updated to output to junit so we can inspect failed tests directly on the pipeline run

Copilot AI review requested due to automatic review settings April 24, 2025 16:17
Copy link
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 updates the acceptance pipeline to resolve installation issues with Playwright on Linux by only installing the Chromium browser and enhancing test result reporting.

  • Updated reporter configuration in Playwright to output results in both line and JUnit formats for CI.
  • Modified the Azure Pipelines installation command to install only Chromium.
  • Added tasks to publish test artifacts and JUnit test results.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
tests/Umbraco.Tests.AcceptanceTest/playwright.config.ts Adjusted reporter configuration to include JUnit output in CI.
build/azure-pipelines.yml Updated Playwright installation to Chromium only and added test publishing tasks.

Copy link
Contributor

@nhudinh0309 nhudinh0309 left a comment

Choose a reason for hiding this comment

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

LGTM

@andr317c andr317c enabled auto-merge (squash) April 29, 2025 11:18
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