Got rid of static variable uniqueIDgenerator. Instead i will have the uniqueId generator in ContactManager as a private variable. This way i don't have to use static variables that are having strange effects on my tests. Id will now be passed into ContactImpl from contactManager through a constructor
Also got rid of uniqueIdgenerator in MeetingImpl
Added ID to past and future meeting constructors
Added uniqueIdGenerators to contactManager
Made changes to ContactTest needed after i changed the constructor and way the uniqueId is generated. No longer need the tests to run in sequential order.
Added another test to ContactTest to check both parameters work fine in constructor method
Made changes to meetingTest now that uniqueId is generated in a different way. Tests pass as expected
Don't understand why contactset.contains(contact) is returning false after i have added the contact. I think it might have something to do with the hashcode or equals method in contactimpl
Overrode equals method in ContactImpl
The reason the junit test was not working was because eventhough all the variables in the 2 contacts were the same, their hashcodes were not the same. So i overide hashCode in ContactImpl so the hashCode generated will be equal for contactImpl's that have the same variables