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

Unit and Integration testing for server Nodejs application #8

Closed
4 tasks done
mathanraj0601 opened this issue Dec 28, 2023 · 12 comments · Fixed by #64
Closed
4 tasks done

Unit and Integration testing for server Nodejs application #8

mathanraj0601 opened this issue Dec 28, 2023 · 12 comments · Fixed by #64
Assignees
Labels
enhancement New feature or request Medium Medium level issue SWOC24 Social Winter Of Code Season4

Comments

@mathanraj0601
Copy link
Contributor

mathanraj0601 commented Dec 28, 2023

Feature Request: Initiating Unit and Integration Testing

Overview

I am excited to propose the initiation of unit testing and integration testing. This feature aims to enhance the reliability and robustness of our application through comprehensive testing.

Testing Frameworks

  • Unit Testing: Jest

  • Integration Testing: SuperTest

Plan

  1. Unit Testing:

    • Identify critical components for testing.
    • Develop test cases for individual functions and modules.
  2. Integration Testing:

    • Configure Superset for integration testing.
    • Develop and execute integration tests.
@mathanraj0601
Copy link
Contributor Author

I am excited to work on this @AnupK1234 Can you assign me this for SWOC24?

@AnupK1234 AnupK1234 added the SWOC24 Social Winter Of Code Season4 label Dec 29, 2023
@AnupK1234
Copy link
Owner

Hey @mathanraj0601 ,
Sure! You can start working on this issue.
Kindly read the Contributing Guidelines carefully.

Thank You

@AnupK1234 AnupK1234 added enhancement New feature or request Medium Medium level issue labels Jan 1, 2024
@mathanraj0601
Copy link
Contributor Author

That's great @AnupK1234 . Sure i will go through guidelines

@mathanraj0601
Copy link
Contributor Author

Hey @AnupK1234 !, I am figuring out the test case for each module. Try to update with a PR mostly by the End of the week. I believe having separate PRs for each module's test cases would indeed make the review process smoother and more focused. Can I make every module test a separate PR

@AnupK1234
Copy link
Owner

Hey @AnupK1234 !, I am figuring out the test case for each module. Try to update with a PR mostly by the End of the week. I believe having separate PRs for each module's test cases would indeed make the review process smoother and more focused. Can I make every module test a separate PR

No, only make a single PR.

@mathanraj0601
Copy link
Contributor Author

Okay, i will make it as a single PR @AnupK1234 Thank you

@AnupK1234
Copy link
Owner

@mathanraj0601
Are you done with this issue?

@mathanraj0601
Copy link
Contributor Author

Hi @AnupK1234 , I have done integration testing for contact module. Need to cover other module integration and unit test. Most probably will update you with a PR this weekend. ( will update the task in the description accordingly )

@AnupK1234
Copy link
Owner

@mathanraj0601
Are you done with this issue?

@mathanraj0601
Copy link
Contributor Author

I am halfway through the issue i have exams and internship works so tight schedule hope you understand i will complete it ASAP

@AnupK1234
Copy link
Owner

@mathanraj0601
Are you done with issue ?

@mathanraj0601
Copy link
Contributor Author

Hey @AnupK1234, Tested two major backend routes. Will complete other-module within a day

mathanraj0601 referenced this issue in mathanraj0601/MealMatters Jan 21, 2024
mathanraj0601 referenced this issue in mathanraj0601/MealMatters Jan 21, 2024
mathanraj0601 referenced this issue in mathanraj0601/MealMatters Jan 21, 2024
AnupK1234 pushed a commit that referenced this issue Jan 26, 2024
* Solve issue #8 : Added test for Auth routes

* solve issue #8 : Added test for contact routes

* solve issue #8 : Updated Auth roue test

* solve isseu #8 : updated Contact route test

* Solve issue #8 : Added product route test

* Solve issue #8 : Added unit testing for Auth middlewares

* solve issue #8 : Update app.js with environment variable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Medium Medium level issue SWOC24 Social Winter Of Code Season4
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants