Skip to content

Commit bc5a7ab

Browse files
committed
test(user_ldap): move chunking comment above loop
The comment explaining why 66640 entries force multiple chunked queries belongs above the loop, not inside it. Note: the implementation chunks at a 65000 total-parameter limit (not Postgres's 65535 IN-list limit), so the comment retains 65000. Signed-off-by: Anna Larch <anna@nextcloud.com> AI-Assisted-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 40456ae commit bc5a7ab

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

apps/user_ldap/tests/Mapping/AbstractMappingTestCase.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -283,8 +283,8 @@ public function testGetListOfIdsByDn(): void {
283283
[$mapper,] = $this->initTest();
284284

285285
$listOfDNs = [];
286+
// List size exceeds the implementation's 65000-parameter chunk limit, forcing multiple chunked queries
286287
for ($i = 0; $i < 66640; $i++) {
287-
// List size exceeds the implementation's 65000-parameter chunk limit, forcing multiple chunked queries
288288
$name = 'as_' . $i;
289289
$dn = 'uid=' . $name . ',dc=example,dc=org';
290290
$listOfDNs[] = $dn;

0 commit comments

Comments
 (0)