-
Notifications
You must be signed in to change notification settings - Fork 38
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
Adding Cypress to the project #24
Comments
Hello there! Can I take a look at it? |
Hi @Loriage , You are welcome. You can take a look into it. Thanks. |
Hello @iusehooks ! Thanks in advance. |
Hi @Loriage , It should run using cypress cli. we should create a new scripts: "scripts": {
"cypress": "cypress run"
} no need for running in a website. Hope it helped. |
Hi, Of course for the new script! My question is: Should I run the test on a demo website/page. Usetheform is a node module if I have well understand, not a react application itself. So if I don't misunderstand, cypress is a webpage tester, no? |
Hi @Loriage, I got what you meant now. Tests should run locally. Thanks. |
Hello @iusehooks, I'm facing an issue when I'm trying to run a react app with the content of the repo as module. src/usetheform/hooks/useField.js Any ideas? Thanks. |
Hi @Loriage, may you copy and paste some code example? Thanks. |
@iusehooks Of course! ./testers/App.js:
|
Would you also please take a screen shoot of the project structure folders in your IDE? |
Here my installation script: So basically, I copy-paste the sourcefiles of usetheform to my react app, loading it as module and run the app. Thanks. |
testers/usrc/usetheform/ is a copy of src/ |
Would you please try to run in your local npm run build command and than copy and paste the /build/index.js file into src folder and rename it as usetheform.js, so you can remove the usetheform folder. A codesandbox example: https://codesandbox.io/s/still-hooks-tzodd |
would you please start a clean project downloading it from: https://codesandbox.io/s/still-hooks-tzodd ? |
The issue is how to launch a testable react app inside the usetheform directory without facing conflict between the repository and the test app. |
I think you do not need to use creare-react-app. I would suggest you to look into the folder named examples and take inspiration from it. You can excute the command "npm run dev" and than you may create a new .html file inside the folder named examples, then you are ok to start writing tests in cypress. A web server runs at http://localhost:3000/{yourfile}.html hope it helped. |
Working, thanks. I'll continue my tests! |
Hi @Loriage, which approach did you go for? If you like the library may you please give a github star to it? It helps the project to grow. Thanks. |
Hi @iusehooks, |
Hello @iusehooks ! First PR here: #33 Waiting for your feedback! |
Hey there, is it open so I can start working on this issue. |
Hey I'm new to open source contribution can you asign this challenge that can I enhance my skill... |
Hey I am new to open source, is this still open I just worked on cypress for some other project have a good understanding on this would like to help :) |
Hello @iusehooks , I found this from GoodFirstIssues, and this project seems very interesting to me, is this still open? I’m actually just working on adding Cypress to my company’s NestJS backend server with GraphQL. I’m eager to help contribute and add Cypress to this project ᕦ(ò_óˇ)ᕤ . Please assign this issue to me? Thanks! |
Hey @iusehooks is this issue still open? I want to contribute to this issue . |
Hi |
Hey There, I am new to open source and I found this on GoodFirstIssues so can I give it a try? |
Hi , |
Hi @iusehooks, Can I take up this issue? |
Hi @iusehooks, I have added remaining tests in my PR. Please let me know if anything has to be changed. |
Hey ~ i've been busy with work, But can I get this reassigned to me to complete the cypress suite? |
Hi! I'd like to work on this issue. Could I be assigned to it? |
We would like to add Cypress https://www.cypress.io/ to the project and write some automation tests.
The text was updated successfully, but these errors were encountered: