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 JUnit test code for components #114

Merged
merged 6 commits into from
Oct 24, 2019

Conversation

rohan-av
Copy link

Includes test code for the following classes:

  • Song
  • Bar
  • Note
  • Chord

NOTE: There are test methods not yet implemented. For those who worked on the specific methods, please add the appropriate test methods in the test files, as suggested by the todo markers in the comments of each file.

Works towards the completion of #111

Copy link

@jwyf jwyf left a comment

Choose a reason for hiding this comment

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

Thanks

@rohan-av rohan-av merged commit 0580a7e into AY1920S1-CS2113T-F09-4:master Oct 24, 2019
SalonetheGreat added a commit to SalonetheGreat/main that referenced this pull request Oct 24, 2019
Merge branch 'master' of https://github.com/SalonetheGreat/main

* 'master' of https://github.com/SalonetheGreat/main:
  Add author tags
  Complete Ui and storage support for ViewCommand, AddBarCommand, NewCommand, EditCommand, DeleteCommand and DeleteBarCommand
  Add ascii command syntax
  cleaned up checkstyle
  Added ascii command syntax
  Add JUnit test code for components (AY1920S1-CS2113T-F09-4#114)
  ASCII command modified to extend Commands
  changed storage

# Conflicts:
#	data/todo_list.txt
#	src/main/java/duke/Duke.java
#	src/main/java/duke/Parser.java
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