MNT Tweak some test to account for slightly different sorting logic in PostgreSQL

tests/php/ORM/DataListTest.php

@@ -1958,33 +1958,28 @@ public function provideSortDirectionValidationTwoArgs(): array
     /**
      * Test passing scalar values to sort()
      *
-     * Explicity tests that sort(null) will wipe any existing sort on a DataList
-     *
      * @dataProvider provideSortScalarValues
      */
     public function testSortScalarValues(mixed $emtpyValue, string $type): void
     {
         $this->assertSame(['Subteam 1'], Team::get()->limit(1)->column('Title'));
         $list = Team::get()->sort('Title DESC');
         $this->assertSame(['Team 3'], $list->limit(1)->column('Title'));
-        if ($type !== 'wipes-existing') {
-            $this->expectException(InvalidArgumentException::class);
-        }
+        $this->expectException(InvalidArgumentException::class);
         if ($type === 'invalid-scalar') {
             $this->expectExceptionMessage('sort() arguments must either be a string, an array, or null');
         }
         if ($type === 'empty-scalar') {
             $this->expectExceptionMessage('Invalid sort parameter');
         }
-        // $type === 'wipes-existing' is valid
+
         $list = $list->sort($emtpyValue);
         $this->assertSame(['Subteam 1'], $list->limit(1)->column('Title'));
     }
 
     public function provideSortScalarValues(): array
     {
         return [
-            [null, 'wipes-existing'],
             ['', 'empty-scalar'],
             [[], 'empty-scalar'],
             [false, 'invalid-scalar'],
@@ -1994,6 +1989,27 @@ public function provideSortScalarValues(): array
         ];
     }
 
+    /**
+     * Explicity tests that sort(null) will wipe any existing sort on a DataList
+     */
+    public function testSortNull(): void
+    {
+        $list = Team::get()->sort('Title DESC');
+        $query = $list->dataQuery()->getFinalisedQuery();
+        $this->assertSame(
+            ['"DataObjectTest_Team"."Title"' => 'DESC'],
+            $query->getOrderBy(),
+            'Calling sort on a DataList sets an Orderby on the underlying query.'
+        );
+
+        $list = $list->sort(null);
+        $query = $list->dataQuery()->getFinalisedQuery();
+        $this->assertEmpty(
+            $query->getOrderBy(),
+            'Calling sort with null on a DataList unsets the orderby on the underlying query.'
+        );
+    }
+
     public function testShuffle()
     {
         $list = Team::get()->shuffle();

tests/php/Security/MemberTest.php

@@ -1831,7 +1831,7 @@ public function testMapInCMSGroups(array $groupFixtures, array $groupCodes, arra
         $result = Member::mapInCMSGroups($groups);
         $this->assertInstanceOf(Map::class, $result);
 
-        $this->assertSame($expectedUsers, $result->keys());
+        $this->assertEqualsCanonicalizing($expectedUsers, $result->keys());
     }
 
     public function provideMapInCMSGroups()