[NFC] Add in unit test to ensure that APIv4 Doesn't accept an invalid…

… sort

tests/phpunit/api/v4/Query/Api4SelectQueryTest.php

@@ -105,4 +105,26 @@ public function testOneToManyMultipleJoin() {
     $this->assertCount(2, $result['phones']);
   }
 
+
+  public function testInvaidSort() {
+    $query = new Api4SelectQuery('Contact', FALSE, civicrm_api4('Contact', 'getFields', ['includeCustom' => FALSE, 'checkPermissions' => FALSE, 'action' => 'get'], 'name'));
+    $query->select[] = 'id';
+    $query->select[] = 'first_name';
+    $query->select[] = 'phones.phone';
+    $query->where[] = ['first_name', '=', 'Phoney'];
+    $query->orderBy = ['first_name' => 'sleep(1)'];
+    try {
+      $results = $query->run();
+      $this->fail('An Exception Should have been raised');
+    }
+    catch (\API_Exception $e) {
+    }
+    $query->orderBy = ['sleep(1)', 'ASC'];
+    try {
+      $results = $query->run();
+      $this->fail('An Exception Should have been raised');
+    }
+    catch (\API_Exception $e) {
+    }
+  }
 }