Skip to content

Commit 9f766b5

Browse files
committed
[SCA] Minor fixes on tests
1 parent 53f1cfc commit 9f766b5

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

Tests/Adapter/ExtLdap/AdapterTest.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -87,12 +87,12 @@ public function testLdapQueryScopeBase()
8787
$ldap->getConnection()->bind('cn=admin,dc=symfony,dc=com', 'symfony');
8888

8989
$query = $ldap->createQuery('cn=Fabien Potencier,dc=symfony,dc=com', '(objectclass=*)', [
90-
'scope' => Query::SCOPE_BASE,
90+
'scope' => Query::SCOPE_BASE,
9191
]);
9292
$result = $query->execute();
9393

9494
$entry = $result[0];
95-
$this->assertEquals($result->count(), 1);
95+
$this->assertEquals(1, $result->count());
9696
$this->assertEquals(['Fabien Potencier'], $entry->getAttribute('cn'));
9797
}
9898

@@ -109,7 +109,7 @@ public function testLdapQueryScopeOneLevel()
109109
$subtree_count = $ldap->createQuery('ou=Components,dc=symfony,dc=com', '(objectclass=*)')->execute()->count();
110110

111111
$this->assertNotEquals($one_level_result->count(), $subtree_count);
112-
$this->assertEquals($one_level_result->count(), 1);
113-
$this->assertEquals($one_level_result[0]->getAttribute('ou'), ['Ldap']);
112+
$this->assertEquals(1, $one_level_result->count());
113+
$this->assertEquals(['Ldap'], $one_level_result[0]->getAttribute('ou'));
114114
}
115115
}

Tests/Adapter/ExtLdap/LdapManagerTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ public function testLdapRename()
163163

164164
$result = $this->executeSearchQuery(1);
165165
$renamedEntry = $result[0];
166-
$this->assertEquals($renamedEntry->getAttribute('cn')[0], 'Kevin');
166+
$this->assertEquals('Kevin', $renamedEntry->getAttribute('cn')[0]);
167167

168168
$oldRdn = $entry->getAttribute('cn')[0];
169169
$entryManager->rename($renamedEntry, 'cn='.$oldRdn);

0 commit comments

Comments
 (0)