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 test: calculateBalance() #72

Merged
merged 4 commits into from
Dec 8, 2023
Merged

Conversation

mikejensen0
Copy link
Collaborator

@mikejensen0 mikejensen0 commented Dec 5, 2023

resolves #56

Unit tests for CalculateBalance

Should give the following result by running npm test CalculateBalance.test.ts:
image

Copy link
Collaborator

@UnluckyBird UnluckyBird left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix the title and remove the duplicate, then it looks good

__tests__/unitTests/CalculateBalance.test.ts Outdated Show resolved Hide resolved
__tests__/unitTests/CalculateBalance.test.ts Outdated Show resolved Hide resolved
@mikejensen0 mikejensen0 changed the title Unittest calculatebalance Unit test: calculateBalance() Dec 6, 2023
Copy link
Owner

@andreasalstrup andreasalstrup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@andreasalstrup andreasalstrup merged commit a821579 into main Dec 8, 2023
1 check passed
@andreasalstrup andreasalstrup deleted the unittest-calculatebalance branch December 8, 2023 07:53
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.

Test: CalculateBalance helper function
3 participants