You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jdo-commits@db.apache.org by mb...@apache.org on 2005/12/21 22:52:52 UTC

svn commit: r358395 - /db/jdo/trunk/tck20/test/java/org/apache/jdo/tck/query/result/Having.java

Author: mbo
Date: Wed Dec 21 13:52:40 2005
New Revision: 358395

URL: http://svn.apache.org/viewcvs?rev=358395&view=rev
Log:
JDO-244: Changed the HAVING argument in the positive test to do a COUNT on a 
single valued field instead of a collection field. Also changed the HAVING 
argument in the negative test to be invalid. 

Modified:
    db/jdo/trunk/tck20/test/java/org/apache/jdo/tck/query/result/Having.java

Modified: db/jdo/trunk/tck20/test/java/org/apache/jdo/tck/query/result/Having.java
URL: http://svn.apache.org/viewcvs/db/jdo/trunk/tck20/test/java/org/apache/jdo/tck/query/result/Having.java?rev=358395&r1=358394&r2=358395&view=diff
==============================================================================
--- db/jdo/trunk/tck20/test/java/org/apache/jdo/tck/query/result/Having.java (original)
+++ db/jdo/trunk/tck20/test/java/org/apache/jdo/tck/query/result/Having.java Wed Dec 21 13:52:40 2005
@@ -57,7 +57,7 @@
         /*VARIABLES*/   null,
         /*PARAMETERS*/  null,
         /*IMPORTS*/     null,
-        /*GROUP BY*/    "department HAVING COUNT(department.employees) > 0",
+        /*GROUP BY*/    "department HAVING COUNT(department) > 0",
         /*ORDER BY*/    null,
         /*FROM*/        null,
         /*TO*/          null)
@@ -78,7 +78,7 @@
         /*VARIABLES*/   null,
         /*PARAMETERS*/  null,
         /*IMPORTS*/     null,
-        /*GROUP BY*/    "department HAVING firstname == 'emp1First'",
+        /*GROUP BY*/    "department HAVING firstname",
         /*ORDER BY*/    null,
         /*FROM*/        null,
         /*TO*/          null)
@@ -88,11 +88,13 @@
      * The expected results of valid queries.
      */
     private Object[] expectedResult = {
-            Arrays.asList(new Object[] {
+        Arrays.asList(new Object[] {
+            new Object[] {
                     getTransientCompanyModelInstance("dept1"),
-                    new Double(45000.0),
+                    new Double(45000.0)},
+            new Object[] {
                     getTransientCompanyModelInstance("dept2"),
-                    new Double(58000.0)})
+                    new Double(58000.0)}})
     };
         
     /**