Author: [email protected]
Date: Mon Jul  6 13:52:38 2009
New Revision: 5673

Modified:
    trunk/user/test/com/google/gwt/emultest/java/sql/SqlTimestampTest.java

Log:
Fixed a bug in SqlTimestampTest#testDateComparison() where it would fail if  
the milliseconds portion of the current Date was 000.  We switched the 000  
to 001 to fix a previous bug, but it introduced a new bug later in the  
code.  Ultimately, this test should use a series of defined timestamps and  
test for specific scenarios instead of using the current time.

Patch by: jlabanca
Review by: jat(desk)



Modified:  
trunk/user/test/com/google/gwt/emultest/java/sql/SqlTimestampTest.java
==============================================================================
--- trunk/user/test/com/google/gwt/emultest/java/sql/SqlTimestampTest.java      
 
(original)
+++ trunk/user/test/com/google/gwt/emultest/java/sql/SqlTimestampTest.java      
 
Mon Jul  6 13:52:38 2009
@@ -71,7 +71,7 @@
      }

      Timestamp t2 = new Timestamp(d.getTime());
-    t2.setNanos(2);
+    t2.setNanos(t.getNanos() + 1);

      assertFalse("t.equals(t2)", t.equals(t2));
      assertEquals("hashcode2", t.hashCode(), t2.hashCode());

--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---

Reply via email to