Issues in SamlSpAuthenticationProviderTest.php - New Issues - Inspection of "Merge pull request #110 from tmilos/limit_symfony_..." - aerialship/SamlSPBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( d5bbb9...ab4758 )

by Milos

created 2016-01-03 08:15 UTC

Provider/SamlSpAuthenticationProviderTest.php (31 issues)

Labels

Bug 31

Severity

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php

namespace AerialShip\SamlSPBundle\Tests\Security\Core\Authentication\Provider;

use AerialShip\SamlSPBundle\Security\Core\Authentication\Provider\SamlSpAuthenticationProvider;
use AerialShip\SamlSPBundle\Security\Core\Authentication\Token\SamlSpToken;
use AerialShip\SamlSPBundle\Tests\Bridge\SamlSpInfoHelper;
use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
use Symfony\Component\Security\Core\Exception\AuthenticationServiceException;
use Symfony\Component\Security\Core\Exception\UsernameNotFoundException;

class SamlSpAuthenticationProviderTest extends \PHPUnit_Framework_TestCase
{
    /**
     * @test
     */
    public function couldBeConstructedWithProviderKey()
    {
        new SamlSpAuthenticationProvider('key');
    }


    /**
     * @test
     */
    public function couldBeConstructedWithUserManagerAndUserChecker()
    {
        new SamlSpAuthenticationProvider(
            'main',
            $this->createUserManagerMock(),

            $this->createUserCheckerMock()

        );
    }

    /**
     * @test
     */
    public function couldBeConstructedWithUserManagerAndUserCheckerAndCreateUserIfNotExist()
    {
        new SamlSpAuthenticationProvider(
            'main',
            $this->createUserManagerMock(),

            $this->createUserCheckerMock(),

            true
        );
    }


    /**
     * @test
     * @expectedException \InvalidArgumentException
     */
    public function throwIfTryConstructWithUserManagerButWithoutUserChecker()
    {
        new SamlSpAuthenticationProvider(
            'main',
            $this->createUserManagerMock(),

            null,
            false
        );
    }

    /**
     * @test
     * @expectedException \InvalidArgumentException
     */
    public function throwIfTryConstructWithCreateIfNotExistSetTrueButWithoutUserManager()
    {
        new SamlSpAuthenticationProvider(
            'main',
            null,
            null,
            true
        );
    }


    /**
     * @test
     */
    public function shouldSupportSamlSpToken()
    {
        $providerKey = 'main';
        $authProvider = new SamlSpAuthenticationProvider($providerKey);
        $this->assertTrue($authProvider->supports(new SamlSpToken($providerKey)));
    }

    /**
     * @test
     */
    public function shouldNotSupportNonOpenIdToken()
    {
        $authProvider = new SamlSpAuthenticationProvider('main');

        /** @var $nonOpenIdToken TokenInterface */
        $nonOpenIdToken = $this->getMock('Symfony\Component\Security\Core\Authentication\Token\TokenInterface');

        $this->assertFalse($authProvider->supports($nonOpenIdToken));
        $this->assertNull($authProvider->authenticate($nonOpenIdToken));
    }


    /**
     * @test
     */
    public function shouldNotSupportSamlSpTokenIfProviderKeyDiffers()
    {
        $token = new SamlSpToken('the_other_provider_key');
        $authProvider = new SamlSpAuthenticationProvider('the_provider_key');
        $this->assertFalse($authProvider->supports($token));
    }


    /**
     * @test
     */
    public function shouldCreateAuthenticatedTokenUsingUserAndHisRolesFromToken()
    {
        $samlSpInfoHelper = new SamlSpInfoHelper();

        $providerKey = 'main';
        $expectedSamlSpInfo = $samlSpInfoHelper->getSamlSpInfo();

        $expectedUserMock = $this->createUserMock();
        $expectedUserMock
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                ->expects($this->any())
                ->method('getRoles')
                ->will($this->returnValue(array('foo', 'bar')))
        ;

        $authProvider = new SamlSpAuthenticationProvider(
            $providerKey,
            $userManager = null,
            $this->createUserCheckerMock()

        );

        $token = new SamlSpToken($providerKey);
        $token->setUser($expectedUserMock);
        $token->setSamlSpInfo($expectedSamlSpInfo);

        /** @var  $authenticatedToken SamlSpToken */
        $authenticatedToken = $authProvider->authenticate($token);

        $this->assertInstanceOf('AerialShip\SamlSPBundle\Security\Core\Authentication\Token\SamlSpToken', $authenticatedToken);
        $this->assertNotSame($token, $authenticatedToken);
        $this->assertTrue($authenticatedToken->isAuthenticated());
        $this->assertEquals($expectedSamlSpInfo, $authenticatedToken->getSamlSpInfo());
        $this->assertSame($authenticatedToken->getUser(), $expectedUserMock);

        $roles = $authenticatedToken->getRoles();
        $this->assertInternalType('array', $roles);
        $this->assertCount(2, $roles);

        $this->assertEquals('foo', $roles[0]->getRole());
        $this->assertEquals('bar', $roles[1]->getRole());
    }




    /**
     * @test
     */
    public function shouldCreateAuthenticatedTokenUsingUserFromTokenAndCallPostAuthCheck()
    {
        $samlSpInfoHelper = new SamlSpInfoHelper();

        $providerKey = 'main';
        $expectedSamlSpInfo = $samlSpInfoHelper->getSamlSpInfo();

        $userMock = $this->createUserMock();
        $userMock
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                ->expects($this->any())
                ->method('getRoles')
                ->will($this->returnValue(array()))
        ;

        $userCheckerMock = $this->createUserCheckerMock();
        $userCheckerMock
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                ->expects($this->once())
                ->method('checkPostAuth')
                ->with($userMock)
        ;

        $authProvider = new SamlSpAuthenticationProvider(
            $providerKey,
            $this->createUserManagerMock(),

            $userCheckerMock
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
        );

        $token = new SamlSpToken($providerKey);
        $token->setUser($userMock);
        $token->setSamlSpInfo($expectedSamlSpInfo);

        $authenticatedToken = $authProvider->authenticate($token);

        $this->assertInstanceOf('AerialShip\SamlSPBundle\Security\Core\Authentication\Token\SamlSpToken', $authenticatedToken);
        $this->assertSame($authenticatedToken->getUser(), $userMock);
    }


    /**
     * @test
     */
    public function shouldCreateAuthenticatedTokenUsingIdentityIfUserManagerNotSet()
    {
        $samlSpInfoHelper = new SamlSpInfoHelper();

        $expectedProviderKey = 'the_provider_key';
        $expectedSamlSpInfo = $samlSpInfoHelper->getSamlSpInfo();
        $expectedUsername = $expectedSamlSpInfo->getNameID()->getValue();

        $authProvider = new SamlSpAuthenticationProvider($expectedProviderKey);

        $token = new SamlSpToken($expectedProviderKey);
        $token->setUser('');
        $token->setSamlSpInfo($expectedSamlSpInfo);

        /** @var $authenticatedToken SamlSpToken */
        $authenticatedToken = $authProvider->authenticate($token);

        $this->assertInstanceOf('AerialShip\SamlSPBundle\Security\Core\Authentication\Token\SamlSpToken', $authenticatedToken);
        $this->assertNotSame($token, $authenticatedToken);
        $this->assertTrue($authenticatedToken->isAuthenticated());
        $this->assertEquals($expectedSamlSpInfo, $authenticatedToken->getSamlSpInfo());
        $this->assertEquals($expectedProviderKey, $authenticatedToken->getProviderKey());

        /** @var $user \Symfony\Component\Security\Core\User\User */
        $user = $authenticatedToken->getUser();
        $this->assertInstanceOf('Symfony\Component\Security\Core\User\User', $user);
        $this->assertEquals($expectedUsername, $user->getUsername());

        $roles = $authenticatedToken->getRoles();
        $this->assertInternalType('array', $roles);
        $this->assertCount(1, $roles);
        $this->assertEquals('ROLE_USER', $roles[0]->getRole());
    }


    /**
     * @test
     */
    public function shouldCreateAuthenticatedTokenUsingUserManagerAndSearchBySamlSpInfo()
    {
        $samlSpInfoHelper = new SamlSpInfoHelper();

        $expectedProviderKey = 'the_provider_key';
        $expectedSamlSpInfo = $samlSpInfoHelper->getSamlSpInfo();

        $expectedUserMock = $this->createUserMock();
        $expectedUserMock
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                ->expects($this->any())
                ->method('getRoles')
                ->will($this->returnValue(array('foo', 'bar')))
        ;

        $userManagerMock = $this->createUserManagerMock();
        $userManagerMock
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                ->expects($this->once())
                ->method('loadUserBySamlInfo')
                ->with($expectedSamlSpInfo)
                ->will($this->returnValue($expectedUserMock))
        ;

        $authProvider = new SamlSpAuthenticationProvider(
            $expectedProviderKey,
            $userManagerMock,
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
            $this->createUserCheckerMock()

        );

        $token = new SamlSpToken($expectedProviderKey);
        $token->setUser('');
        $token->setSamlSpInfo($expectedSamlSpInfo);

        /** @var $authenticatedToken SamlSpToken */
        $authenticatedToken = $authProvider->authenticate($token);

        $this->assertInstanceOf('AerialShip\SamlSPBundle\Security\Core\Authentication\Token\SamlSpToken', $authenticatedToken);
        $this->assertNotSame($token, $authenticatedToken);
        $this->assertTrue($authenticatedToken->isAuthenticated());
        $this->assertEquals($expectedSamlSpInfo, $authenticatedToken->getSamlSpInfo());
        $this->assertEquals($expectedProviderKey, $authenticatedToken->getProviderKey());
        $this->assertEquals($expectedUserMock, $authenticatedToken->getUser());

        $roles = $authenticatedToken->getRoles();
        $this->assertInternalType('array', $roles);
        $this->assertCount(2, $roles);

        $this->assertEquals('foo', $roles[0]->getRole());
        $this->assertEquals('bar', $roles[1]->getRole());
    }


    /**
     * @test
     *
     * @expectedException \Symfony\Component\Security\Core\Exception\AuthenticationServiceException
     * @expectedExceptionMessage User provider did not return an implementation of user interface.
     */
    public function throwIfUserManagerReturnNonUserInstance()
    {
        $samlSpInfoHelper = new SamlSpInfoHelper();

        $providerKey = 'main';
        $expectedSamlSpInfo = $samlSpInfoHelper->getSamlSpInfo();

        $userProviderMock = $this->createUserManagerMock();
        $userProviderMock
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                ->expects($this->once())
                ->method('loadUserBySamlInfo')
                ->with($expectedSamlSpInfo)
                ->will($this->returnValue('not-valid-user-instance'))
        ;

        $authProvider = new SamlSpAuthenticationProvider(
            $providerKey,
            $userProviderMock,
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
            $this->createUserCheckerMock()

        );

        $token = new SamlSpToken($providerKey);
        $token->setUser('');
        $token->setSamlSpInfo($expectedSamlSpInfo);

        $authProvider->authenticate($token);
    }


    /**
     * @test
     *
     * @expectedException \Symfony\Component\Security\Core\Exception\UsernameNotFoundException
     * @expectedExceptionMessage Cannot find user by saml sp info
     */
    public function shouldNotCreateUserIfCreateIfNotExistParamIsNotSet()
    {
        $samlSpInfoHelper = new SamlSpInfoHelper();

        $providerKey = 'main';
        $expectedSamlSpInfo = $samlSpInfoHelper->getSamlSpInfo();

        $userManagerMock = $this->createUserManagerMock();
        $userManagerMock
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                ->expects($this->once())
                ->method('loadUserBySamlInfo')
                ->with($expectedSamlSpInfo)
                ->will($this->throwException(new UsernameNotFoundException('Cannot find user by saml sp info')))
        ;
        $userManagerMock
                ->expects($this->never())
                ->method('createUserFromSamlInfo')
        ;

        $authProvider = new SamlSpAuthenticationProvider(
            $providerKey,
            $userManagerMock,
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
            $this->createUserCheckerMock(),

            $createIfNotExist = false
        );

        $token = new SamlSpToken($providerKey);
        $token->setUser('');
        $token->setSamlSpInfo($expectedSamlSpInfo);

        $authProvider->authenticate($token);
    }



    /**
     * @test
     */
    public function shouldCreateAuthenticatedTokenUsingUserManagerCreateFromSamlSpInfoMethod()
    {
        $samlSpInfoHelper = new SamlSpInfoHelper();

        $expectedProviderKey = 'main';
        $expectedSamlSpInfo = $samlSpInfoHelper->getSamlSpInfo();

        $expectedUserMock = $this->createUserMock();
        $expectedUserMock
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                ->expects($this->any())
                ->method('getRoles')
                ->will($this->returnValue(array('foo', 'bar')))
        ;

        $userManagerMock = $this->createUserManagerMock();
        $userManagerMock
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                ->expects($this->once())
                ->method('loadUserBySamlInfo')
                ->with($expectedSamlSpInfo)
                ->will($this->throwException(new UsernameNotFoundException('Cannot find user by saml sp info')))
        ;
        $userManagerMock
                ->expects($this->once())
                ->method('createUserFromSamlInfo')
                ->with($expectedSamlSpInfo)
                ->will($this->returnValue($expectedUserMock))
        ;

        $authProvider = new SamlSpAuthenticationProvider(
            $expectedProviderKey,
            $userManagerMock,
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
            $this->createUserCheckerMock(),

            $createIfNotExist = true
        );

        $token = new SamlSpToken($expectedProviderKey);
        $token->setUser('');
        $token->setSamlSpInfo($expectedSamlSpInfo);

        /** @var $authenticatedToken SamlSpToken */
        $authenticatedToken = $authProvider->authenticate($token);

        $this->assertInstanceOf('AerialShip\SamlSPBundle\Security\Core\Authentication\Token\SamlSpToken', $authenticatedToken);
        $this->assertNotSame($token, $authenticatedToken);
        $this->assertTrue($authenticatedToken->isAuthenticated());
        $this->assertEquals($expectedSamlSpInfo, $authenticatedToken->getSamlSpInfo());
        $this->assertEquals($expectedProviderKey, $authenticatedToken->getProviderKey());
        $this->assertEquals($expectedUserMock, $authenticatedToken->getUser());

        $roles = $authenticatedToken->getRoles();
        $this->assertInternalType('array', $roles);
        $this->assertCount(2, $roles);

        $this->assertEquals('foo', $roles[0]->getRole());
        $this->assertEquals('bar', $roles[1]->getRole());
    }


    /**
     * @test
     *
     * @expectedException \Symfony\Component\Security\Core\Exception\AuthenticationServiceException
     * @expectedExceptionMessage User provider did not return an implementation of user interface.
     */
    public function throwIfUserManagerCreateNotUserInstance()
    {
        $samlSpInfoHelper = new SamlSpInfoHelper();

        $providerKey = 'main';
        $expectedSamlSpInfo = $samlSpInfoHelper->getSamlSpInfo();

        $userManagerMock = $this->createUserManagerMock();
        $userManagerMock
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                ->expects($this->once())
                ->method('loadUserBySamlInfo')
                ->with($expectedSamlSpInfo)
                ->will($this->throwException(new UsernameNotFoundException('Cannot find user by saml sp info')))
        ;
        $userManagerMock
                ->expects($this->once())
                ->method('createUserFromSamlInfo')
                ->with($expectedSamlSpInfo)
                ->will($this->returnValue('not-a-user-instance'))
        ;

        $authProvider = new SamlSpAuthenticationProvider(
            $providerKey,
            $userManagerMock,
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
            $this->createUserCheckerMock(),

            $createIfNotExist = true
        );

        $token = new SamlSpToken($providerKey);
        $token->setUser('');
        $token->setSamlSpInfo($expectedSamlSpInfo);

        $authProvider->authenticate($token);
    }



    /**
     * @test
     */
    public function shouldWrapAnyThrownExceptionsAsAuthenticatedServiceException()
    {
        $samlSpInfoHelper = new SamlSpInfoHelper();
        $providerKey = 'main';
        $expectedSamlSpInfo = $samlSpInfoHelper->getSamlSpInfo();
        $expectedPreviousException = new \Exception(
            $expectedMessage = 'Something goes wrong',
            $expectedCode = 21
        );

        $userProviderMock = $this->createUserManagerMock();
        $userProviderMock
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                ->expects($this->once())
                ->method('loadUserBySamlInfo')
                ->will($this->throwException($expectedPreviousException))
        ;

        $authProvider = new SamlSpAuthenticationProvider(
            $providerKey,
            $userProviderMock,
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
            $this->createUserCheckerMock()

        );

        $token = new SamlSpToken($providerKey);
        $token->setUser('');
        $token->setSamlSpInfo($expectedSamlSpInfo);

        try {
            $authProvider->authenticate($token);
        } catch (AuthenticationServiceException $e) {
            $this->assertSame($expectedPreviousException, $e->getPrevious(), $e->getPrevious());
            $this->assertEquals($expectedMessage, $e->getMessage());
            $this->assertEquals($expectedCode, $e->getCode());
            $this->assertNull($e->getToken());

            return;
        }

        $this->fail('Expected exception: AuthenticationServiceException was not thrown');
    }



    /**
     * @return \PHPUnit_Framework_MockObject_MockObject|\AerialShip\SamlSPBundle\Security\Core\User\UserManagerInterface
     */
    protected function createUserManagerMock()
    {
        return $this->getMock('AerialShip\SamlSPBundle\Security\Core\User\UserManagerInterface');
    }

    /**
     * @return \PHPUnit_Framework_MockObject_MockObject|\Symfony\Component\Security\Core\User\UserCheckerInterface
     */
    protected function createUserCheckerMock()
    {
        return $this->getMock('Symfony\Component\Security\Core\User\UserCheckerInterface');
    }

    /**
     * @return \PHPUnit_Framework_MockObject_MockObject|\Symfony\Component\Security\Core\User\UserInterface
     */
    protected function createUserMock()
    {
        return $this->getMock('Symfony\Component\Security\Core\User\UserInterface');
    }
}


Push — master ( d5bbb9...ab4758 )

Provider/SamlSpAuthenticationProviderTest.php (31 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

1			<?php
2
3			namespace AerialShip\SamlSPBundle\Tests\Security\Core\Authentication\Provider;
4
5			use AerialShip\SamlSPBundle\Security\Core\Authentication\Provider\SamlSpAuthenticationProvider;
6			use AerialShip\SamlSPBundle\Security\Core\Authentication\Token\SamlSpToken;
7			use AerialShip\SamlSPBundle\Tests\Bridge\SamlSpInfoHelper;
8			use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
9			use Symfony\Component\Security\Core\Exception\AuthenticationServiceException;
10			use Symfony\Component\Security\Core\Exception\UsernameNotFoundException;
11
12			class SamlSpAuthenticationProviderTest extends \PHPUnit_Framework_TestCase
13			{
14			/**
15			* @test
16			*/
17			public function couldBeConstructedWithProviderKey()
18			{
19			new SamlSpAuthenticationProvider('key');
20			}
21
22
23			/**
24			* @test
25			*/
26			public function couldBeConstructedWithUserManagerAndUserChecker()
27			{
28			new SamlSpAuthenticationProvider(
29			'main',
30			$this->createUserManagerMock(),
			0 ignored issues – show Bug introduced 2016-01-03 08:18 UTC by Report Bug Copy Issue Report Show Similar Issues like this It seems like `$this->createUserManagerMock()` targeting `AerialShip\SamlSPBundle\...createUserManagerMock()` can also be of type `object<PHPUnit_Framework_MockObject_MockObject>`; however, `AerialShip\SamlSPBundle\...Provider::__construct()` does only seem to accept `null\|object<AerialShip\S...r\UserManagerInterface>`, maybe add an additional type check? This check looks at variables that are passed out again to other methods. If the outgoing method call has stricter type requirements than the method itself, an issue is raised. An additional type check may prevent trouble. Loading history...
31			$this->createUserCheckerMock()
			0 ignored issues – show Bug introduced 2016-01-03 08:18 UTC by Report Bug Copy Issue Report Show Similar Issues like this It seems like `$this->createUserCheckerMock()` targeting `AerialShip\SamlSPBundle\...createUserCheckerMock()` can also be of type `object<PHPUnit_Framework_MockObject_MockObject>`; however, `AerialShip\SamlSPBundle\...Provider::__construct()` does only seem to accept `null\|object<Symfony\Comp...r\UserCheckerInterface>`, maybe add an additional type check? This check looks at variables that are passed out again to other methods. If the outgoing method call has stricter type requirements than the method itself, an issue is raised. An additional type check may prevent trouble. Loading history...
32			);
33			}
34
35			/**
36			* @test
37			*/
38			public function couldBeConstructedWithUserManagerAndUserCheckerAndCreateUserIfNotExist()

aerialship / SamlSPBundle

Push — master ( d5bbb9...ab4758 )

Provider/SamlSpAuthenticationProviderTest.php (31 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like