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

feat: add new test for increased coverage with keploy #182

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Sonichigo
Copy link

This pull request introduces comprehensive unit tests for the body-parser.js and error-handler.js module, significantly improving the project's test coverage from 0% to 80%+. I have also added the new tests for cookies.js increasing from 66.66% to 100%

The new tests cover various scenarios, ensuring the reliability and robustness of the functionalities.

Current Coverage

image

New Coverage

image

Changes:

  • Developed unit tests targeting key functions .
  • Validated edge cases and error handling mechanisms.
  • Ensured compatibility with existing codebase and testing frameworks.

Testing:

  • All new and existing tests have been executed successfully.
  • No regressions or issues were identified during testing.

Impact:

This enhancement bolsters code reliability and facilitates future development by providing a more robust testing framework.

Checklist:

  • Code builds cleanly without errors or warnings.
  • Unit tests have been added and pass successfully.

The PR content was generated automatically using testing-agent

@CLAassistant
Copy link

CLAassistant commented Feb 4, 2025

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Sonichigo
Copy link
Author

recheck

@Sonichigo Sonichigo changed the title feat: add new test for increased coverage feat: add new test for increased coverage with keploy Feb 4, 2025
@jackkav
Copy link
Contributor

jackkav commented Feb 5, 2025

If there's only white space change in package jsons please revert them. If not please don't reformat just install the package its too hard to read the white space diff

@Sonichigo
Copy link
Author

hi @jackkav, i have revert the package.json and added required libraries, as whitespaces were introduced due to my linter extension.

@jackkav
Copy link
Contributor

jackkav commented Feb 5, 2025

hi @jackkav, i have revert the package.json and added required libraries, as whitespaces were introduced due to my linter extension.

We use biome to lint and format

Signed-off-by: Animesh Pathak <[email protected]>
Signed-off-by: Animesh Pathak <[email protected]>
@Sonichigo
Copy link
Author

hi @jackkav, i have revert the package.json and added required libraries, as whitespaces were introduced due to my linter extension.

We use biome to lint and format

Done, I have fixed linting and formatting via Biome, now. Thanks for pointing to biome.

@Sonichigo
Copy link
Author

Hi @jackkav any updates on the pr or it's contents?

@jackkav
Copy link
Contributor

jackkav commented Feb 10, 2025

I need to discuss further with the team.

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.

3 participants