Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transition of PCP autotests is completed #2950

Open
wants to merge 7 commits into
base: trunk
Choose a base branch
from

Conversation

sedatbsr
Copy link
Collaborator

Issue: #

Ticket:

Slack Thread:


Description

Steps to Test

  1. Go to '…'
  2. Click on '…'
  3. Scroll down to …'

Documentation

  • This PR needs documentation (has the "Documentation" label).

Changelog Entry

Enter a summary of all changes on this Pull Request. This will appear in the changelog if accepted.

Closes # .

@sedatbsr sedatbsr requested a review from mishautkin February 3, 2025 09:34
-Visual tests moved to PCP repo.
-isCurrentStep() and gotoInitialOnboardingPage() funtions fixed.
Copy link
Collaborator

@mishautkin mishautkin left a comment

Choose a reason for hiding this comment

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

There are many comments mostly related to the cleanup of the functionality related to older version of UI. Additionallyt this is a "fork" PR so it can't be actually merged inside the repo. Please creata a qa branch and only transfer there updates relevant to new UI.

tests/qa/tests/.setup/woocommerce.setup.ts Show resolved Hide resolved
Copy link
Collaborator

Choose a reason for hiding this comment

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

This file contains older user accounts, which most likely will not work with new UI. Only actual (+ invalid) users should be present.

tests/qa/.github/workflows/pcp-tests.yml Show resolved Hide resolved
tests/qa/.gitignore Outdated Show resolved Hide resolved
Copy link
Collaborator

Choose a reason for hiding this comment

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

The functionality in this file must be refined according to current stage of automation for new UI.

tests/qa/resources/paypal-button-colors.json Show resolved Hide resolved
tests/qa/resources/pcp-config.ts Show resolved Hide resolved
Copy link
Collaborator

Choose a reason for hiding this comment

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

Check if this setup data can be used with new UI. Same for all other files in /tests/.setup

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