From 9cc37c9cc3a16c90f86aa5551346d42127e52417 Mon Sep 17 00:00:00 2001 From: Rita Lei Date: Sat, 14 Oct 2023 22:54:53 -0400 Subject: [PATCH 1/3] Notes for installing node.js I ran into a issue while installing cypress and this is how others can prevent making the same mistake. --- _docs/developer/testing/cypress.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/_docs/developer/testing/cypress.md b/_docs/developer/testing/cypress.md index 900c7f7e..a360aa7f 100644 --- a/_docs/developer/testing/cypress.md +++ b/_docs/developer/testing/cypress.md @@ -14,6 +14,8 @@ used to use Selenium, but has been rewritten in using Cypress. Make sure you have `node.js` version 10 or higher installed locally. You can check using the command: `node -v` +Make sure to install [`node.js`](https://www.nodejs.org/) on your desktop (inside Program Files) and not inside your local Submitty repository on your desktop. + ```bash # from the site directory npm install From 452825e802783736ae7c02e35bd8474137f529f4 Mon Sep 17 00:00:00 2001 From: Rita Lei Date: Sun, 15 Oct 2023 00:10:20 -0400 Subject: [PATCH 2/3] [Documentation: Developer] Note for people on WSL that may run into a issue installing Cypress Changes: Added "If you are on WSL and run into an error, make sure you installed `node.js` on your desktop (Program Files) and not inside your local Submitty repository." in cypress.md (https://submitty.org/developer/testing/cypress) WHY: I ran into a issue while installing cypress (I installed node.js in the wrong file) and this is to help others who might run into the same issue. --- _docs/developer/testing/cypress.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/_docs/developer/testing/cypress.md b/_docs/developer/testing/cypress.md index a360aa7f..b048eea7 100644 --- a/_docs/developer/testing/cypress.md +++ b/_docs/developer/testing/cypress.md @@ -14,7 +14,7 @@ used to use Selenium, but has been rewritten in using Cypress. Make sure you have `node.js` version 10 or higher installed locally. You can check using the command: `node -v` -Make sure to install [`node.js`](https://www.nodejs.org/) on your desktop (inside Program Files) and not inside your local Submitty repository on your desktop. +If you have WSL and run into an error, make sure you installed [`node.js`](https://www.nodejs.org/) on your desktop (Program Files) and not inside your local Submitty repository. ```bash # from the site directory From fa555f82cdf4e82b216a7b368ed17eb2bd005b81 Mon Sep 17 00:00:00 2001 From: Rita Lei Date: Sun, 15 Oct 2023 00:15:31 -0400 Subject: [PATCH 3/3] Updates --- _docs/developer/testing/cypress.md | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/_docs/developer/testing/cypress.md b/_docs/developer/testing/cypress.md index b048eea7..9947a64f 100644 --- a/_docs/developer/testing/cypress.md +++ b/_docs/developer/testing/cypress.md @@ -11,10 +11,7 @@ directly, but rather focuses on user navigation through the website. It used to use Selenium, but has been rewritten in using Cypress. ## Installation - -Make sure you have `node.js` version 10 or higher installed locally. You can check using the command: `node -v` - -If you have WSL and run into an error, make sure you installed [`node.js`](https://www.nodejs.org/) on your desktop (Program Files) and not inside your local Submitty repository. +Make sure you have [`node.js`](https://www.nodejs.org/) version 10 or higher installed locally. You can check using the command: `node -v` ```bash # from the site directory @@ -23,6 +20,8 @@ npm install If you are on Linux, view [this page](https://docs.cypress.io/guides/getting-started/installing-cypress.html#Linux) to see what dependencies you may need to install additionally for Cypress. +If you are on WSL and run into an error, make sure you installed `node.js` on your desktop (Program Files) and not inside your local Submitty repository. + ## Cypress Test Runner You can open the test runner by navigating into `site/` and using the command: