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/change cypress paths #401

Merged
merged 2 commits into from
Nov 30, 2022
Merged

Fix/change cypress paths #401

merged 2 commits into from
Nov 30, 2022

Conversation

vibalre
Copy link
Member

@vibalre vibalre commented Nov 30, 2022

Changes

Closes #400

How to test

  1. Create a new Rhino project.
  2. Push project to Github repository.
  3. Run Cypress end-to-end tests passes.

Notes

See rhino-showcase updated to Rhino 1.2.0 passing CI for example.

@vibalre vibalre force-pushed the fix/change-cypress-paths branch from ea7afbb to 3c688d5 Compare November 30, 2022 10:20
Changes working directory for cypress from .rhino/node to
.rhino, and project path from root/tests to ../tests.
All node stuff are now in .rhino and not in .rhino/node.
When cypress run tests, cypress looks for cypress.json
in .rhino/root/tests when it should look at the ./tests.
Cypress seems to be still stuck in .rhino, so it needs
to move a directory higher (..) to locates tests/ and
cypress.json.
@vibalre vibalre force-pushed the fix/change-cypress-paths branch from 3c688d5 to ed5f179 Compare November 30, 2022 10:28
@kamilzyla kamilzyla merged commit 2854094 into main Nov 30, 2022
@kamilzyla kamilzyla deleted the fix/change-cypress-paths branch November 30, 2022 10:31
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.

Cypress test fails in Github Actions due to wrong node and test directory
2 participants