diff options
author | Côme Chilliet <come.chilliet@nextcloud.com> | 2022-06-13 16:19:25 +0300 |
---|---|---|
committer | Côme Chilliet <come.chilliet@nextcloud.com> | 2022-06-13 16:19:25 +0300 |
commit | 129584c96613902fd0e4cc934b40d0c24c77e4c3 (patch) | |
tree | b85eed3921947959ef517eda7f0e1c497247b964 | |
parent | 4f230877e92571a4ba5708ad6ad0e2bf0d620e74 (diff) |
Remove deprecated at() matcher from user_ldap tests
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
-rw-r--r-- | apps/user_ldap/tests/LDAPProviderTest.php | 28 | ||||
-rw-r--r-- | apps/user_ldap/tests/Migration/UUIDFixInsertTest.php | 19 | ||||
-rw-r--r-- | apps/user_ldap/tests/User/UserTest.php | 14 |
3 files changed, 27 insertions, 34 deletions
diff --git a/apps/user_ldap/tests/LDAPProviderTest.php b/apps/user_ldap/tests/LDAPProviderTest.php index 59f51f204bf..f937ecd66c2 100644 --- a/apps/user_ldap/tests/LDAPProviderTest.php +++ b/apps/user_ldap/tests/LDAPProviderTest.php @@ -58,7 +58,7 @@ class LDAPProviderTest extends \Test\TestCase { ->setMethods(['getUserManager', 'getBackends', 'getGroupManager']) ->setConstructorArgs(['', new \OC\Config(\OC::$configDir)]) ->getMock(); - $server->expects($this->at(1)) + $server->expects($this->never()) ->method('getBackends') ->willReturn([$userBackend]); $server->expects($this->any()) @@ -136,10 +136,10 @@ class LDAPProviderTest extends \Test\TestCase { ->setMethods(['userExists', 'getLDAPAccess', 'username2dn']) ->disableOriginalConstructor() ->getMock(); - $userBackend->expects($this->at(0)) + $userBackend->expects($this->once()) ->method('userExists') ->willReturn(true); - $userBackend->expects($this->at(2)) + $userBackend->expects($this->once()) ->method('username2dn') ->willReturn('cn=existing_user,ou=Are Sufficient To,ou=Test,dc=example,dc=org'); $userBackend->expects($this->any()) @@ -186,10 +186,10 @@ class LDAPProviderTest extends \Test\TestCase { ->disableOriginalConstructor() ->getMock(); - $groupBackend->expects($this->at(0)) + $groupBackend->expects($this->once()) ->method('groupExists') ->willReturn(true); - $groupBackend->expects($this->at(2)) + $groupBackend->expects($this->once()) ->method('groupname2dn') ->willReturn('cn=existing_group,ou=Are Sufficient To,ou=Test,dc=example,dc=org'); $groupBackend->expects($this->any()) @@ -473,10 +473,10 @@ class LDAPProviderTest extends \Test\TestCase { ->setMethods(['userExists', 'getLDAPAccess', 'getConnection', 'clearCache']) ->disableOriginalConstructor() ->getMock(); - $userBackend->expects($this->at(0)) + $userBackend->expects($this->once()) ->method('userExists') ->willReturn(true); - $userBackend->expects($this->at(3)) + $userBackend->expects($this->once()) ->method('clearCache') ->willReturn(true); $userBackend->expects($this->any()) @@ -518,10 +518,10 @@ class LDAPProviderTest extends \Test\TestCase { ->setMethods(['groupExists', 'getLDAPAccess', 'getConnection', 'clearCache']) ->disableOriginalConstructor() ->getMock(); - $groupBackend->expects($this->at(0)) + $groupBackend->expects($this->once()) ->method('groupExists') ->willReturn(true); - $groupBackend->expects($this->at(3)) + $groupBackend->expects($this->once()) ->method('clearCache') ->willReturn(true); $groupBackend->expects($this->any()) @@ -598,10 +598,10 @@ class LDAPProviderTest extends \Test\TestCase { ->setMethods(['userExists', 'getLDAPAccess', 'getConnection', 'getConfiguration']) ->disableOriginalConstructor() ->getMock(); - $userBackend->expects($this->at(0)) + $userBackend->expects($this->once()) ->method('userExists') ->willReturn(true); - $userBackend->expects($this->at(3)) + $userBackend->expects($this->once()) ->method('getConfiguration') ->willReturn(['ldap_display_name' => 'displayName']); $userBackend->expects($this->any()) @@ -636,10 +636,10 @@ class LDAPProviderTest extends \Test\TestCase { ->setMethods(['userExists', 'getLDAPAccess', 'getConnection', 'getConfiguration']) ->disableOriginalConstructor() ->getMock(); - $userBackend->expects($this->at(0)) + $userBackend->expects($this->once()) ->method('userExists') ->willReturn(true); - $userBackend->expects($this->at(3)) + $userBackend->expects($this->once()) ->method('getConfiguration') ->willReturn(['ldap_email_attr' => 'mail']); $userBackend->expects($this->any()) @@ -684,7 +684,7 @@ class LDAPProviderTest extends \Test\TestCase { ->disableOriginalConstructor() ->getMock(); - $groupBackend->expects($this->at(0)) + $groupBackend->expects($this->once()) ->method('groupExists') ->willReturn(true); $groupBackend->expects($this->any()) diff --git a/apps/user_ldap/tests/Migration/UUIDFixInsertTest.php b/apps/user_ldap/tests/Migration/UUIDFixInsertTest.php index b13efa14e5c..49431ec0d9a 100644 --- a/apps/user_ldap/tests/Migration/UUIDFixInsertTest.php +++ b/apps/user_ldap/tests/Migration/UUIDFixInsertTest.php @@ -159,18 +159,15 @@ class UUIDFixInsertTest extends TestCase { ->with(0, 50) ->willReturn($groupBatches[0]); - $this->jobList->expects($this->at(0)) + $this->jobList->expects($this->exactly(5)) ->method('add') - ->willThrowException(new \InvalidArgumentException('Background job arguments can\'t exceed 4000 etc')); - $this->jobList->expects($this->at(1)) - ->method('add') - ->willThrowException(new \InvalidArgumentException('Background job arguments can\'t exceed 4000 etc')); - $this->jobList->expects($this->at(2)) - ->method('add'); - $this->jobList->expects($this->at(3)) - ->method('add'); - $this->jobList->expects($this->at(4)) - ->method('add'); + ->willReturnOnConsecutiveCalls( + $this->throwException(new \InvalidArgumentException('Background job arguments can\'t exceed 4000 etc')), + $this->throwException(new \InvalidArgumentException('Background job arguments can\'t exceed 4000 etc')), + null, + null, + null, + ); /** @var IOutput $out */ $out = $this->createMock(IOutput::class); diff --git a/apps/user_ldap/tests/User/UserTest.php b/apps/user_ldap/tests/User/UserTest.php index e86eb5e9e2e..d2113de842e 100644 --- a/apps/user_ldap/tests/User/UserTest.php +++ b/apps/user_ldap/tests/User/UserTest.php @@ -254,16 +254,12 @@ class UserTest extends \Test\TestCase { } public function testUpdateQuotaDefaultProvided() { - $this->connection->expects($this->at(0)) - ->method('__get') - ->with($this->equalTo('ldapQuotaAttribute')) - ->willReturn('myquota'); - $this->connection->expects($this->at(1)) - ->method('__get') - ->with($this->equalTo('ldapQuotaDefault')) - ->willReturn('25 GB'); $this->connection->expects($this->exactly(2)) - ->method('__get'); + ->method('__get') + ->willReturnMap([ + ['ldapQuotaAttribute', 'myquota'], + ['ldapQuotaDefault', '25 GB'], + ]); $this->access->expects($this->once()) ->method('readAttribute') |