Repository: incubator-rya Updated Branches: refs/heads/master 193723396 -> 55716326a
RYA-163; Done. PCJDetails Test fails twice. Default timezone fixed at EST to match expected string. Closes #80 from DLotts/RYA-163_PCJDetails-non-EST Project: http://git-wip-us.apache.org/repos/asf/incubator-rya/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-rya/commit/55716326 Tree: http://git-wip-us.apache.org/repos/asf/incubator-rya/tree/55716326 Diff: http://git-wip-us.apache.org/repos/asf/incubator-rya/diff/55716326 Branch: refs/heads/master Commit: 55716326a7652c8aa1231cc257e20a3bb4e68f53 Parents: 1937233 Author: David W. Lotts <david.lo...@parsons.com> Authored: Wed Aug 24 14:52:19 2016 -0400 Committer: Aaron Mihalik <miha...@alum.mit.edu> Committed: Thu Aug 25 20:21:58 2016 -0400 ---------------------------------------------------------------------- .../src/test/java/mvm/rya/shell/RyaAdminCommandsTest.java | 4 ++++ .../test/java/mvm/rya/shell/util/RyaDetailsFormatterTest.java | 3 +++ 2 files changed, 7 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-rya/blob/55716326/extras/rya.console/src/test/java/mvm/rya/shell/RyaAdminCommandsTest.java ---------------------------------------------------------------------- diff --git a/extras/rya.console/src/test/java/mvm/rya/shell/RyaAdminCommandsTest.java b/extras/rya.console/src/test/java/mvm/rya/shell/RyaAdminCommandsTest.java index 1baec41..f842a93 100644 --- a/extras/rya.console/src/test/java/mvm/rya/shell/RyaAdminCommandsTest.java +++ b/extras/rya.console/src/test/java/mvm/rya/shell/RyaAdminCommandsTest.java @@ -27,6 +27,7 @@ import static org.mockito.Mockito.when; import java.io.IOException; import java.util.Date; import java.util.List; +import java.util.TimeZone; import org.junit.Test; @@ -123,6 +124,9 @@ public class RyaAdminCommandsTest { @Test public void getInstanceDetails() throws InstanceDoesNotExistException, RyaClientException { + // This test is failed if the default timezone was not EST, so now it's fixed at EST. + // If you get assert mismatch of EST!=EDT, try the deprecated getTimeZone("EST") instead. + TimeZone.setDefault(TimeZone.getTimeZone("America/New_York")); // Mock the object that performs the get operation. final GetInstanceDetails mockGetInstanceDetails = mock(GetInstanceDetails.class); final String instanceName = "test_instance"; http://git-wip-us.apache.org/repos/asf/incubator-rya/blob/55716326/extras/rya.console/src/test/java/mvm/rya/shell/util/RyaDetailsFormatterTest.java ---------------------------------------------------------------------- diff --git a/extras/rya.console/src/test/java/mvm/rya/shell/util/RyaDetailsFormatterTest.java b/extras/rya.console/src/test/java/mvm/rya/shell/util/RyaDetailsFormatterTest.java index d618283..62d94bd 100644 --- a/extras/rya.console/src/test/java/mvm/rya/shell/util/RyaDetailsFormatterTest.java +++ b/extras/rya.console/src/test/java/mvm/rya/shell/util/RyaDetailsFormatterTest.java @@ -21,6 +21,7 @@ package mvm.rya.shell.util; import static org.junit.Assert.assertEquals; import java.util.Date; +import java.util.TimeZone; import org.junit.Test; @@ -45,6 +46,8 @@ public class RyaDetailsFormatterTest { @Test public void format() { + // This test failed if the default timezone was not EST, so now it's fixed at EST. + TimeZone.setDefault(TimeZone.getTimeZone("America/New_York")); // Create the object that will be formatted. final RyaDetails details = RyaDetails.builder().setRyaInstanceName("test_instance") .setRyaVersion("1.2.3.4")
