Skip to content

Conversation

@JP-sDEV
Copy link

@JP-sDEV JP-sDEV commented Oct 13, 2024

  • refactored test directory to include unit tests - allows for flexibility (e.g. include integration tests in the future)
  • added test script npm run unit:test to run all unit tests
  • included the edge and "happy days" test cases to all functions found in lib/utils.js

Suggestion: combine test/package.json with package.json (i.e. found in the root), to simplify the scripts and requirements. This also allows for jest to analyze test coverage.

@vercel
Copy link

vercel bot commented Oct 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
hasty-server ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 13, 2024 3:15am

@IntegerAlex
Copy link
Owner

Looks clean. Let me test

@JP-sDEV
Copy link
Author

JP-sDEV commented Oct 13, 2024

To run the test cd in the test directory and run npm run test:unit

@IntegerAlex
Copy link
Owner

i am extremely sorry, I didn't opened my system today

@IntegerAlex
Copy link
Owner

@JP-sDEV I am working on some patches i will merge it after making some changes

@IntegerAlex IntegerAlex added hacktoberfest-accepted Test Any thing related to tests labels Oct 13, 2024
@IntegerAlex
Copy link
Owner

Many tests are failing almost every @JP-sDEV But it shouldn't all expected tests are not as per format. HTTP format is not like just simple text. Make it at least pass 80%

@JP-sDEV
Copy link
Author

JP-sDEV commented Oct 14, 2024

@IntegerAlex sure, I will revise the tests to match the project specs. What content-types are supported? Also is queryParser still in development, I ask because it is failing most of the tests.

@IntegerAlex
Copy link
Owner

Okay i will fix Query parser

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted Test Any thing related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants