-
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
Add tests for RecycleBin, Remove Outdated AddressBook Files #219
Conversation
Moved the restore command to function on the deleted list (recycle bin) which is an improvement in functionality. Supporting tests and documentation have also been added and verified. |
minor check style issue. I think you can remove the ignore import for the parser util test since there is no more ignores inside that test file. |
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.
Other than minor issues, I see no other glaring problems. Approved.
EmptyBinCommandTest
ExitBinCommandTest
RecycleBinCommandTest
with completed test functionalityRestoreCommandTest
which is still currently in progressrestore
command from SourceManagerParser (added from PR Update for Biblio related features #217) as it should now only be supported in RecycleBinParserImplements #226
Partially Implements #215