Merge pull request #118 from phpcr/sort_test

provide a separate test for ordering with and without missing properties

Author: lsmith77

fixtures/general/base.xml

@@ -100,6 +100,10 @@
                 <sv:value>2009-04-27T13:01:07.472+02:00</sv:value>
             </sv:property>
 
+            <sv:property sv:name="foo" sv:type="String">
+                <sv:value>bar2</sv:value>
+            </sv:property>
+
             <sv:property sv:name="jcr:mimeType" sv:type="String">
                 <sv:value>text/plain</sv:value>
             </sv:property>

tests/06_Query/QuerySql2OperationsTest.php

@@ -247,6 +247,30 @@ public function testQueryJoinReference()
     }
 
     public function testQueryOrder()
+    {
+        /** @var $query QueryInterface */
+        $query = $this->sharedFixture['qm']->createQuery('
+            SELECT data.foo
+            FROM [nt:unstructured] AS data
+            WHERE ISDESCENDANTNODE([/tests_general_base]) AND data.foo IS NOT NULL
+            ORDER BY data.foo
+            ',
+            QueryInterface::JCR_SQL2
+        );
+
+        $this->assertInstanceOf('\PHPCR\Query\QueryInterface', $query);
+        $result = $query->execute();
+        $this->assertInstanceOf('\PHPCR\Query\QueryResultInterface', $result);
+        $vals = array();
+        foreach ($result->getRows() as $row) {
+            $vals[] = $row->getValue('data.foo');
+        }
+
+        // rows that do not have that field are empty string. empty is before fields with values
+        $this->assertEquals(array('bar', 'bar2'), $vals);
+    }
+
+    public function testQueryOrderWithMissingProperty()
     {
         /** @var $query QueryInterface */
         $query = $this->sharedFixture['qm']->createQuery('
@@ -265,8 +289,8 @@ public function testQueryOrder()
         foreach ($result->getRows() as $row) {
             $vals[] = $row->getValue('data.zeronumber');
         }
-        // rows that do not have that field are null. empty is before fields with values
-        $this->assertEquals(array(null, null, null, null, null, null, null, null, 0), $vals);
+        // rows that do not have that field are empty string. empty is before fields with values
+        $this->assertEquals(array('', '', '', '', '', '', '', '', 0), $vals);
     }
 
     public function testQueryMultiValuedProperty()