aerialship / SamlSPBundle

src/AerialShip/SamlSPBundle/Tests/RelyingParty/RelyingPartyCollectionTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -233,7 +233,7 @@ 
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|\Symfony\Component\HttpFoundation\Request
+     * @return \Symfony\Component\HttpFoundation\Request
      */
     public function createRequestMock()
     {
@@ -241,7 +241,7 @@ 
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|\AerialShip\SamlSPBundle\RelyingParty\RelyingPartyInterface
+     * @return \AerialShip\SamlSPBundle\RelyingParty\RelyingPartyInterface
      */
     public function createRelyingPartyMock()
     {

src/AerialShip/SamlSPBundle/Tests/Security/Http/SamlSpAuthenticationListenerTest.php

Doc Comments +14 added lines, -11 removed lines

@@ -255,7 +255,7 @@ 
      * @param bool $hasPreviousSession
      * @param mixed $duplicateReturn
      * @param mixed $getSessionReturn
-     * @return \PHPUnit_Framework_MockObject_MockObject|Request
+     * @return Request|null
      */
     private function createRequestStub($hasSessionReturn = null, $hasPreviousSession = null, $duplicateReturn = null, $getSessionReturn = null)
     {
@@ -286,13 +286,16 @@ 
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|RelyingPartyInterface
+     * @return RelyingPartyInterface
      */
     private function createRelyingPartyMock()
     {
         return $this->getMock('AerialShip\SamlSPBundle\RelyingParty\RelyingPartyInterface');
     }
 
+    /**
+     * @param boolean $supportsReturn
+     */
     private function createRelyingPartyStub($supportsReturn = null, $manageReturn = null)
     {
         $relyingPartyMock = $this->createRelyingPartyMock();
@@ -321,7 +324,7 @@ 
 
     /**
      * @param Request|null $request
-     * @return \PHPUnit_Framework_MockObject_MockObject|GetResponseEvent
+     * @return GetResponseEvent
      */
     private function createGetResponseEventStub($request = null)
     {
@@ -346,7 +349,7 @@ 
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|SecurityContextInterface
+     * @return \Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface
      */
     private function createSecurityContextMock()
     {
@@ -362,7 +365,7 @@ 
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|SessionAuthenticationStrategyInterface
+     * @return SessionAuthenticationStrategyInterface
      */
     private function createSessionAuthenticationStrategyMock()
     {
@@ -370,7 +373,7 @@ 
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|HttpUtils
+     * @return HttpUtils
      */
     private function createHttpUtilsMock()
     {
@@ -378,8 +381,8 @@ 
     }
 
     /**
-     * @param null $checkRequestPathResult
-     * @param null $createRedirectResponseReturn
+     * @param boolean $checkRequestPathResult
+     * @param RedirectResponse $createRedirectResponseReturn
      * @return \PHPUnit_Framework_MockObject_MockObject|HttpUtils
      */
     private function createHttpUtilsStub($checkRequestPathResult = null, $createRedirectResponseReturn = null)
@@ -402,7 +405,7 @@ 
 
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|AuthenticationSuccessHandlerInterface
+     * @return AuthenticationSuccessHandlerInterface
      */
     private function createAuthenticationSuccessHandlerMock()
     {
@@ -410,7 +413,7 @@ 
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|AuthenticationSuccessHandlerInterface
+     * @return AuthenticationSuccessHandlerInterface
      */
     protected function createAuthenticationSuccessHandlerStub()
     {
@@ -426,7 +429,7 @@ 
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|AuthenticationFailureHandlerInterface
+     * @return AuthenticationFailureHandlerInterface
      */
     private function createAuthenticationFailureHandlerMock()
     {

src/AerialShip/SamlSPBundle/Bridge/Discovery.php

Doc Comments +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
      * @param \Symfony\Component\HttpFoundation\Request $request
      * @throws \RuntimeException  if no signing provider set
      * @throws \InvalidArgumentException if cannot manage the Request
-     * @return \Symfony\Component\HttpFoundation\Response|SamlSpInfo
+     * @return Response
      */
     function manage(Request $request)
     {

src/AerialShip/SamlSPBundle/Bridge/FederationMetadata.php

Doc Comments +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
      * @param \Symfony\Component\HttpFoundation\Request $request
      * @throws \RuntimeException  if no signing provider set
      * @throws \InvalidArgumentException if cannot manage the Request
-     * @return \Symfony\Component\HttpFoundation\Response|SamlSpInfo
+     * @return Response
      */
     function manage(Request $request)
     {

src/AerialShip/SamlSPBundle/Bridge/LogoutFallback.php

Doc Comments +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
      * @param \Symfony\Component\HttpFoundation\Request $request
      * @throws \RuntimeException
      * @throws \InvalidArgumentException if cannot manage the Request
-     * @return \Symfony\Component\HttpFoundation\Response|SamlSpInfo|null
+     * @return \Symfony\Component\HttpFoundation\RedirectResponse
      */
     public function manage(Request $request)
     {