cross-solution / YAWIK

module/Organizations/test/OrganizationsTest/Controller/InviteEmployeeControllerTest.php

Indentation +14 added lines, -14 removed lines

@@ -59,31 +59,31 @@
 
 
         $plugins = $this
-	        ->getMockBuilder('\Zend\Mvc\Controller\PluginManager')
-	        ->disableOriginalConstructor()
-	        ->getMock()
+            ->getMockBuilder('\Zend\Mvc\Controller\PluginManager')
+            ->disableOriginalConstructor()
+            ->getMock()
         ;
         $plugins
-	        ->expects($this->any())
-	        ->method('get')
-	        ->will($this->returnCallback(array($this, 'retrievePluginMock')))
+            ->expects($this->any())
+            ->method('get')
+            ->will($this->returnCallback(array($this, 'retrievePluginMock')))
         ;
 
         $this->target->setPluginManager($plugins);
 
         $params = $this->getMockBuilder('\Zend\Mvc\Controller\Plugin\Params')
-                       ->disableOriginalConstructor()
-                       ->getMock()
+                        ->disableOriginalConstructor()
+                        ->getMock()
         ;
         $params
-	        ->expects($this->any())
-	        ->method('__invoke')
-	        ->will($this->returnSelf())
+            ->expects($this->any())
+            ->method('__invoke')
+            ->will($this->returnSelf())
         ;
         $params
-	        ->expects($this->any())
-	        ->method('fromQuery')
-	        ->will($this->returnCallback(array($this, 'retrieveQueryParam')))
+            ->expects($this->any())
+            ->method('fromQuery')
+            ->will($this->returnCallback(array($this, 'retrieveQueryParam')))
         ;
 
         $this->pluginsMockMap['params'] = $params;

Spacing +2 added lines, -2 removed lines

@@ -94,7 +94,7 @@ 
         return isset($this->pluginsMockMap[$name]) ? $this->pluginsMockMap[$name] : null;
     }
 
-    public function retrieveQueryParam($name, $default=null)
+    public function retrieveQueryParam($name, $default = null)
     {
         return isset($this->queryParams[$name]) ? $this->queryParams[$name] : $default;
     }
@@ -127,7 +127,7 @@ 
 
     private function setupOrganizationEntityRetrievalMocks($name)
     {
-        $id = $name. "Id";
+        $id = $name."Id";
         $organization = new Organization();
         $organization->setOrganizationName(new OrganizationName($name));
 

module/Organizations/test/OrganizationsTest/Repository/Filter/PaginationQueryFactoryTest.php

Spacing +5 added lines, -5 removed lines

@@ -35,16 +35,16 @@
         $container->expects($this->any())
             ->method('get')
             ->willReturnMap([
-                ['Core/RepositoryService',$container],
-                ['Jobs/Job',$jobRepository],
-                ['AuthenticationService',$auth]
+                ['Core/RepositoryService', $container],
+                ['Jobs/Job', $jobRepository],
+                ['AuthenticationService', $auth]
             ])
         ;
 
-        $factory = new PaginationQueryFactory($auth,$jobRepository);
+        $factory = new PaginationQueryFactory($auth, $jobRepository);
         $this->assertInstanceOf(
             PaginationQuery::class,
-            $factory($container,'some-name')
+            $factory($container, 'some-name')
         );
     }
 }

module/Organizations/test/OrganizationsTest/Repository/Filter/ListJobQueryTest.php

Spacing +3 added lines, -3 removed lines

@@ -43,7 +43,7 @@ 
 
         $builder->expects($this->once())
             ->method('in')
-            ->with([Status::ACTIVE,Status::PUBLISH])
+            ->with([Status::ACTIVE, Status::PUBLISH])
             ->willReturn($builder)
         ;
         $builder->expects($this->exactly(2))
@@ -56,7 +56,7 @@ 
         ;
 
 
-        $params=['organization_id' =>$organization->getId()];
-        $target->createQuery($params,$builder);
+        $params = ['organization_id' =>$organization->getId()];
+        $target->createQuery($params, $builder);
     }
 }

module/Organizations/test/OrganizationsTest/Repository/Filter/PaginationQueryTest.php

Spacing +12 added lines, -12 removed lines

@@ -56,7 +56,7 @@ 
 
         $this->jobRepository = $this->createMock(JobRepository::class);
         $this->authService = $this->createMock(AuthenticationService::class);
-        $this->target = new PaginationQuery($this->jobRepository,$this->authService);
+        $this->target = new PaginationQuery($this->jobRepository, $this->authService);
     }
 
     public function testCreateQuery()
@@ -77,7 +77,7 @@ 
         $builder->expects($this->any())
             ->method('field')
             ->willReturnMap([
-                ['permissions.view',$builder]
+                ['permissions.view', $builder]
             ])
         ;
 
@@ -95,7 +95,7 @@ 
         $params = [
             'q' => 'some text'
         ];
-        $target->createQuery($params,$builder);
+        $target->createQuery($params, $builder);
     }
 
     public function testCreateQueryForProfile()
@@ -120,20 +120,20 @@ 
         $builder->expects($this->any())
             ->method('field')
             ->willReturnMap([
-                ['profileSetting',$builder],
-                ['id',$builder],
-                ['organization',$builder],
-                ['status.name',$builder],
-                ['isDraft',$builder],
-                ['profileSetting',$builder]
+                ['profileSetting', $builder],
+                ['id', $builder],
+                ['organization', $builder],
+                ['status.name', $builder],
+                ['isDraft', $builder],
+                ['profileSetting', $builder]
             ])
         ;
 
         $builder->expects($this->exactly(2))
             ->method('notIn')
             ->willReturnMap([
-                [ [StatusInterface::EXPIRED, StatusInterface::INACTIVE], $builder],
-                [ ['some-id'],$builder ]
+                [[StatusInterface::EXPIRED, StatusInterface::INACTIVE], $builder],
+                [['some-id'], $builder]
             ])
         ;
 
@@ -175,6 +175,6 @@ 
         $params = new Parameters([
             'type' => 'profile'
         ]);
-        $target->createQuery($params,$builder);
+        $target->createQuery($params, $builder);
     }
 }

module/Organizations/test/OrganizationsTest/Repository/Event/InjectOrganizationReferenceListenerTest.php

Indentation +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
         $rep = $this->getMockBuilder('\Organizations\Repository\Organization')->disableOriginalConstructor()->getMock();
 
         $dm->expects($this->once())->method('getRepository')->with('Organizations\Entity\Organization')
-           ->willReturn($rep);
+            ->willReturn($rep);
 
         $doc->setId('test1234');
 

module/Organizations/test/OrganizationsTest/Mail/EmployeeInvitationFactoryTest.php

Indentation +18 added lines, -18 removed lines

@@ -116,9 +116,9 @@ 
 
         $orgRep = $this->getMockBuilder('\Organizations\Repository\Organization')->disableOriginalConstructor()->getMock();
         $orgRep->expects($this->exactly(2))
-               ->method('findByUser')
-               ->withConsecutive(array('testOwner'), array('testUser'))
-               ->will($this->onConsecutiveCalls($ownerOrg, $userOrg));
+                ->method('findByUser')
+                ->withConsecutive(array('testOwner'), array('testUser'))
+                ->will($this->onConsecutiveCalls($ownerOrg, $userOrg));
 
 
         $ownerOrgRef = new OrganizationReference('testOwner', $orgRep);
@@ -135,37 +135,37 @@ 
 
         $router = $this->getMockForAbstractClass(RouteStackInterface::class);
         $router->expects($this->once())
-               ->method('assemble')
-               ->with(array('action' => 'accept'),
-                      array('name' => 'lang/organizations/invite',
+                ->method('assemble')
+                ->with(array('action' => 'accept'),
+                        array('name' => 'lang/organizations/invite',
                             'query' => array('token' => $options['token'], 'organization' => $ownerOrg->getId()))
-               )
-               ->willReturn('testUrl');
+                )
+                ->willReturn('testUrl');
 
         $mailService = $this->getMockBuilder('\Core\Mail\MailService')->disableOriginalConstructor()->getMock();
 
         $services = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager')->disableOriginalConstructor()->getMock();
 
         $services->expects($this->exactly(3))
-                 ->method('get')
-                 ->withConsecutive(
+                    ->method('get')
+                    ->withConsecutive(
                         array('AuthenticationService'),
                         array('Router'),
                         ['Core/MailService']
-                 )->will($this->onConsecutiveCalls($authService, $router, $mailService));
+                    )->will($this->onConsecutiveCalls($authService, $router, $mailService));
 
         $mailMock = new HTMLTemplateMessage(new \Zend\ServiceManager\ServiceManager());
         $translator = $this->getMockBuilder('\Zend\I18n\Translator\Translator')->disableOriginalConstructor()->getMock();
         $translator
-	        ->expects($this->any())
-	        ->method('translate')
-	        ->will($this->returnArgument(0));
+            ->expects($this->any())
+            ->method('translate')
+            ->will($this->returnArgument(0));
         $mailMock->setTranslator($translator);
         $mailService
-	        ->expects($this->once())
-	        ->method('get')
-	        ->with('htmltemplate')
-	        ->willReturn($mailMock);
+            ->expects($this->once())
+            ->method('get')
+            ->with('htmltemplate')
+            ->willReturn($mailMock);
 
 
         $target = new EmployeeInvitationFactory();

Spacing +1 added lines, -1 removed lines

@@ -169,7 +169,7 @@
 
 
         $target = new EmployeeInvitationFactory();
-        $mail = $target->__invoke($services,'irrelevant',$options);
+        $mail = $target->__invoke($services, 'irrelevant', $options);
 
 
         $vars = $mail->getVariables()->getArrayCopy();

module/Organizations/test/OrganizationsTest/Entity/OrganizationTest.php

Spacing +28 added lines, -28 removed lines

@@ -60,74 +60,74 @@ 
         $parent = new Organization();
         $permissions = $this->createMock(Permissions::class);
         return [
-            [ 'externalId', '1234' ],
-            [ 'name', [ 'value'=> '', 'ignore_setter' => true] ],
-            [ 'name', [ 'pre' => 'setupGetName', 'value' => 'Test', 'ignore_setter' => true] ],
-            [ 'organizationName' , [
-                '@value' => [ '\Organizations\Entity\OrganizationName', [ 'Test' ] ],
+            ['externalId', '1234'],
+            ['name', ['value'=> '', 'ignore_setter' => true]],
+            ['name', ['pre' => 'setupGetName', 'value' => 'Test', 'ignore_setter' => true]],
+            ['organizationName', [
+                '@value' => ['\Organizations\Entity\OrganizationName', ['Test']],
                 'post' => 'organizationNamePostAsserts'
             ]],
-            [ 'organizationName', [
+            ['organizationName', [
                 'pre' => 'setupOrganizationName',
-                '@value' => [ '\Organizations\Entity\OrganizationName', [ 'TestNew' ] ],
+                '@value' => ['\Organizations\Entity\OrganizationName', ['TestNew']],
                 'post' => 'assertOrganizationNameCounter'
             ]],
-            [ 'permissions',[
+            ['permissions', [
                 'value' => $permissions
             ]],
-            [ 'image',[
+            ['image', [
                 'value' => new OrganizationImage()
             ]],
-            [ 'images',[
+            ['images', [
                 'value' => new ImageSet(),
             ]],
-            [ 'isDraft',[
+            ['isDraft', [
                 'default' => false,
                 'value' => true,
                 'getter_method' => 'isDraft'
             ]],
-            [ 'contact',[
+            ['contact', [
                 'value' => new OrganizationContact()
             ]],
-            [ 'description',[
+            ['description', [
                 'default' => null,
                 'value' => 'Some Description'
             ]],
-            [ 'parent',[
+            ['parent', [
                 'default' => null,
                 'value' => $parent,
             ]],
-            [ 'hiringOrganizations',[
+            ['hiringOrganizations', [
                 'default' => null,
             ]],
 
-            [ 'employees',[
+            ['employees', [
                 '@default' => ArrayCollection::class,
                 'value' => new ArrayCollection()
             ]],
 
-            [ 'jobs',[
+            ['jobs', [
                 'default' => null,
             ]],
 
-            [ 'user',[
+            ['user', [
                 'value' => new User()
             ]],
 
-            [ 'template',[
+            ['template', [
                 'value' => new Template(),
             ]],
 
-            [ 'workflowSettings',[
+            ['workflowSettings', [
                 'value' => new WorkflowSettings(),
             ]],
 
-            [ 'profileSetting',[
+            ['profileSetting', [
                 'value' => Organization::PROFILE_ALWAYS_ENABLE,
                 'default' => null,
             ]],
 
-            [ 'hydrator', [
+            ['hydrator', [
                 '@default' => EntityHydrator::class
             ]]
 
@@ -176,8 +176,8 @@ 
         $organization->setEmployees($employees);
 
         $organization->getEmployee($user);
-        $this->assertEquals($employee,$organization->getEmployee($user));
-        $this->assertEquals($employee,$organization->getEmployee($user->getId()));
+        $this->assertEquals($employee, $organization->getEmployee($user));
+        $this->assertEquals($employee, $organization->getEmployee($user->getId()));
         $this->assertNull($organization->getEmployee('foobar'));
         $this->assertEquals(
             [$employee],
@@ -186,7 +186,7 @@ 
                 ->toArray()
         );
 
-        $this->assertEquals(1,$organization->getEmployees()->count());
+        $this->assertEquals(1, $organization->getEmployees()->count());
         $organization->setParent($parent);
         $this->assertEquals(
             0,
@@ -199,12 +199,12 @@ 
     {
         $image = new OrganizationImage();
         $images = new ImageSet();
-        $images->set(ImageSet::ORIGINAL,$image);
+        $images->set(ImageSet::ORIGINAL, $image);
 
         $organization = new Organization();
         $organization->setImages($images);
         $organization->removeImages();
-        $this->assertNotEquals($images,$organization->getImages());
+        $this->assertNotEquals($images, $organization->getImages());
         $this->assertNull($organization->getImages()->get(ImageSet::ORIGINAL));
     }
 
@@ -299,7 +299,7 @@ 
         $permissions = $this->createMock(Permissions::class);
         $permissions->expects($this->once())
             ->method('grant')
-            ->with($user,PermissionsInterface::PERMISSION_CHANGE,false)
+            ->with($user, PermissionsInterface::PERMISSION_CHANGE, false)
         ;
 
         $organization = new Organization();

module/Organizations/test/OrganizationsTest/Entity/OrganizationReferenceTest.php

Indentation +12 added lines, -12 removed lines

@@ -41,8 +41,8 @@ 
     public function testOrganizationReferenceImplementsInterfaces()
     {
         $target = $this->getMockBuilder('\Organizations\Entity\OrganizationReference')
-                       ->disableOriginalConstructor()
-                       ->getMock();
+                        ->disableOriginalConstructor()
+                        ->getMock();
 
         $this->assertInstanceOf('\Organizations\Entity\OrganizationReferenceInterface', $target);
         $this->assertInstanceOf('\Organizations\Entity\OrganizationInterface', $target);
@@ -59,21 +59,21 @@ 
             $organization = new Organization();
 
             $repository = $this->getMockBuilder('\Organizations\Repository\Organization')
-                               ->disableOriginalConstructor()->getMock();
+                                ->disableOriginalConstructor()->getMock();
 
             $repository->expects($this->once())
-                       ->method('findByUser')
-                       ->with('1234')
-                       ->willReturn('owner' == $type ? $organization : null);
+                        ->method('findByUser')
+                        ->with('1234')
+                        ->willReturn('owner' == $type ? $organization : null);
 
             if ('owner' == $type) {
                 $repository->expects($this->never())
-                           ->method('findByEmployee');
+                            ->method('findByEmployee');
             } else {
                 $repository->expects($this->once())
-                           ->method('findByEmployee')
-                           ->with('1234')
-                           ->willReturn('employee' == $type ? $organization : null);
+                            ->method('findByEmployee')
+                            ->with('1234')
+                            ->willReturn('employee' == $type ? $organization : null);
             }
 
             $target = new OrganizationReference('1234', $repository);
@@ -196,8 +196,8 @@ 
             [['setEmployees', 'getEmployees'], [[$emps], []], ['__self__', $emps]],
             ['getEmployee', ['4321'], null],
             [['setUser', 'getUser', 'getPermissionsUserIds'],
-                  [[$user], [], []],
-                  ['__self__', $user, [PermissionsInterface::PERMISSION_ALL => [$user->getId()]]]
+                    [[$user], [], []],
+                    ['__self__', $user, [PermissionsInterface::PERMISSION_ALL => [$user->getId()]]]
             ],
             ['getJobs', [], null],
             [['setPermissions', 'getPermissions'], [[$perms], []], ['__self__', $perms]],

Spacing +3 added lines, -3 removed lines

@@ -114,7 +114,7 @@ 
             $args = array($args);
         }
 
-        for ($i=0,$c=count($func); $i<$c; $i+=1) {
+        for ($i = 0, $c = count($func); $i < $c; $i += 1) {
             $f = $func[$i];
             $a = $args[$i];
 
@@ -141,7 +141,7 @@ 
             $func = array($func); $args = array($args); $expected = array($expected);
         }
 
-        for ($i=0, $c=count($func); $i<$c; $i+=1) {
+        for ($i = 0, $c = count($func); $i < $c; $i += 1) {
             $f = $func[$i];
             $a = $args[$i];
             $e = $expected[$i];
@@ -171,7 +171,7 @@ 
         $perms = new Permissions();
         $contact = new OrganizationContact();
         $image = new OrganizationImage();
-        $externalId='myReference';
+        $externalId = 'myReference';
         $template = new Template();
         $template->setLabelBenefits('These are your Benefits');
         $workflowSettings = new WorkflowSettings();

module/Organizations/test/OrganizationsTest/Entity/OrganizationImageTest.php

Indentation +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 
     public function testGetResourceId()
     {
-       $this->assertSame($this->target->getResourceId(), 'Entity/OrganizationImage');
+        $this->assertSame($this->target->getResourceId(), 'Entity/OrganizationImage');
     }
 
     public function testGetUriWithAndWithoutFilename()

Spacing +2 added lines, -2 removed lines

@@ -54,14 +54,14 @@
 
     public function testGetUriWithAndWithoutFilename()
     {
-        $filename="My File.png";
+        $filename = "My File.png";
         $this->target->setName($filename);
         $this->assertSame($this->target->getUri(), '/file/Organizations.OrganizationImage/'.$this->target->getId().'/'.urlencode($this->target->getName()));
         $this->target->setName(null);
         $this->assertSame($this->target->getUri(), '/file/Organizations.OrganizationImage/'.$this->target->getId());
     }
 
-    public function testSetGetOrganization(){
+    public function testSetGetOrganization() {
         $organization = new Organization();
         $this->target->setOrganization($organization);
         $this->assertSame($this->target->getOrganization(), $organization);