[google-web-toolkit commit] r5638 - Fixed a broken test that would fail 0.1% of the time because it assumes that a Date has a...

0 views
Skip to first unread message

codesite...@google.com

unread,
Jun 29, 2009, 4:50:16 PM6/29/09
to gwt...@gmail.com
Author: jlab...@google.com
Date: Mon Jun 29 13:22:41 2009
New Revision: 5638

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

Log:
Fixed a broken test that would fail 0.1% of the time because it assumes
that a Date has a non-zero millsecond field.

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 Jun 29 13:22:41 2009
@@ -22,7 +22,7 @@
import java.util.Date;

/**
- * Tests {@link java.sql.Timestame}. We assume that the underlying
+ * Tests {@link java.sql.Timestamp}. We assume that the underlying
* {@link java.util.Date} implementation is correct and concentrate only
on the
* differences between the two.
*/
@@ -32,6 +32,7 @@
/**
* Sets module name so that javascript compiler can operate.
*/
+ @Override
public String getModuleName() {
return "com.google.gwt.emultest.EmulSuite";
}
@@ -45,7 +46,11 @@
Date d = new Date(now);

Timestamp t = new Timestamp(d.getTime());
- t.setNanos(1);
+ if (now % 1000 == 0) {
+ t.setNanos(1000001);
+ } else {
+ t.setNanos(1);
+ }

// Timestamps are stored at second-level precision
Date d2 = new Date(t.getTime());

Reply all
Reply to author
Forward
0 new messages