-
Notifications
You must be signed in to change notification settings - Fork 5
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
Added test for BiblioEditCommand #227
Added test for BiblioEditCommand #227
Conversation
Update merge from 13-3/main
Looking at the failed tests, it seems like the typical sources have not been updated appropriately. Perhaps you can check if the biblio fields in the typical sources have been accounted for in the .equals() override command (I had a similar problem for author but rectified it relatively easily). Can also check the typical sources file itself and see if the bibilo fields have been entered accurately. If they are not correct then the parser will not work properly and cause it to fail. |
…runned with all other tests
…ot when runned with all other tests" This reverts commit 486159f.
Tests are passing when run on their own but not when runned with all other tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Additional test functionality for the biblio edit command. One of the test is set to @ignore due to unforeseen circumstances involving test interaction.
Approved for now but will need to see if the test can be rectified in the future (by future i mean tomorrow).
Updated Default sources to have more fields that are filled.
Updated Sample sources to match test TypicalSources