Skip to content
This repository has been archived by the owner on Jul 26, 2019. It is now read-only.

Add unit tests for command classes #77

Closed
wants to merge 3 commits into from
Closed

Add unit tests for command classes #77

wants to merge 3 commits into from

Conversation

xjh666
Copy link

@xjh666 xjh666 commented Mar 2, 2017 •
edited
Loading

Add AddCommandTest, DeleteCommandTest, FindCommandTest, ViewAllCommandTest and ViewCommandTest
Copy the code form level 2
Modify some parts to make the code suitable for level 3
#52 #53 #54 #55

@xjh666
Use the AddCommandTest in AddressBook level 2
c08dc86 
Modify Address class to ensure that empty address is not allowed
@xjh666
Add unit tests to the DeleteCommand class
84a2dc1
@xjh666 xjh666 changed the title Add unit tests to the AddCommand class Add unit tests to the AddCommand, DeleteCommand class Mar 2, 2017
@xjh666 xjh666 changed the title Add unit tests to the AddCommand, DeleteCommand class Add unit tests for command classes Mar 3, 2017
@MightyCupcakes
Copy link
Contributor

Hello,

Addressbook is currently following the PR workflow outlined in OSS generic

In particular:

  • Please only fix one issue per PR. If you want to work on multiple issues, open separate PRs for each issue.
  • PR title should be the issue title including its identification number

@xjh666 xjh666 closed this Mar 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xjh666 @MightyCupcakes