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

Add toString tests to MarkLoanCommand, UnmarkLoanCommand and DeleteLoanCommand #209

Merged

Conversation

kjw142857
Copy link

Adds toString tests to the MarkLoanCommandTest, UnmarkLoanCommandTest and DeleteLoanCommandTest classes, to standardise unit tests with the other commands.

@kjw142857 kjw142857 added priority.Low Unlikely to do type.Testing All things related to software testing labels Apr 14, 2024
@kjw142857 kjw142857 added this to the v1.4 milestone Apr 14, 2024
@kjw142857 kjw142857 requested a review from xiaorui-ui April 14, 2024 10:06
@kjw142857 kjw142857 self-assigned this Apr 14, 2024
Copy link

@xiaorui-ui xiaorui-ui left a comment

Choose a reason for hiding this comment

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

lgtm

@xiaorui-ui xiaorui-ui merged commit 36c639e into AY2324S2-CS2103T-W13-1:master Apr 14, 2024
3 checks passed
@kjw142857 kjw142857 mentioned this pull request Apr 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority.Low Unlikely to do type.Testing All things related to software testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants