Pierce Fu

FileUtil: isFileExists(File) may be unnecessary

3

The current method `FileUtil#isFileExists(File)` is basically a combination of two boolean method calls: ``` public static boolean isFileExists(File file) { return file.exists() && file.isFile(); } ``` However, as per the...

Modules used to be named `ModuleNameCheck`, but after #279, they are now named `ModuleName`. This change has made `altname` metadata tags redundant as their values are of the form `ModuleName`,...

Unit tests in AddressBookTest needs improvement

1

Unit tests do not confirm that a person is added correctly in `AddressBook#addPerson()`.