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

Set Up Cypress #5

Merged
merged 5 commits into from Apr 1, 2019

Conversation

Projects
None yet
5 participants
@amandagibson
Copy link
Member

commented Apr 1, 2019

Set up Cypress

Description

As a development team
In order to feature test our code
I would like to set up Cypress
Changes proposed in this pull request:

  • addition of cypress for testing

What I have learned working on this feature:

How to install cypress for testing

@LauraRe
Copy link
Collaborator

left a comment

cypress/integration/examples folder could be deleted, unless we want to keep it as reference.

@magnus-thor
Copy link
Member

left a comment

Either delete or gitignore the examples

@vick3d vick3d self-requested a review Apr 1, 2019

@vick3d vick3d self-assigned this Apr 1, 2019

@vick3d vick3d removed their assignment Apr 1, 2019

@perhegg

perhegg approved these changes Apr 1, 2019

@@ -0,0 +1 @@
{}

This comment has been minimized.

Copy link
@magnus-thor

magnus-thor Apr 1, 2019

Member

Add "video": false in there, otherwise it will record video

{
    "video": false
}

@perhegg perhegg merged commit d60822c into CraftAcademy:development Apr 1, 2019

"test": "react-scripts test",
"eject": "react-scripts eject",
"features": "jest -c jest.config.js",
"cypress:open": "cypress open"

This comment has been minimized.

Copy link
@magnus-thor

magnus-thor Apr 1, 2019

Member

rename the cypress open and add cypress run, this way is more inline with normal cypress setup

    "cy:run": "cypress run",
    "cy:open": "cypress open"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.