Merge pull request google#19 from google/move-time

vaage · web-flow · commit f058608e1b7f · 2017-04-04T15:12:31.000-04:00
Reworking Test
diff --git a/src/codeu/chat/client/View.java b/src/codeu/chat/client/View.java
@@ -23,10 +23,10 @@
 import codeu.chat.common.LogicalView;
 import codeu.chat.common.Message;
 import codeu.chat.common.NetworkCode;
-import codeu.chat.common.Time;
 import codeu.chat.common.User;
 import codeu.chat.util.Logger;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 import codeu.chat.util.connections.Connection;
 import codeu.chat.util.connections.ConnectionSource;
diff --git a/src/codeu/chat/common/Conversation.java b/src/codeu/chat/common/Conversation.java
@@ -22,6 +22,7 @@
 
 import codeu.chat.util.Serializer;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 
 public final class Conversation {
diff --git a/src/codeu/chat/common/ConversationSummary.java b/src/codeu/chat/common/ConversationSummary.java
@@ -20,6 +20,7 @@
 
 import codeu.chat.util.Serializer;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 
 public final class ConversationSummary implements ListViewable {
diff --git a/src/codeu/chat/common/LogicalView.java b/src/codeu/chat/common/LogicalView.java
@@ -16,6 +16,7 @@
 
 import java.util.Collection;
 
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 
 // LOGICAL VIEW
diff --git a/src/codeu/chat/common/Message.java b/src/codeu/chat/common/Message.java
@@ -20,6 +20,7 @@
 
 import codeu.chat.util.Serializer;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 
 public final class Message {
diff --git a/src/codeu/chat/common/RawController.java b/src/codeu/chat/common/RawController.java
@@ -14,6 +14,7 @@
 
 package codeu.chat.common;
 
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 
 // RAW CONTROLLER
diff --git a/src/codeu/chat/common/Relay.java b/src/codeu/chat/common/Relay.java
@@ -16,7 +16,7 @@
 
 import java.util.Collection;
 
-import codeu.chat.common.Time;
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 
 // RELAY
diff --git a/src/codeu/chat/common/User.java b/src/codeu/chat/common/User.java
@@ -20,6 +20,7 @@
 
 import codeu.chat.util.Serializer;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 
 public final class User {
diff --git a/src/codeu/chat/relay/Server.java b/src/codeu/chat/relay/Server.java
@@ -24,7 +24,8 @@
 
 import codeu.chat.common.LinearUuidGenerator;
 import codeu.chat.common.Relay;
-import codeu.chat.common.Time;
+import codeu.chat.util.Logger;
+import codeu.chat.util.Time;
 import codeu.chat.util.Logger;
 import codeu.chat.util.Uuid;
 
diff --git a/src/codeu/chat/relay/ServerFrontEnd.java b/src/codeu/chat/relay/ServerFrontEnd.java
@@ -21,10 +21,10 @@
 
 import codeu.chat.common.NetworkCode;
 import codeu.chat.common.Relay;
-import codeu.chat.common.Time;
 import codeu.chat.util.Logger;
 import codeu.chat.util.Serializer;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 import codeu.chat.util.connections.Connection;
 
diff --git a/src/codeu/chat/server/Controller.java b/src/codeu/chat/server/Controller.java
@@ -20,9 +20,9 @@
 import codeu.chat.common.Conversation;
 import codeu.chat.common.Message;
 import codeu.chat.common.RawController;
-import codeu.chat.common.Time;
 import codeu.chat.common.User;
 import codeu.chat.util.Logger;
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 
 public final class Controller implements RawController, BasicController {
diff --git a/src/codeu/chat/server/Model.java b/src/codeu/chat/server/Model.java
@@ -20,11 +20,11 @@
 import codeu.chat.common.ConversationSummary;
 import codeu.chat.common.LinearUuidGenerator;
 import codeu.chat.common.Message;
-import codeu.chat.common.Time;
 import codeu.chat.common.User;
+import codeu.chat.util.Time;
+import codeu.chat.util.Uuid;
 import codeu.chat.util.store.Store;
 import codeu.chat.util.store.StoreAccessor;
-import codeu.chat.util.Uuid;
 
 public final class Model {
 
diff --git a/src/codeu/chat/server/NoOpRelay.java b/src/codeu/chat/server/NoOpRelay.java
@@ -18,10 +18,9 @@
 import java.util.Collection;
 
 import codeu.chat.common.Relay;
-import codeu.chat.common.Time;
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 
-
 // NO OP RELAY
 //
 // The no op relay is an empty implementation of the relay interface
diff --git a/src/codeu/chat/server/RemoteRelay.java b/src/codeu/chat/server/RemoteRelay.java
@@ -22,10 +22,10 @@
 
 import codeu.chat.common.NetworkCode;
 import codeu.chat.common.Relay;
-import codeu.chat.common.Time;
 import codeu.chat.util.Logger;
 import codeu.chat.util.Serializer;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 import codeu.chat.util.connections.Connection;
 import codeu.chat.util.connections.ConnectionSource;
diff --git a/src/codeu/chat/server/Server.java b/src/codeu/chat/server/Server.java
@@ -29,10 +29,10 @@
 import codeu.chat.common.Message;
 import codeu.chat.common.NetworkCode;
 import codeu.chat.common.Relay;
-import codeu.chat.common.Time;
 import codeu.chat.common.User;
 import codeu.chat.util.Logger;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 import codeu.chat.util.Timeline;
 import codeu.chat.util.Uuid;
 import codeu.chat.util.connections.Connection;
diff --git a/src/codeu/chat/server/View.java b/src/codeu/chat/server/View.java
@@ -33,9 +33,9 @@
 import codeu.chat.common.LogicalView;
 import codeu.chat.common.Message;
 import codeu.chat.common.SinglesView;
-import codeu.chat.common.Time;
 import codeu.chat.common.User;
 import codeu.chat.util.Logger;
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 import codeu.chat.util.store.StoreAccessor;
 
diff --git a/src/codeu/chat/util/Time.java b/src/codeu/chat/util/Time.java
@@ -12,25 +12,22 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package codeu.chat.common;
+package codeu.chat.util;
 
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
 import java.text.SimpleDateFormat;
 import java.util.Date;
 
-import codeu.chat.util.Serializer;
-import codeu.chat.util.Serializers;
-
 public final class Time implements Comparable<Time> {
 
   public static final Serializer<Time> SERIALIZER = new Serializer<Time>() {
 
     @Override
     public void write(OutputStream out, Time value) throws IOException {
 
-      Serializers.LONG.write(out, value.totalMs);
+      Serializers.LONG.write(out, value.inMs());
 
     }
 
@@ -45,24 +42,24 @@ public Time read(InputStream in) throws IOException {
   private static final SimpleDateFormat formatter =
       new SimpleDateFormat("dd-MMM-yyyy HH:mm:ss.SSS");
 
-  private final long totalMs;
+  private final Date date;
 
-  private Time(long totalMs) { this.totalMs = totalMs; }
+  private Time(long totalMs) { this.date = new Date(totalMs); }
 
-  public long inMs() { return totalMs; }
+  public long inMs() { return date.getTime(); }
 
   @Override
   public int compareTo(Time other) {
-    return Long.compare(totalMs, other.totalMs);
+    return date.compareTo(other.date);
   }
 
   public boolean inRange(Time start, Time end) {
-    return totalMs >= start.totalMs && totalMs <= end.totalMs;
+    return this.compareTo(start) >= 0 && this.compareTo(end) <= 0;
   }
 
   @Override
   public String toString() {
-    return formatter.format(new Date(totalMs));
+    return formatter.format(date);
   }
 
   public static Time fromMs(long ms) { return new Time(ms); }
diff --git a/test/codeu/chat/TestRunner.java b/test/codeu/chat/TestRunner.java
@@ -26,6 +26,7 @@ public static void main(String[] args) {
              codeu.chat.relay.ServerTest.class,
              codeu.chat.server.BasicControllerTest.class,
              codeu.chat.server.RawControllerTest.class,
+             codeu.chat.util.TimeTest.class,
              codeu.chat.util.UuidTest.class,
              codeu.chat.util.store.StoreTest.class
          );
diff --git a/test/codeu/chat/relay/ServerTest.java b/test/codeu/chat/relay/ServerTest.java
@@ -21,7 +21,7 @@
 import org.junit.Before;
 
 import codeu.chat.common.Relay;
-import codeu.chat.common.Time;
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 
 public final class ServerTest {
diff --git a/test/codeu/chat/server/RawControllerTest.java b/test/codeu/chat/server/RawControllerTest.java
@@ -21,8 +21,8 @@
 import codeu.chat.common.Conversation;
 import codeu.chat.common.Message;
 import codeu.chat.common.RawController;
-import codeu.chat.common.Time;
 import codeu.chat.common.User;
+import codeu.chat.util.Time;
 import codeu.chat.util.Uuid;
 
 public final class RawControllerTest {
diff --git a/test/codeu/chat/util/TimeTest.java b/test/codeu/chat/util/TimeTest.java
@@ -0,0 +1,28 @@
+// Copyright 2017 Google Inc.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//    http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package codeu.chat.util;
+
+import static org.junit.Assert.*;
+import org.junit.Test;
+import org.junit.Before;
+
+public final class TimeTest {
+
+  @Test
+  public void testFromMs() {
+    assertEquals(0, Time.fromMs(0).inMs());
+    assertEquals(10, Time.fromMs(10).inMs());
+  }
+}
