Tags: nus-cs2103-AY1718S1/addressbook-level4-old
Tags
build.gradle: upgrade TestFX to 4.0.7-alpha TestFX 4.0.7-alpha has been released. Besides being the latest and greatest version, 4.0.7-alpha also contains a proper working headless mode for Linux. Without this upgrade, attempting to run `./gradlew headless guiTests` on Linux would result with the GUI tests failing with the following exception: java.lang.RuntimeException: java.lang.RuntimeException: java.lang.IllegalAccessException: Class org.testfx.toolkit.impl.ApplicationLauncherImpl can not access a member of class com.sun.glass.ui.monocle.HeadlessPlatform with modifiers "" Caused by: java.lang.RuntimeException: java.lang.IllegalAccessException: Class org.testfx.toolkit.impl.ApplicationLauncherImpl can not access a member of class com.sun.glass.ui.monocle.HeadlessPlatform with modifiers "" Caused by: java.lang.IllegalAccessException: Class org.testfx.toolkit.impl.ApplicationLauncherImpl can not access a member of class com.sun.glass.ui.monocle.HeadlessPlatform with modifiers "" Let's upgrade TestFX to 4.0.7-alpha to have happy Linux developers. This working headless mode was also present in the 4.0.6-alpha, but we couldn't upgrade to it as it broke our gui tests. (See e1eafd1 ([se-edu#353] All gui tests are failing (se-edu#354), 2017-03-23)) Furthermore, since headless mode now works on Linux, our Travis CI builds do not need to run xvfb any more. As such, let's remove its usage from .travis.yml.
[se-edu#621] UniquePersonList: Simplify logic of updatePerson(...) (s… …e-edu#616) This method retrieves the person in the list to be updated, and calls Person#resetData(ReadOnlyPerson) to update the values of that person. We do not need to retrieve the person in the list to be updated, to update the list accordingly. Let’s simplify the logic by setting the updated Person object in place of the person to be updated.
[se-edu#534] HistoryCommand: Returns previous commands in reverse chr… …onological order (se-edu#537) The HistoryCommand returns the previous commands in chronological order. This goes against the intuition that history ought to be viewed in reverse chronological order. We are expecting that the user will want to see what he has typed most recently, just like how Google Chrome History works. Let’s update HistoryCommand to return previous commands in reverse chronological order.