test: resolve the rename-codemod report for tests

The CR dry run over tests/ and bin/ left these to hand-resolve. Query::TYPE_ELEM_MATCH is deleted (DEC-55), so the
fingerprint and shape tests build elemMatch through Query::elemMatch(). getRelationshipHook() is @internal in 8.0,
so the e2e setUp swaps the relationship hook the way it already swaps the permission hook; the unit tests of the hook
mechanics keep reading it. getConnectionId() returns null without Feature\Connection, so the e2e check asserts a
string before comparing it with ''. The skipDuplicates() statement test is named after ignoreDuplicates(), which it
drives.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Author: abnegate

tests/e2e/Adapter/Base.php

@@ -93,11 +93,9 @@ protected function setUp(): void
 
         self::$authorization->addRole('any');
 
-        $db = $this->getDatabase();
-        if ($db->getRelationshipHook() === null) {
-            $db->addHook(new Relationships());
-        }
-        $db->removeHook(Permissions::class)->addHook(new Permissions());
+        $this->getDatabase()
+            ->removeHook(Relationships::class)->addHook(new Relationships())
+            ->removeHook(Permissions::class)->addHook(new Permissions());
     }
 
     protected function tearDown(): void

tests/e2e/Adapter/Scopes/CollectionTests.php

@@ -444,7 +444,9 @@ public function testGetCollectionId(): void
             return;
         }
 
-        $this->assertNotSame('', $database->getConnectionId());
+        $id = $database->getConnectionId();
+        $this->assertIsString($id);
+        $this->assertNotSame('', $id);
     }
 
     public function testKeywords(): void

…/Adapter/SkipDuplicatesStatementTest.php …dapter/IgnoreDuplicatesStatementTest.phptests/unit/Adapter/SkipDuplicatesStatementTest.php renamed to tests/unit/Adapter/IgnoreDuplicatesStatementTest.php tests/unit/Adapter/SkipDuplicatesStatementTest.php renamed to tests/unit/Adapter/IgnoreDuplicatesStatementTest.php

@@ -15,9 +15,9 @@
 use Utopia\Database\Helpers\Role;
 
 /**
- * The statements skipDuplicates() sends on the engines the host cannot run.
+ * The statements ignoreDuplicates() sends on the engines the host cannot run.
  */
-final class SkipDuplicatesStatementTest extends TestCase
+final class IgnoreDuplicatesStatementTest extends TestCase
 {
     private const string STORED = 'stored';
 

tests/unit/QueryTest.php

@@ -573,12 +573,12 @@ public function testFingerprintNestedLogicalQueries(): void
         $this->assertNotSame(Query::fingerprint([$andOne]), Query::fingerprint([$andTwo]));
 
         // elemMatch attribute matters: same inner shape on different fields must NOT collide
-        $elemTags = new Query(Query::TYPE_ELEM_MATCH, 'tags', [Query::equal('name', ['php'])]);
-        $elemCategories = new Query(Query::TYPE_ELEM_MATCH, 'categories', [Query::equal('name', ['php'])]);
+        $elemTags = Query::elemMatch('tags', [Query::equal('name', ['php'])]);
+        $elemCategories = Query::elemMatch('categories', [Query::equal('name', ['php'])]);
         $this->assertNotSame(Query::fingerprint([$elemTags]), Query::fingerprint([$elemCategories]));
 
         // elemMatch values-only change (same field, same child shape) still collides — as expected
-        $elemTagsOther = new Query(Query::TYPE_ELEM_MATCH, 'tags', [Query::equal('name', ['js'])]);
+        $elemTagsOther = Query::elemMatch('tags', [Query::equal('name', ['js'])]);
         $this->assertSame(Query::fingerprint([$elemTags]), Query::fingerprint([$elemTagsOther]));
     }
 
@@ -599,7 +599,7 @@ public function testShape(): void
         $this->assertSame('and:(equal:name|greaterThan:age)', $and->shape());
 
         // elemMatch preserves the attribute (the field being matched)
-        $elem = new Query(Query::TYPE_ELEM_MATCH, 'tags', [Query::equal('name', ['php'])]);
+        $elem = Query::elemMatch('tags', [Query::equal('name', ['php'])]);
         $this->assertSame('elemMatch:tags(equal:name)', $elem->shape());
 
         // Deeply nested — iterative traversal must match recursive result