Skip to content

Cypress Runner #9875

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

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

Cypress Runner #9875

wants to merge 7 commits into from

Conversation

ksolo
Copy link
Contributor

@ksolo ksolo commented Dec 12, 2021

Description

Create a runner script for locally executing cypress tests.

Extracted vars to file that can be imported to each of the cypress runner scripts (ci, local), and update MAKEFILE to run them locally.

Issue ##10027

Copy link
Contributor

@nutrina nutrina left a comment

Choose a reason for hiding this comment

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

This works for me. Although some of the tests are failing. I assume fixing them is a separate issue.

@ksolo
Copy link
Contributor Author

ksolo commented Dec 13, 2021

@nutrina which tests are failing for you?

@nutrina
Copy link
Contributor

nutrina commented Dec 13, 2021

@ksolo right now it is not running any more. It can't connect to Chrome .... Do I need to reset something?

image

This is where Chrome got stuck:
Screenshot 2021-12-13 at 16 44 05

@thelostone-mc
Copy link
Contributor

  • It gets stuck at github login
  • System runs out of memory cause it takes in 96GB while running on my local machine

Screenshot 2021-12-14 at 10 33 27 AM

Screenshot 2021-12-14 at 10 31 46 AM

@thelostone-mc thelostone-mc marked this pull request as draft January 7, 2022 09:25
@ksolo ksolo force-pushed the testing/local-cypress-runner branch from df30e25 to 40e4fc7 Compare January 11, 2022 19:04
--browser chrome \
--headed \
--record \
--key 23c824d9-b9eb-4aea-88fd-d0bb06a9eb51
Copy link
Contributor

Choose a reason for hiding this comment

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

@ksolo : I would add --config video=false for local runs and remove the --record --key ....

@nutrina
Copy link
Contributor

nutrina commented Jan 14, 2022

I would also increase the default ETH balance, to make sure you can perform more transaction or run tests multiple times, by adding this param to the testrpc service: --defaultBalanceEther 1000000000

  testrpc:
    image: trufflesuite/ganache-cli
    ports:
      - "8545:8545"
    command: '--host 0.0.0.0 --mnemonic "${TEST_MNEMONIC}" --defaultBalanceEther 1000000000'
    deploy:
      replicas: 1
      resources:
        limits:
          memory: 1024M
        reservations:
          memory: 128M

@nutrina
Copy link
Contributor

nutrina commented Jan 14, 2022

Other than the observations above this looks good to me.

@ksolo
Copy link
Contributor Author

ksolo commented Feb 14, 2022

@nutrina Should I close this now since you have what you need in the other branch (which I believe is all merged)

@gdixon gdixon force-pushed the master branch 3 times, most recently from a15aeb3 to c0d45b0 Compare May 24, 2022 22:55
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.

3 participants