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

fix: qualify E2E testing scope #797

Merged
merged 2 commits into from
Mar 4, 2024

Conversation

MikeMcC399
Copy link
Collaborator

Issue

The repo description does not take account of the addition of Component Testing to the previously available End-to-End Testing. cypress-example-kitchensink provides for E2E Testing only, not Component Testing.

The repo also claims that "every API command in Cypress" is tested, which is not completely true.

Changes

The following locations are updated:

  1. The testing is qualified as "Cypress.io End-to-End (E2E) testing"
  2. A link to cypress-io/cypress-realworld-app is added to the README. ""

@cypress-app-bot
Copy link
Collaborator

@MikeMcC399
Copy link
Collaborator Author

Preview

README

Locally

npm start

Open http://localhost:8080/

image

@MikeMcC399

This comment was marked as resolved.

@MikeMcC399

This comment was marked as outdated.

@MikeMcC399

This comment was marked as resolved.

add link to cypress-io/cypress-realworld-app
@jennifer-shehane
Copy link
Member

@MikeMcC399 Is this ready for review?

@MikeMcC399 MikeMcC399 marked this pull request as ready for review March 4, 2024 19:23
@MikeMcC399
Copy link
Collaborator Author

@jennifer-shehane

Is this ready for review?

Yes, I was just waiting for CI to successfully complete, which it has now done.

README.md Show resolved Hide resolved
@jennifer-shehane jennifer-shehane merged commit 50e9439 into cypress-io:master Mar 4, 2024
10 checks passed
@cypress-app-bot
Copy link
Collaborator

🎉 This PR is included in version 2.0.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@MikeMcC399 MikeMcC399 deleted the testing-scope branch March 5, 2024 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing API commands
3 participants