lightSAML / lightSAML

tests/LightSaml/Tests/Action/Profile/Inbound/Message/DestinationValidatorResponseActionTest.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 
         $endpointResolverMock->expects($this->once())
             ->method('resolve')
-            ->willReturnCallback(function (CriteriaSet $criteriaSet, array $endpoints) {
+            ->willReturnCallback(function(CriteriaSet $criteriaSet, array $endpoints) {
                 $this->assertTrue($criteriaSet->has(ServiceTypeCriteria::class));
                 $arr = $criteriaSet->get(ServiceTypeCriteria::class);
                 $this->assertCount(1, $arr);

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 use LightSaml\Model\Protocol\AuthnRequest;
 use LightSaml\Profile\Profiles;
 use LightSaml\Resolver\Endpoint\Criteria\ServiceTypeCriteria;
-use LightSaml\Resolver\Endpoint\EndpointResolverInterface;
 use LightSaml\Tests\BaseTestCase;
 
 class DestinationValidatorResponseActionTest extends BaseTestCase

tests/LightSaml/Tests/Action/Profile/FlushRequestStatesActionTest.php

Unused Use Statements -2 removed lines

@@ -9,9 +9,7 @@
 use LightSaml\Context\Profile\RequestStateContext;
 use LightSaml\Profile\Profiles;
 use LightSaml\State\Request\RequestState;
-use LightSaml\Store\Request\RequestStateStoreInterface;
 use LightSaml\Tests\BaseTestCase;
-use Psr\Log\LoggerInterface;
 
 class FlushRequestStatesActionTest extends BaseTestCase
 {

tests/LightSaml/Tests/Action/Profile/Inbound/Message/IssuerValidatorActionTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|\LightSaml\Validator\Model\NameId\NameIdValidatorInterface
+     * @return NameIdValidatorInterface
      */
     public function getNameIdValidatorMock()
     {

tests/LightSaml/Tests/Action/Profile/Inbound/Message/ResolvePartyEntityIdActionTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -193,7 +193,7 @@
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|\LightSaml\Store\TrustOptions\TrustOptionsStoreInterface
+     * @return TrustOptionsStoreInterface
      */
     private function getTrustOptionsStore()
     {

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

Doc Comments +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
      * @param LoggerInterface           $logger
      * @param EndpointResolverInterface $endpointResolver
      *
-     * @return ResolveEndpointBaseAction
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function createAction(LoggerInterface $logger, EndpointResolverInterface $endpointResolver)
     {

Spacing +4 added lines, -4 removed lines

@@ -41,7 +41,7 @@ 
         $context = $this->createContext(ProfileContext::ROLE_IDP, $message);
 
         $endpoint = null;
-        $this->setEndpointResolver(true, function (CriteriaSet $criteriaSet, array $endpointCandidates) use (&$endpoint) {
+        $this->setEndpointResolver(true, function(CriteriaSet $criteriaSet, array $endpointCandidates) use (&$endpoint) {
             $this->criteriaSetShouldHaveBindingCriteria(
                 $criteriaSet,
                 [SamlConstants::BINDING_SAML2_HTTP_POST, SamlConstants::BINDING_SAML2_HTTP_REDIRECT]
@@ -67,7 +67,7 @@ 
         $context = $this->createContext(ProfileContext::ROLE_IDP, $message);
 
         $endpoint = null;
-        $this->setEndpointResolver(true, function () {
+        $this->setEndpointResolver(true, function() {
             return [];
         });
 
@@ -80,7 +80,7 @@ 
         $message->setAssertionConsumerServiceIndex($index = 2);
         $context = $this->createContext(ProfileContext::ROLE_IDP, $message);
 
-        $this->setEndpointResolver(true, function (CriteriaSet $criteriaSet) use ($index) {
+        $this->setEndpointResolver(true, function(CriteriaSet $criteriaSet) use ($index) {
             $this->criteriaSetShouldHaveIndexCriteria($criteriaSet, $index);
 
             return [$this->getEndpointReferenceMock($endpoint = new SingleSignOnService())];
@@ -95,7 +95,7 @@ 
         $message->setAssertionConsumerServiceURL($url = 'http://domain.com/acs');
         $context = $this->createContext(ProfileContext::ROLE_IDP, $message);
 
-        $this->setEndpointResolver(true, function (CriteriaSet $criteriaSet) use ($url) {
+        $this->setEndpointResolver(true, function(CriteriaSet $criteriaSet) use ($url) {
             $this->criteriaSetShouldHaveLocationCriteria($criteriaSet, $url);
 
             return [$this->getEndpointReferenceMock($endpoint = new SingleSignOnService())];

tests/LightSaml/Tests/Helper/ContactPersonChecker.php

Doc Comments +5 added lines

@@ -7,6 +7,11 @@
 
 class ContactPersonChecker
 {
+    /**
+     * @param string $givenName
+     * @param string $surName
+     * @param string $email
+     */
     public static function check(
         BaseTestCase $test,
         $type,

tests/LightSaml/Tests/Helper/IndexedEndpointChecker.php

Doc Comments +5 added lines

@@ -7,6 +7,11 @@
 
 class IndexedEndpointChecker
 {
+    /**
+     * @param string $location
+     * @param integer $index
+     * @param boolean $isDefault
+     */
     public static function check(BaseTestCase $test, $binding, $location, $index, $isDefault, IndexedEndpoint $svc = null)
     {
         EndpointChecker::check($test, $binding, $location, $svc);

tests/LightSaml/Tests/Helper/KeyDescriptorChecker.php

Doc Comments +3 added lines

@@ -7,6 +7,9 @@
 
 class KeyDescriptorChecker
 {
+    /**
+     * @param string $cn
+     */
     public static function checkCertificateCN(BaseTestCase $test, $use, $cn, KeyDescriptor $kd = null)
     {
         $test->assertNotNull($kd);

tests/LightSaml/Tests/Helper/OrganizationChecker.php

Doc Comments +5 added lines

@@ -7,6 +7,11 @@
 
 class OrganizationChecker
 {
+    /**
+     * @param string $name
+     * @param string $display
+     * @param string $url
+     */
     public static function check(BaseTestCase $test, $name, $display, $url, Organization $organization = null)
     {
         $test->assertNotNull($organization);