Merge pull request #313 from montymxb/test-fixes

Updates various tests.

Author: Arthur Cinader

src/Parse/ParseClient.php

@@ -111,7 +111,7 @@ final class ParseClient
      *
      * @var string
      */
-    const VERSION_STRING = 'php1.2.2';
+    const VERSION_STRING = 'php1.2.7';
 
     /**
      * Parse\Client::initialize, must be called before using Parse features.

tests/Parse/ParseClientTest.php

@@ -514,7 +514,8 @@ public function testBadApiResponse()
         $httpClient = ParseClient::getHttpClient();
 
         // create a mock of the current http client
-        $stubClient = $this->createMock(get_class($httpClient));
+        $stubClient = $this->getMockBuilder(get_class($httpClient))
+            ->getMock();
 
         // stub the response type to return
         // something we will try to work with

tests/Parse/ParseQueryTest.php

@@ -1499,20 +1499,23 @@ public function testOrderByCreatedAtDesc()
         }
     }
 
+    /**
+     * @group order-by-updated-at
+     */
     public function testOrderByUpdatedAtAsc()
     {
         $numbers = [3, 1, 2];
         $objects = [];
-        $this->saveObjects(
-            3,
-            function ($i) use ($numbers, &$objects) {
-                $obj = ParseObject::create('TestObject');
-                $obj->set('number', $numbers[$i]);
-                $objects[] = $obj;
 
-                return $obj;
-            }
-        );
+        foreach($numbers as $num) {
+            $obj = ParseObject::create('TestObject');
+            $obj->set('number', $num);
+            $obj->save();
+            $objects[]  = $obj;
+            sleep(1);
+
+        }
+
         $objects[1]->set('number', 4);
         $objects[1]->save();
         $query = new ParseQuery('TestObject');
@@ -1541,16 +1544,16 @@ public function testOrderByUpdatedAtDesc()
     {
         $numbers = [3, 1, 2];
         $objects = [];
-        $this->saveObjects(
-            3,
-            function ($i) use ($numbers, &$objects) {
-                $obj = ParseObject::create('TestObject');
-                $obj->set('number', $numbers[$i]);
-                $objects[] = $obj;
 
-                return $obj;
-            }
-        );
+        foreach($numbers as $num) {
+            $obj = ParseObject::create('TestObject');
+            $obj->set('number', $num);
+            $obj->save();
+            $objects[]  = $obj;
+            sleep(1);
+
+        }
+
         $objects[1]->set('number', 4);
         $objects[1]->save();
         $query = new ParseQuery('TestObject');
@@ -1561,7 +1564,8 @@ function ($i) use ($numbers, &$objects) {
             count($results),
             'Did not return correct number of objects.'
         );
-        $expected = [4, 3, 2];
+
+        $expected = [4, 2, 3];
         for ($i = 0; $i < 3; ++$i) {
             $this->assertEquals(
                 $expected[$i],