Reworking Test

vaage · vaage · commit a42c9ed252af · 2017-04-02T12:08:46.000-07:00
Moved Test from common to Util.

Changed the implementation to use Date all the time and not just for
converting it to a string.

Added the first test for Time to ensure that "fromMs" and "toMs"
yielded the same values.
diff --git a/src/codeu/chat/client/View.java b/src/codeu/chat/client/View.java
@@ -23,12 +23,12 @@
 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.common.Uuid;
 import codeu.chat.common.Uuids;
 import codeu.chat.util.Logger;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 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
@@ -24,6 +24,7 @@
 import codeu.chat.common.Uuids;
 import codeu.chat.util.Serializer;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 
 public final class Conversation {
 
diff --git a/src/codeu/chat/common/ConversationSummary.java b/src/codeu/chat/common/ConversationSummary.java
@@ -22,6 +22,7 @@
 import codeu.chat.common.Uuids;
 import codeu.chat.util.Serializer;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 
 public final class ConversationSummary implements ListViewable {
 
diff --git a/src/codeu/chat/common/LogicalView.java b/src/codeu/chat/common/LogicalView.java
@@ -16,7 +16,7 @@
 
 import java.util.Collection;
 
-import codeu.chat.common.Uuid;
+import codeu.chat.util.Time;
 
 // LOGICAL VIEW
 //
diff --git a/src/codeu/chat/common/Message.java b/src/codeu/chat/common/Message.java
@@ -18,10 +18,11 @@
 import java.io.InputStream;
 import java.io.OutputStream;
 
-import codeu.chat.util.Serializer;
-import codeu.chat.util.Serializers;
 import codeu.chat.common.Uuid;
 import codeu.chat.common.Uuids;
+import codeu.chat.util.Serializer;
+import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 
 public final class Message {
 
diff --git a/src/codeu/chat/common/RawController.java b/src/codeu/chat/common/RawController.java
@@ -14,6 +14,8 @@
 
 package codeu.chat.common;
 
+import codeu.chat.util.Time;
+
 // RAW CONTROLLER
 //
 // A controller that grants a large amount of control over how data is inserted
diff --git a/src/codeu/chat/common/Relay.java b/src/codeu/chat/common/Relay.java
@@ -16,8 +16,7 @@
 
 import java.util.Collection;
 
-import codeu.chat.common.Time;
-import codeu.chat.common.Uuid;
+import codeu.chat.util.Time;
 
 // RELAY
 //
diff --git a/src/codeu/chat/common/User.java b/src/codeu/chat/common/User.java
@@ -18,10 +18,11 @@
 import java.io.InputStream;
 import java.io.OutputStream;
 
-import codeu.chat.util.Serializer;
-import codeu.chat.util.Serializers;
 import codeu.chat.common.Uuid;
 import codeu.chat.common.Uuids;
+import codeu.chat.util.Serializer;
+import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 
 public final class User {
 
diff --git a/src/codeu/chat/relay/Server.java b/src/codeu/chat/relay/Server.java
@@ -24,10 +24,10 @@
 
 import codeu.chat.common.LinearUuidGenerator;
 import codeu.chat.common.Relay;
-import codeu.chat.common.Time;
 import codeu.chat.common.Uuid;
 import codeu.chat.common.Uuids;
 import codeu.chat.util.Logger;
+import codeu.chat.util.Time;
 
 public final class Server implements Relay {
 
diff --git a/src/codeu/chat/relay/ServerFrontEnd.java b/src/codeu/chat/relay/ServerFrontEnd.java
@@ -21,12 +21,12 @@
 
 import codeu.chat.common.NetworkCode;
 import codeu.chat.common.Relay;
-import codeu.chat.common.Time;
 import codeu.chat.common.Uuid;
 import codeu.chat.common.Uuids;
 import codeu.chat.util.Logger;
 import codeu.chat.util.Serializer;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 import codeu.chat.util.connections.Connection;
 
 public final class ServerFrontEnd {
diff --git a/src/codeu/chat/server/Controller.java b/src/codeu/chat/server/Controller.java
@@ -20,11 +20,11 @@
 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.common.Uuid;
 import codeu.chat.common.Uuids;
 import codeu.chat.util.Logger;
+import codeu.chat.util.Time;
 
 public final class Controller implements RawController, BasicController {
 
diff --git a/src/codeu/chat/server/Model.java b/src/codeu/chat/server/Model.java
@@ -20,9 +20,9 @@
 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.common.Uuid;
+import codeu.chat.util.Time;
 import codeu.chat.util.store.Store;
 import codeu.chat.util.store.StoreAccessor;
 
diff --git a/src/codeu/chat/server/NoOpRelay.java b/src/codeu/chat/server/NoOpRelay.java
@@ -18,8 +18,8 @@
 import java.util.Collection;
 
 import codeu.chat.common.Relay;
-import codeu.chat.common.Time;
 import codeu.chat.common.Uuid;
+import codeu.chat.util.Time;
 
 
 // NO OP RELAY
diff --git a/src/codeu/chat/server/RemoteRelay.java b/src/codeu/chat/server/RemoteRelay.java
@@ -22,12 +22,12 @@
 
 import codeu.chat.common.NetworkCode;
 import codeu.chat.common.Relay;
-import codeu.chat.common.Time;
 import codeu.chat.common.Uuid;
 import codeu.chat.common.Uuids;
 import codeu.chat.util.Logger;
 import codeu.chat.util.Serializer;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 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,12 +29,12 @@
 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.common.Uuid;
 import codeu.chat.common.Uuids;
 import codeu.chat.util.Logger;
 import codeu.chat.util.Serializers;
+import codeu.chat.util.Time;
 import codeu.chat.util.Timeline;
 import codeu.chat.util.connections.Connection;
 
diff --git a/src/codeu/chat/server/View.java b/src/codeu/chat/server/View.java
@@ -33,10 +33,10 @@
 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.common.Uuid;
 import codeu.chat.util.Logger;
+import codeu.chat.util.Time;
 import codeu.chat.util.store.StoreAccessor;
 
 public final class View implements BasicView, LogicalView, SinglesView {
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
@@ -28,6 +28,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.store.StoreTest.class
          );
       for (final Failure failure : result.getFailures()) {
diff --git a/test/codeu/chat/relay/ServerTest.java b/test/codeu/chat/relay/ServerTest.java
@@ -21,9 +21,9 @@
 import org.junit.Before;
 
 import codeu.chat.common.Relay;
-import codeu.chat.common.Time;
 import codeu.chat.common.Uuid;
 import codeu.chat.common.Uuids;
+import codeu.chat.util.Time;
 
 public final class ServerTest {
 
diff --git a/test/codeu/chat/server/RawControllerTest.java b/test/codeu/chat/server/RawControllerTest.java
@@ -21,10 +21,10 @@
 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.common.Uuid;
 import codeu.chat.common.Uuids;
+import codeu.chat.util.Time;
 
 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());
+  }
+}
