Added some tests for Time Identified Classes and SalesHistory

[ParasK26]

No description provided.

[coveralls]

Pull Request Test Coverage Report for Build 316

• 3 of 32 (9.38%) changed or added relevant lines in 8 files are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage increased (+2.7%) to 20.767%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/main/java/seedu/address/logic/commands/RemoveReminderCommand.java	0	1	0.0%
src/main/java/seedu/address/logic/parser/AllCommandsParser.java	0	2	0.0%
src/main/java/seedu/address/logic/parser/FinishedReminderCommandParser.java	0	2	0.0%
src/main/java/seedu/address/model/timeidentifiedclass/TimeIdentifiedClass.java	2	4	50.0%
src/main/java/seedu/address/logic/LogicManager.java	0	3	0.0%
src/main/java/seedu/address/model/ModelManager.java	0	9	0.0%
src/main/java/seedu/address/logic/commands/ViewAllRemindersCommand.java	0	10	0.0%

Files with Coverage Reduction	New Missed Lines	%
src/main/java/seedu/address/model/ModelManager.java	1	23.04%

Totals
Change from base Build 311:	2.7%
Covered Lines:	704
Relevant Lines:	3390

---

💛 - Coveralls