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.

Sign up for GitHub

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

Jump to bottom

test(protocol-designer): redesign E2E test beginning #16514

Open
wants to merge 2 commits into
base: edge
Choose a base branch
from
Open

test(protocol-designer): redesign E2E test beginning #16514

wants to merge 2 commits into from

Conversation

y3rsh
Copy link
Collaborator

@y3rsh y3rsh commented Oct 17, 2024 •
edited
Loading

Overview

Get something going to start exercising the PD revamp.

📢 To Alex for his initiative and working with me on the enableRedesign function

PDE2EConvoStarter.mp4

Goal 🎯

  • Set a strategic direction, guiding principles, and goals to establish effective E2E tests that the team collaboratively maintains.

TODO

  • How to name and organize tests for the revamp?
  • How to re-use the strings in contains? Not page objects exactly but what is the right pattern for Cypress?
  • Optimal it() organization and beforeEach for the new world
  • How much in a test?
  • I really want visual testing 😿 💸 💸 💸

@y3rsh y3rsh self-assigned this Oct 17, 2024
@y3rsh y3rsh requested a review from a team as a code owner October 17, 2024 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shlokamin shlokamin Awaiting requested review from shlokamin

@jerader jerader Awaiting requested review from jerader

At least 1 approving review is required to merge this pull request.

Assignees

@y3rsh y3rsh

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@y3rsh