lightSAML / lightSAML

tests/LightSaml/Tests/Action/Profile/Outbound/AuthnRequest/ACSUrlActionTest.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         $endpointResolverMock->expects($this->once())
             ->method('resolve')
             ->with($this->isInstanceOf(CriteriaSet::class), $this->isType('array'))
-            ->willReturnCallback(function (CriteriaSet $criteriaSet, array $candidates) {
+            ->willReturnCallback(function(CriteriaSet $criteriaSet, array $candidates) {
                 $this->assertTrue($criteriaSet->has(DescriptorTypeCriteria::class));
                 $this->assertEquals(SpSsoDescriptor::class, $criteriaSet->getSingle(DescriptorTypeCriteria::class)->getDescriptorType());
 

tests/LightSaml/Tests/Action/Profile/Outbound/Message/SaveRequestStateActionTest.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
         $requestStateStoreMock->expects($this->once())
             ->method('set')
             ->with($this->isInstanceOf(RequestState::class))
-            ->willReturnCallback(function (RequestState $requestState) use ($id) {
+            ->willReturnCallback(function(RequestState $requestState) use ($id) {
                 $this->assertEquals($id, $requestState->getId());
             })
         ;

tests/LightSaml/Tests/Context/AbstractContextTest.php

Spacing +1 added lines, -1 removed lines

@@ -222,7 +222,7 @@
         $profileContext->addSubContext('assertion_01', $assertionSubContext01 = new AssertionContext());
         $assertionSubContext01->addSubContext('rs', new RequestStateContext());
 
-        $actual = (string)$profileContext;
+        $actual = (string) $profileContext;
 
         $expected = <<<EOT
 {

tests/LightSaml/Tests/Functional/Bridge/Pimple/ProfileTest.php

Spacing +3 added lines, -3 removed lines

@@ -198,14 +198,14 @@ 
         // SYSTEM
         $buildContainer->getPimple()->register(new \LightSaml\Bridge\Pimple\Container\Factory\SystemContainerProvider(true));
         if ($timeProvider) {
-            $pimple[SystemContainer::TIME_PROVIDER] = function () use ($timeProvider) {
+            $pimple[SystemContainer::TIME_PROVIDER] = function() use ($timeProvider) {
                 return $timeProvider;
             };
         }
 
         // PARTY
         $buildContainer->getPimple()->register(new \LightSaml\Bridge\Pimple\Container\Factory\PartyContainerProvider());
-        $pimple[PartyContainer::IDP_ENTITY_DESCRIPTOR] = function () {
+        $pimple[PartyContainer::IDP_ENTITY_DESCRIPTOR] = function() {
             $idpProvider = new \LightSaml\Store\EntityDescriptor\FixedEntityDescriptorStore();
             $idpProvider->add(
                 \LightSaml\Model\Metadata\EntitiesDescriptor::load(__DIR__.'/../../../../../../web/sp/testshib-providers.xml')
@@ -227,7 +227,7 @@ 
             )
         );
         if ($inResponseTo) {
-            $pimple[StoreContainer::REQUEST_STATE_STORE] = function () use ($inResponseTo) {
+            $pimple[StoreContainer::REQUEST_STATE_STORE] = function() use ($inResponseTo) {
                 $store = new RequestStateArrayStore();
                 $store->set(new RequestState($inResponseTo));
 

Doc Comments +3 added lines

@@ -164,6 +164,9 @@
         $this->assertInstanceOf(\LightSaml\Store\EntityDescriptor\EntityDescriptorStoreInterface::class, $buildContainer->getPartyContainer()->getSpEntityDescriptorStore());
     }
 
+    /**
+     * @param string $inResponseTo
+     */
     private function getBuildContainer($inResponseTo = null, TimeProviderInterface $timeProvider = null)
     {
         $buildContainer = new BuildContainer($pimple = new Container());

tests/LightSaml/Tests/Store/Request/RequestStateSessionStoreTest.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 
         $sessionMock->method('set')
             ->with('main_saml_request_state_', $this->isType('array'))
-            ->willReturnCallback(function ($name, $value) use ($requestState) {
+            ->willReturnCallback(function($name, $value) use ($requestState) {
                 $this->assertArrayHasKey('aaa', $value);
                 $this->assertSame($requestState, $value['aaa']);
             });

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace LightSaml\Tests\Store\Request;
 
-use LightSaml\Meta\ParameterBag;
 use LightSaml\State\Request\RequestState;
 use LightSaml\Store\Request\RequestStateSessionStore;
 use LightSaml\Tests\BaseTestCase;

tests/LightSaml/Tests/HelperTest.php

Spacing +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
         $this->assertEquals(43, strlen($id));
 
         $arr = array();
-        for ($i = 0; $i<strlen($id); $i++) {
+        for ($i = 0; $i < strlen($id); $i++) {
             $ch = $id[$i];
             $arr[$ch] = true;
         }

src/LightSaml/Model/Protocol/LogoutRequest.php

Doc Comments +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
     }
 
     /**
-     * @param int|\DateTime|string $notOnOrAfter
+     * @param \DateTime $notOnOrAfter
      *
      * @return LogoutRequest
      */

tests/LightSaml/Tests/Model/Protocol/ResponseTest.php

Unused Use Statements -2 removed lines

@@ -18,8 +18,6 @@
 use LightSaml\Model\Assertion\SubjectConfirmation;
 use LightSaml\Model\Assertion\SubjectConfirmationData;
 use LightSaml\Model\Protocol\Response;
-use LightSaml\Model\Protocol\Status;
-use LightSaml\Model\Protocol\StatusCode;
 use LightSaml\Model\XmlDSig\SignatureWriter;
 use LightSaml\SamlConstants;
 use LightSaml\Credential\KeyHelper;

src/LightSaml/Model/Protocol/AuthnRequest.php

Doc Comments +2 added lines, -2 removed lines

@@ -56,7 +56,7 @@ 
     protected $subject;
 
     /**
-     * @param Subject|null $subject
+     * @param Subject $subject
      *
      * @return AuthnRequest
      */
@@ -116,7 +116,7 @@ 
     }
 
     /**
-     * @param NameIDPolicy|null $nameIDPolicy
+     * @param NameIDPolicy $nameIDPolicy
      *
      * @return AuthnRequest
      */