se-edu / addressbook-level4 / 4620

Builds Branch Commit Type Ran Committer Via Coverage
4620 getting-started-ui-tag PersonCard: Add colors to tag labels The tags are all displayed with the same color. It is hard to differentiate between tags visually across contacts. For example, it is hard to visually group all contacts tagged with "colleague". It is also ha... Pull #798 web-flow travis-ci pending completion   set done
4616 master ClearCommandTest: fix tautological tests Tests in ClearCommandTest always pass because the actual and expected model refer to the same instance; `assertEquals(expectedModel, actualModel)` is always true because expectedModel and actualModel refer... push Zhiyuan-Amos travis-ci pending completion   set done
4615 remove-addressbookname UserPrefs: remove addressBookName It is not used in our application at all. Pull #899 web-flow travis-ci pending completion   set done
4614 command-stateless Command: remove unnecessary internal state Pull #898 web-flow travis-ci pending completion   set done
4612 clear-command-test-fix ClearCommandTest: fix tautological tests Tests in ClearCommandTest always pass because the actual and expected model refer to the same instance; `assertEquals(expectedModel, actualModel)` is always true because expectedModel and actualModel refer... Pull #897 web-flow travis-ci pending completion   set done
4611 clear-command-test-fix ClearCommandTest: fix tautological tests `assertEquals(expectedModel, actualModel)` is always true because expectedModel and actualModel reference the same object. As a result, the model's state isn't correctly verified. Let's update these test... Pull #897 web-flow travis-ci pending completion   set done
4610 clear-command-test-fix ClearCommandTest: fix tautological tests Tests in ClearCommandTest always pass because the actual and expected model refer to the same instance; `assertEquals(expectedModel, actualModel)` is always true. As a result, the model's state isn't corr... Pull #897 web-flow travis-ci pending completion   set done
4609 master ParserUtil: remove `Optional` variants of parse methods As the comment[1] at the top of ParserUtil says, methods that take `Optional` as a parameter are bad. So, let's remove them. (It could be argued that leaving these methods in the code base... push pyokagan travis-ci pending completion   set done
4608 no-checked-exceptions UniquePersonListTest: add some more tests More test coverage is always nice. In particular, we want to check that UniquePersonList's methods throw DuplicatePersonException and PersonNotFoundException where relevant. These cases were not tested b... Pull #896 web-flow travis-ci pending completion   set done
4607 no-checked-exceptions UniquePersonListTest: add some more tests More test coverage is always nice. Pull #896 web-flow travis-ci pending completion   set done
4606 no-checked-exceptions UniquePersonListTest: add some more tests More test coverage is always nice. Pull #896 web-flow travis-ci pending completion   set done
4605 no-checked-exceptions UniquePersonListTest: add some more tests Pull #896 web-flow travis-ci pending completion   set done
4603 no-checked-exceptions UniquePersonListTest: add some more tests Pull #896 web-flow travis-ci pending completion   set done
4602 no-checked-exceptions commons: remove DuplicateDataException It is not used anymore. Pull #896 web-flow travis-ci pending completion   set done
4601 no-checked-exceptions AddressBookSystemTest: remove unnecessary catch block Pull #896 web-flow travis-ci pending completion   set done