insomnia-mockbin icon indicating copy to clipboard operation
insomnia-mockbin copied to clipboard

feat: add new test for increased coverage with keploy

Open Sonichigo opened this issue 10 months ago • 9 comments

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:

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

The PR content was generated automatically using testing-agent

Sonichigo avatar Feb 04 '25 10:02 Sonichigo

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Feb 04 '25 10:02 CLAassistant

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.

CLAassistant avatar Feb 04 '25 10:02 CLAassistant

recheck

Sonichigo avatar Feb 04 '25 11:02 Sonichigo

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

jackkav avatar Feb 05 '25 07:02 jackkav

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

Sonichigo avatar Feb 05 '25 09:02 Sonichigo

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

jackkav avatar Feb 05 '25 12:02 jackkav

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 avatar Feb 05 '25 13:02 Sonichigo

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

Sonichigo avatar Feb 10 '25 10:02 Sonichigo

I need to discuss further with the team.

jackkav avatar Feb 10 '25 10:02 jackkav