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

Toll Fee calculation done in Node Js #100

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

Conversation

karthick-krishnan
Copy link

@karthick-krishnan karthick-krishnan commented Mar 6, 2022

Hi Team,

This is Karthick Krishnan. I have done the Toll fee calculation in Node Js. Please do check it out. Hopefully we can have a chat soon regarding this. I have made some small changes in the logic of the calculation and made changes in getting the toll fee for each hour dynamically. I have added Unit test cases. I believe i have covered all the scenarios. I firmly believe that when a functionality is unit tested thoroughly with full coverage results then only its production ready. I have made my part as a production ready code please do check and let me know if i have to make any changes. I am listing the commands to run and the screenshot of the unit cases results coverage report below.

The movie name in the GIF is Hackers :)

Installation Requirements
node js
npm
jest

Commands to run the test cases:
npm install
npm run test

Screen Shot 2022-03-07 at 2 58 00 AM

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.

2 participants