]> source.dussan.org Git - nextcloud-server.git/commitdiff
php-cs happyness 24716/head
authorArthur Schiwon <blizzz@arthur-schiwon.de>
Fri, 27 Nov 2020 18:34:35 +0000 (19:34 +0100)
committerbackportbot[bot] <backportbot[bot]@users.noreply.github.com>
Tue, 15 Dec 2020 21:37:58 +0000 (21:37 +0000)
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
apps/user_ldap/tests/Group_LDAPTest.php

index 63de2ee9481270050505bb651100a6edf6de33ed..bc582ab90b51b29d0c036bec09dffd1ff90924d0 100644 (file)
@@ -50,6 +50,47 @@ use Test\TestCase;
  * @package OCA\User_LDAP\Tests
  */
 class Group_LDAPTest extends TestCase {
+       public function testCountEmptySearchString() {
+               $access = $this->getAccessMock();
+               $pluginManager = $this->getPluginManagerMock();
+               $groupDN = 'cn=group,dc=foo,dc=bar';
+
+               $this->enableGroups($access);
+
+               $access->expects($this->any())
+                       ->method('groupname2dn')
+                       ->willReturn($groupDN);
+               $access->expects($this->any())
+                       ->method('readAttribute')
+                       ->willReturnCallback(function ($dn) use ($groupDN) {
+                               if ($dn === $groupDN) {
+                                       return [
+                                               'uid=u11,ou=users,dc=foo,dc=bar',
+                                               'uid=u22,ou=users,dc=foo,dc=bar',
+                                               'uid=u33,ou=users,dc=foo,dc=bar',
+                                               'uid=u34,ou=users,dc=foo,dc=bar'
+                                       ];
+                               }
+                               return [];
+                       });
+               $access->expects($this->any())
+                       ->method('isDNPartOfBase')
+                       ->willReturn(true);
+               // for primary groups
+               $access->expects($this->once())
+                       ->method('countUsers')
+                       ->willReturn(2);
+
+               $access->userManager->expects($this->any())
+                       ->method('getAttributes')
+                       ->willReturn(['displayName', 'mail']);
+
+               $groupBackend = new GroupLDAP($access, $pluginManager);
+               $users = $groupBackend->countUsersInGroup('group');
+
+               $this->assertSame(6, $users);
+       }
+
        /**
         * @return MockObject|Access
         */
@@ -98,47 +139,6 @@ class Group_LDAPTest extends TestCase {
                        });
        }
 
-       public function testCountEmptySearchString() {
-               $access = $this->getAccessMock();
-               $pluginManager = $this->getPluginManagerMock();
-               $groupDN = 'cn=group,dc=foo,dc=bar';
-
-               $this->enableGroups($access);
-
-               $access->expects($this->any())
-                       ->method('groupname2dn')
-                       ->willReturn($groupDN);
-               $access->expects($this->any())
-                       ->method('readAttribute')
-                       ->willReturnCallback(function ($dn) use ($groupDN) {
-                               if ($dn === $groupDN) {
-                                       return [
-                                               'uid=u11,ou=users,dc=foo,dc=bar',
-                                               'uid=u22,ou=users,dc=foo,dc=bar',
-                                               'uid=u33,ou=users,dc=foo,dc=bar',
-                                               'uid=u34,ou=users,dc=foo,dc=bar'
-                                       ];
-                               }
-                               return [];
-                       });
-               $access->expects($this->any())
-                       ->method('isDNPartOfBase')
-                       ->willReturn(true);
-               // for primary groups
-               $access->expects($this->once())
-                       ->method('countUsers')
-                       ->willReturn(2);
-
-               $access->userManager->expects($this->any())
-                       ->method('getAttributes')
-                       ->willReturn(['displayName', 'mail']);
-
-               $groupBackend = new GroupLDAP($access, $pluginManager);
-               $users = $groupBackend->countUsersInGroup('group');
-
-               $this->assertSame(6, $users);
-       }
-
        public function testCountWithSearchString() {
                $access = $this->getAccessMock();
                $pluginManager = $this->getPluginManagerMock();
@@ -503,7 +503,7 @@ class Group_LDAPTest extends TestCase {
                $groupBackend->inGroup($uid, $gid);
        }
 
-       public function  groupWithMembersProvider() {
+       public function groupWithMembersProvider() {
                return [
                        [
                                'someGroup',
@@ -539,7 +539,7 @@ class Group_LDAPTest extends TestCase {
                                        case 'ldapDynamicGroupMemberURL':
                                                return '';
                                        case 'hasPrimaryGroups':
-                                       case 'ldapNestedGroups';
+                                       case 'ldapNestedGroups':
                                                return 0;
                                        default:
                                                return 1;
@@ -585,7 +585,7 @@ class Group_LDAPTest extends TestCase {
                                        case 'ldapDynamicGroupMemberURL':
                                                return '';
                                        case 'hasPrimaryGroups':
-                                       case 'ldapNestedGroups';
+                                       case 'ldapNestedGroups':
                                                return 0;
                                        default:
                                                return 1;
@@ -641,7 +641,7 @@ class Group_LDAPTest extends TestCase {
                                        case 'ldapLoginFilter':
                                                return 'uid=%uid';
                                        case 'hasPrimaryGroups':
-                                       case 'ldapNestedGroups';
+                                       case 'ldapNestedGroups':
                                                return 0;
                                        default:
                                                return 1;
@@ -894,8 +894,8 @@ class Group_LDAPTest extends TestCase {
 
        public function nestedGroupsProvider(): array {
                return [
-                       [ true ],
-                       [ false ],
+                       [true],
+                       [false],
                ];
        }