Skip to content

Conversation

@st3phhays
Copy link
Member

@st3phhays st3phhays commented May 5, 2025

Description Of Changes

Introduces the new choco-playwright package for Playwright tests.

Motivation and Context

We needed to centralize Playwright "things" so we could reuse across projects. This helps with maintainability.

Testing

  1. Run yarn.
  2. Run yarn choco-theme.
  3. Run yarn playwright. Wait a short time. You should see tests start, they should all pass.

Change Types Made

  • Minor documentation fix (typos etc.).
  • Major documentation change (refactoring, reformatting or adding documentation to existing page).
  • New documentation page added.
  • The change I have made should have a video added, and I have raised an issue for this.

Change Checklist

  • Requires a change to menu structure (top or left-hand side)/
  • Menu structure has been updated

Related Issue

@st3phhays st3phhays force-pushed the choco-playwright branch 4 times, most recently from f08a6a3 to 5af66ab Compare May 7, 2025 17:42
@st3phhays st3phhays changed the title WIP choco-playwright (#1178) Initialize choco-playwright May 7, 2025
@st3phhays st3phhays force-pushed the choco-playwright branch 2 times, most recently from f549a0a to 91776d3 Compare May 8, 2025 18:04
st3phhays added 2 commits May 8, 2025 13:38
This updates the tsconfig.json and files to
reference imports of choco-astro with the new path.
This was done to better create separation between
choco-astro paths and the new choco-playwright
paths.
The changes here utilize the new choco-astro package. A "baseline" test
is added here that was previously housed in choco-theme. This tests
performs basic actions to ensure Playwright is set up and running
correclty. The tsconfig.json has been updated slightly to remove options
that are not needed, as they are provided in the base tsconfig that it
is extending from already.
@st3phhays st3phhays force-pushed the choco-playwright branch from 91776d3 to c6597b4 Compare May 8, 2025 18:47
@st3phhays
Copy link
Member Author

A new version of choco-theme and choco-playwright must be released before this can be merged.

@st3phhays st3phhays requested a review from corbob May 8, 2025 20:00
@st3phhays st3phhays self-assigned this May 8, 2025
@st3phhays st3phhays force-pushed the choco-playwright branch 6 times, most recently from a0003a0 to ca2ba1f Compare May 13, 2025 17:11
This upgrades choco-theme to 1.2.0 and also installs choco-playwright
0.1.0.
@st3phhays st3phhays marked this pull request as ready for review May 13, 2025 17:51
Copy link
Member

@corbob corbob left a comment

Choose a reason for hiding this comment

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

Looks Good to Me

@corbob corbob merged commit 5d65069 into chocolatey:master May 13, 2025
1 check passed
@corbob
Copy link
Member

corbob commented May 13, 2025

Thanks for getting this fixed up @st3phhays

marcinho188

This comment was marked as resolved.

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.

3 participants