Author: hlship
Date: Sun May 13 12:26:23 2007
New Revision: 537644

URL: http://svn.apache.org/viewvc?view=rev&rev=537644
Log:
Fix a broken test whose behavior varied arbitrary (due to comparison of null to 
null).

Modified:
    
tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry/internal/grid/ListGridDataSourceTest.java

Modified: 
tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry/internal/grid/ListGridDataSourceTest.java
URL: 
http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry/internal/grid/ListGridDataSourceTest.java?view=diff&rev=537644&r1=537643&r2=537644
==============================================================================
--- 
tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry/internal/grid/ListGridDataSourceTest.java
 (original)
+++ 
tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry/internal/grid/ListGridDataSourceTest.java
 Sun May 13 12:26:23 2007
@@ -44,7 +44,7 @@
     private final List _raw = Arrays.asList(
             new Datum(FRED, "Fred"),
             new Datum(BARNEY, "Barney"),
-            new Datum(WILMA, null),
+            new Datum(WILMA, "Wilma"),
             new Datum(BETTY, null));
 
     private final ListGridDataSource _source = new ListGridDataSource(_raw);
@@ -95,13 +95,13 @@
         // Without a secondary sort column, it's kind of arbitrary whether 
WILMA or BETTY is sorted
         // first.
 
-        sort("value", true, WILMA, BETTY, BARNEY, FRED);
+        sort("value", true, BETTY, BARNEY, FRED, WILMA);
     }
 
     @Test
     public void sort_on_string_value_descending()
     {
-        sort("value", false, FRED, BARNEY, WILMA, BETTY);
+        sort("value", false, WILMA, FRED, BARNEY, BETTY);
     }
 
     private void sort(String propertyName, boolean ascending, int... ids)


Reply via email to