lightSAML / lightSAML

tests/LightSaml/Tests/Resolver/Credential/PrivateKeyResolverTest.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use LightSaml\Criteria\CriteriaSet;
 use LightSaml\Resolver\Credential\PrivateKeyResolver;
 use LightSaml\Tests\BaseTestCase;
-use RobRichards\XMLSecLibs\XMLSecurityKey;
 
 class PrivateKeyResolverTest extends BaseTestCase
 {

tests/LightSaml/Tests/Criteria/CriteriaSetTest.php

Spacing +2 added lines, -2 removed lines

@@ -39,11 +39,11 @@
     {
         $criteriaSet = new CriteriaSet();
 
-        $criteriaSet->addIf(false, function () {
+        $criteriaSet->addIf(false, function() {
             return $this->getCriteriaMock();
 
         });
-        $criteriaSet->addIf(true, function () {
+        $criteriaSet->addIf(true, function() {
             return $this->getCriteriaMock();
 
         });

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

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
         $message = new AuthnRequest();
         $context = $this->createContext(ProfileContext::ROLE_IDP, $message);
 
-        $this->setEndpointResolver(true, function (CriteriaSet $criteriaSet) {
+        $this->setEndpointResolver(true, function(CriteriaSet $criteriaSet) {
             $this->criteriaSetShouldHaveServiceTypeCriteria($criteriaSet, SingleSignOnService::class);
 
             return [$this->getEndpointReferenceMock($endpoint = new SingleSignOnService())];

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

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
         $message = new AuthnRequest();
         $context = $this->createContext(ProfileContext::ROLE_IDP, $message);
 
-        $this->setEndpointResolver(true, function (CriteriaSet $criteriaSet) {
+        $this->setEndpointResolver(true, function(CriteriaSet $criteriaSet) {
             $this->criteriaSetShouldHaveServiceTypeCriteria($criteriaSet, AssertionConsumerService::class);
 
             return [$this->getEndpointReferenceMock($endpoint = new AssertionConsumerService())];

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

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
         $context->getOwnEntityContext()->setEntityDescriptor(new EntityDescriptor($ownEntityId = 'http://own.id'));
         $context->getLogoutContext()->setSsoSessionState((new SsoSessionState())->setIdpEntityId($ownEntityId));
 
-        $this->setEndpointResolver(true, function (CriteriaSet $criteriaSet) {
+        $this->setEndpointResolver(true, function(CriteriaSet $criteriaSet) {
             $this->criteriaSetShouldHaveServiceTypeCriteria($criteriaSet, SingleLogoutService::class);
 
             return [$this->getEndpointReferenceMock($endpoint = new SingleLogoutService())];
@@ -53,7 +53,7 @@ 
         $context->getOwnEntityContext()->setEntityDescriptor(new EntityDescriptor($ownEntityId = 'http://own.id'));
         $context->getLogoutContext()->setSsoSessionState((new SsoSessionState())->setIdpEntityId($ownEntityId));
 
-        $this->setEndpointResolver(true, function (CriteriaSet $criteriaSet) {
+        $this->setEndpointResolver(true, function(CriteriaSet $criteriaSet) {
             $this->criteriaSetShouldHaveDescriptorTypeCriteria($criteriaSet, SpSsoDescriptor::class);
 
             return [$this->getEndpointReferenceMock($endpoint = new SingleLogoutService())];
@@ -70,7 +70,7 @@ 
         $context->getOwnEntityContext()->setEntityDescriptor(new EntityDescriptor($ownEntityId = 'http://own.id'));
         $context->getLogoutContext()->setSsoSessionState((new SsoSessionState())->setSpEntityId($ownEntityId));
 
-        $this->setEndpointResolver(true, function (CriteriaSet $criteriaSet) {
+        $this->setEndpointResolver(true, function(CriteriaSet $criteriaSet) {
             $this->criteriaSetShouldHaveDescriptorTypeCriteria($criteriaSet, IdpSsoDescriptor::class);
 
             return [$this->getEndpointReferenceMock($endpoint = new SingleLogoutService())];

tests/LightSaml/Tests/Action/Assertion/Inbound/RecipientValidatorActionTest.php

Spacing +2 added lines, -2 removed lines

@@ -96,7 +96,7 @@ 
         $endpointResolver->expects($this->once())
             ->method('resolve')
             ->with($this->isInstanceOf(CriteriaSet::class), $this->isType('array'))
-            ->willReturnCallback(function (CriteriaSet $criteriaSet) use ($recipient) {
+            ->willReturnCallback(function(CriteriaSet $criteriaSet) use ($recipient) {
                 $this->assertCriteria($criteriaSet, DescriptorTypeCriteria::class, 'getDescriptorType', SpSsoDescriptor::class);
                 $this->assertCriteria($criteriaSet, ServiceTypeCriteria::class, 'getServiceType', AssertionConsumerService::class);
                 $this->assertCriteria($criteriaSet, LocationCriteria::class, 'getLocation', $recipient);
@@ -130,7 +130,7 @@ 
 
         $endpointResolver->expects($this->once())
             ->method('resolve')
-            ->willReturnCallback(function () use ($recipient) {
+            ->willReturnCallback(function() use ($recipient) {
                 return [$this->getEndpointReferenceMock(new AssertionConsumerService())];
             });
 

tests/LightSaml/Tests/Resolver/Signature/OwnSignatureResolverTest.php

Spacing +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
 
         $credentialResolverMock->method('query')->willReturn($query = new CredentialResolverQuery($credentialResolverMock));
         $credentialResolverMock->method('resolve')
-            ->willReturnCallback(function (CriteriaSet $criteriaSet) use ($ownEntityId, $expectedMetadataType) {
+            ->willReturnCallback(function(CriteriaSet $criteriaSet) use ($ownEntityId, $expectedMetadataType) {
                 $this->assertCriteria($criteriaSet, EntityIdCriteria::class, 'getEntityId', $ownEntityId);
                 $this->assertCriteria($criteriaSet, UsageCriteria::class, 'getUsage', UsageType::SIGNING);
                 $this->assertCriteria($criteriaSet, X509CredentialCriteria::class, null, null);

src/LightSaml/Credential/AbstractCredential.php

Doc Comments +2 added lines, -2 removed lines

@@ -150,7 +150,7 @@ 
     }
 
     /**
-     * @param XMLSecurityKey|null $privateKey
+     * @param XMLSecurityKey $privateKey
      *
      * @return AbstractCredential
      */
@@ -162,7 +162,7 @@ 
     }
 
     /**
-     * @param XMLSecurityKey|null $publicKey
+     * @param XMLSecurityKey $publicKey
      *
      * @return AbstractCredential
      */

src/LightSaml/Model/AbstractSamlModel.php

Doc Comments +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
     }
 
     /**
-     * @param array|string[] $names
+     * @param string[] $names
      * @param string|null    $namespace
      */
     protected function singleElementsToXml(array $names, \DOMNode $parent, SerializationContext $context, $namespace = null)
@@ -165,7 +165,7 @@ 
     }
 
     /**
-     * @param array|string[] $names
+     * @param string[] $names
      */
     protected function attributesToXml(array $names, \DOMElement $element)
     {