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

[Feature Request] Unit Testing Rendering Functions #79

Open
shravan20 opened this issue Nov 30, 2020 · 9 comments
Open

[Feature Request] Unit Testing Rendering Functions #79

shravan20 opened this issue Nov 30, 2020 · 9 comments
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest hacktoberfest-accepted help wanted Extra attention is needed

Comments

@shravan20
Copy link
Owner

Is your feature request related to a problem? Please describe.
We have cross checked the working. Still, want an Unit Testing of UI components rendered to ensure the working.

Describe the solution you'd like
A proper unit testing of the functions that responsible for rendering svg component using Jest Library.

Describe alternatives you've considered
Have used Mocha for testing. But as you know, Mocha cannot test components rendering and stuff, we removed it in the initial release of v0.9. If you want to propose alternatives for Jest, please feel free to comment on the same.

@shravan20 shravan20 added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Nov 30, 2020
@Chahat226
Copy link

Hello,
Can I design a testing system using cypress as I have worked a lot on cypress and it's way easier to test in cypress rather than jest?

@shravan20
Copy link
Owner Author

Hi @Chahat226

We would love to take a view of the system on testing using Cypress. E2E Testing would be way better than Unit Testing alone..
Thanks a lot for considering to contribute to this project.

Your efforts counts.

If possible, can you give a proposal on how you are going to design the testing system?

Thank you so much :)

@devihall
Copy link

@shravan20 I have worked with Jest before. I'm happy to try my hand at this. Can you please assign it to me?

@rudransh61
Copy link

Hey is this issue still unsolved ?
Can i work on this ?
Please tell if there is any other issue also to resolve
Thanks for reading

@shravan20
Copy link
Owner Author

Hey is this issue still unsolved ?
Can i work on this ?
Please tell if there is any other issue also to resolve
Thanks for reading

Hi @rudransh61

Hey this is open for dev, you can give it a go.. 🙌🏼

@rudransh61
Copy link

Sorry to disturb you but i cannot find ui components in the code ?
Also Please can you explain more about the problem please ....

Thanks for reading and giving some time to me

@shravan20
Copy link
Owner Author

Sorry to disturb you but i cannot find ui components in the code ?
Also Please can you explain more about the problem please ....

Thanks for reading and giving some time to me

No worries, check frontend/src, this holds complete UI code.

@rudransh61
Copy link

...I was looking at the file frontend/src/App/App.test.js
And i dont know why ?
But its not working ??

 render(<App />);
         ^
SyntaxError: Unexpected token '<'
    at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:152:18)
    at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:299:14)
    at async link (node:internal/modules/esm/module_job:67:21)

Node.js v18.20.2

Also , do i have to write tests like this file for all components file ?

@rudransh61
Copy link

..?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest hacktoberfest-accepted help wanted Extra attention is needed
Projects
Github Readme Quote
Awaiting triage
Development

No branches or pull requests

4 participants