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

Cypress Testing on CI #9478

Merged
merged 4 commits into from
Sep 17, 2021
Merged

Cypress Testing on CI #9478

merged 4 commits into from
Sep 17, 2021

Conversation

ksolo
Copy link
Contributor

@ksolo ksolo commented Sep 13, 2021

Description

Adding Cypress tests with CI configurations to run tests on Travis.

Includes:

  • ignoring artifacts from cypress run in gitignore (videos and screenshots)
  • update travis yml to include the xvfb service and chrome addon
  • update travis yml to call the cypress-run script
  • cypress-run script handles additional configuration to setup ganache, db, static assets and also run the app and cypress tests
  • update docker files to build chrome to run tests in a container
  • add initial cypress tests

@ksolo ksolo changed the title Red squirrel Cypress Testing on CI Sep 14, 2021
cypress.json Show resolved Hide resolved
@thelostone-mc
Copy link
Member

thelostone-mc commented Sep 17, 2021

The commit history seems to have other commit not related this PR !
Any idea why that's there ?

Would we need additional documentation to explain how to use/run this locally / is it not required?
I haven't pulled this in locally -> but codewise LGTM

@ksolo
Copy link
Contributor Author

ksolo commented Sep 17, 2021

@thelostone-mc I can rebase this and remove the other commit. I'll add documentation for running tests locally as well.

ksolo and others added 3 commits September 17, 2021 11:26
* updating dependencies to use custom fork of cypress-metamask

* add script and options for running cypress tests

* add wait-on package for scripting cypress runs

* travis docs indicate that bionic will start xvfb by adding it as a service

* add migration step

* another attempt at including cypress tests

* redirect webserver logs to /dev/null

* bad idea to move stdout to dev/null

* add NETWORK_NAME to travis env file

* update runserver command to go to background

* updating existing cypress tests to run on ci

* update docker-compose file: use default accounts for ganache

* add set -x to get more visibility into cypress command args

* update call to cypress run to see if travis will recognize flags

* try running cypress directly from node_modules

* remove set -x now that arguments are making their way to cypress
* updating dependencies to use custom fork of cypress-metamask

* add script and options for running cypress tests

* redirect webserver logs to /dev/null

* update docker-compose file: use default accounts for ganache

* test running connect site with metamask on ci

* attempt at passing in env from config

* add ganache-cli to dev dependencies

* updated script call

* moving where NETWORK_NAME is defined

* build please

* moving network_name again

* remove network_name

* add superuser creation to script

* take two at creating a superuser

* third times a charm - create superuser

* try loading user data from fixtures

* scroll metamask button into view

* add cypress settings to gain access to screenshots/videos

* no longer need the scrolling command
@zlsgh zlsgh merged commit 29e8f78 into gitcoinco:master Sep 17, 2021
@ksolo ksolo deleted the red-squirrel branch September 20, 2021 19:22
jeremyschuurmans pushed a commit that referenced this pull request Sep 22, 2021
* Travis running cypress (#9411)

* updating dependencies to use custom fork of cypress-metamask

* add script and options for running cypress tests

* add wait-on package for scripting cypress runs

* travis docs indicate that bionic will start xvfb by adding it as a service

* add migration step

* another attempt at including cypress tests

* redirect webserver logs to /dev/null

* bad idea to move stdout to dev/null

* add NETWORK_NAME to travis env file

* update runserver command to go to background

* updating existing cypress tests to run on ci

* update docker-compose file: use default accounts for ganache

* add set -x to get more visibility into cypress command args

* update call to cypress run to see if travis will recognize flags

* try running cypress directly from node_modules

* remove set -x now that arguments are making their way to cypress

* Cypress testing/add connection metamask wallet test (#9429)

* updating dependencies to use custom fork of cypress-metamask

* add script and options for running cypress tests

* redirect webserver logs to /dev/null

* update docker-compose file: use default accounts for ganache

* test running connect site with metamask on ci

* attempt at passing in env from config

* add ganache-cli to dev dependencies

* updated script call

* moving where NETWORK_NAME is defined

* build please

* moving network_name again

* remove network_name

* add superuser creation to script

* take two at creating a superuser

* third times a charm - create superuser

* try loading user data from fixtures

* scroll metamask button into view

* add cypress settings to gain access to screenshots/videos

* no longer need the scrolling command

* add updated cypress-run to red-squirrel

* create make target and documentation for running cypress tests
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.

4 participants