SecurityAuthenticatorTest - Code Metrics - Inspection of "evo-2472 and evo-2476 security authentication" - libgraviton/graviton - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#353)

by Jan

created 2016-01-27 16:16 UTC

SecurityAuthenticatorTest A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	236
Duplicated Lines	4.24 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	9
c	1
b	0
f	0
lcom	1
cbo	8
dl	10
loc	236
rs	10

9 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	7	1
B	testCreateToken()	0	33	1
A	stringProvider()	10	10	1
B	testAuthenticateToken()	0	43	1
B	testAuthenticateTokenExpectingException()	0	31	1
A	testSupportsToken()	0	22	1
B	testOnAuthenticationFailure()	0	25	1
A	getProviderMock()	0	9	1
A	getStrategyMock()	0	6	1

How to fix Duplicated Code

<?php
/**
 * main checks for airlock authenticator
 */

namespace Graviton\SecurityBundle\Authentication;

use Graviton\SecurityBundle\Authentication\Strategies\StrategyInterface;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\Security\Core\Authentication\Token\PreAuthenticatedToken;

/**
 * Class AirlockAuthenticationKeyAuthenticatorTest
 *
 * @author   List of contributors <https://github.com/libgraviton/graviton/graphs/contributors>
 * @license  http://opensource.org/licenses/gpl-license.php GNU Public License
 * @link     http://swisscom.ch
 */
class SecurityAuthenticatorTest extends \PHPUnit_Framework_TestCase
{
    /** @var \Psr\Log\LoggerInterface|\PHPUnit_Framework_MockObject_MockObject logger */
    private $logger;

    /**
     * @return void
     */
    protected function setUp()
    {
        /** @var \Psr\Log\LoggerInterface|\PHPUnit_Framework_MockObject_MockObject logger */
        $this->logger = $this->getMockBuilder('\Psr\Log\LoggerInterface')
            ->setMethods(array('warning', 'info'))
            ->getMockForAbstractClass();
    }


    /**
     * @dataProvider stringProvider
     *
     * @param string $headerFieldValue value to check with
     *
     * @return void
     */
    public function testCreateToken($headerFieldValue)
    {
        $userProviderMock = $this
            ->getMockBuilder('Graviton\SecurityBundle\Authentication\Provider\AuthenticationProvider')
            ->disableOriginalConstructor()
            ->setMethods(array('loadUserByUsername'))
            ->getMock();

        $strategy = $this->getMockBuilder('\Graviton\SecurityBundle\Authentication\Strategies\StrategyInterface')
            ->setMethods(array('apply'))
            ->getMockForAbstractClass();
        $strategy
            ->expects($this->once())
            ->method('apply')
            ->will($this->returnValue($headerFieldValue));

        $authenticator = new SecurityAuthenticator(true, true, true, $userProviderMock, $strategy, $this->logger);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $server = array(
            'HTTP_X_IDP_USERNAME' => $headerFieldValue, //"example-authentication-header",
        );

        $request = new Request(array(), array(), array(), array(), array(), $server);

        $token = $authenticator->createToken($request, 'AirlockProviderKey');

        $this->assertInstanceOf(
            '\Symfony\Component\Security\Core\Authentication\Token\PreAuthenticatedToken',
            $token
        );

        $this->assertFalse($token->isAuthenticated());
    }

    /**
     * @return array<string>
     */
    public function stringProvider()

    {
        return array(
            'plain string, no special chars' => array('exampleAuthenticationHeader'),
            'string with special chars' => array("$-_.+!*'(),{}|\\^~[]`<>#%;/?:@&=."),
            'string with octal chars' => array("a: \141, A: \101"),
            'string with hex chars' => array("a: \x61, A: \x41"),
            'live example' => array("10N0000188"),
        );
    }

    /**
     * @return void
     */
    public function testAuthenticateToken()
    {
        $providerKey = 'some providerKey';
        $apiKey = 'exampleAuthenticationHeader';

        $securityUserMock =  $this->getMockBuilder('Symfony\Component\Security\Core\User\UserInterface')
            ->setMethods(array('getRoles'))
            ->getMockForAbstractClass();
        $securityUserMock
            ->expects($this->never())
            ->method('getRoles')
            ->will($this->returnValue(array('ROLE_GRAVITON_USER')));

        $userProviderMock = $this->getProviderMock(array('loadUserByUsername'));
        $userProviderMock
            ->expects($this->once())
            ->method('loadUserByUsername')
            ->will($this->returnValue($securityUserMock));

        $anonymousToken = new PreAuthenticatedToken(
            'anon.',
            $apiKey,
            $providerKey
        );

        $authenticator = new SecurityAuthenticator(
            true,
            true,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            true,
            $userProviderMock,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->getStrategyMock(),
            $this->logger
        );

        $token = $authenticator->authenticateToken($anonymousToken, $userProviderMock, $providerKey);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $this->assertInstanceOf(
            '\Symfony\Component\Security\Core\Authentication\Token\PreAuthenticatedToken',
            $token
        );

        $this->assertTrue($token->isAuthenticated());
    }

    /**
     * @return void
     */
    public function testAuthenticateTokenExpectingException()
    {
        $providerKey = 'some providerKey';
        $apiKey = 'exampleAuthenticationHeader';

        $userProviderMock = $this->getProviderMock(array('loadUserByUsername'));
        $userProviderMock
            ->expects($this->once())
            ->method('loadUserByUsername')
            ->with($this->equalTo($apiKey))
            ->will($this->returnValue(false));

        $anonymousToken = new PreAuthenticatedToken(
            'anon.',
            $apiKey,
            $providerKey
        );

        $authenticator = new SecurityAuthenticator(
            true,
            false,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            false,
            $userProviderMock,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->getStrategyMock(),
            $this->logger
        );

        $this->setExpectedException('\Symfony\Component\Security\Core\Exception\AuthenticationException');

        $authenticator->authenticateToken($anonymousToken, $userProviderMock, $providerKey);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @return void
     */
    public function testSupportsToken()
    {
        $providerKey = 'some providerKey';
        $apiKey = 'exampleAuthenticationHeader';

        $anonymousToken = new PreAuthenticatedToken(
            'anon.',
            $apiKey,
            $providerKey
        );

        $authenticator = new SecurityAuthenticator(
            true,
            true,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            true,
            $this->getProviderMock(),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->getStrategyMock(),
            $this->logger
        );

        $this->assertTrue($authenticator->supportsToken($anonymousToken, $providerKey));
    }

    /**
     * @return void
     */
    public function testOnAuthenticationFailure()
    {
        $exceptionDouble = $this->getMockBuilder('\Symfony\Component\Security\Core\Exception\AuthenticationException')
            ->disableOriginalConstructor()
            ->setMethods(array('getMessageKey'))
            ->getMock();
        $exceptionDouble
            ->expects($this->once())
            ->method('getMessageKey')
            ->will($this->returnValue('test_message'));

        $authenticator = new SecurityAuthenticator(
            true,
            true,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            true,
            $this->getProviderMock(),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->getStrategyMock(),
            $this->logger
        );

        $response = $authenticator->onAuthenticationFailure(new Request(), $exceptionDouble);

        $this->assertEquals('test_message', $response->getContent());
        $this->assertEquals(511, $response->getStatusCode());
    }

    /**
     * @param string[] $methods methods to mock
     *
     * @return \PHPUnit_Framework_MockObject_MockObject|SecurityAuthenticator
     */
    private function getProviderMock(array $methods = array())
    {
        $userProviderMock = $this
            ->getMockBuilder('Graviton\SecurityBundle\Authentication\Provider\AuthenticationProvider')
            ->disableOriginalConstructor()
            ->setMethods($methods)
            ->getMock();
        return $userProviderMock;
    }

    /**
     * @param array $methods methods to mock
     *
     * @return StrategyInterface|\PHPUnit_Framework_MockObject_MockObject
     */
    private function getStrategyMock(array $methods = array('apply'))
    {
        return $this->getMockBuilder('\Graviton\SecurityBundle\Authentication\Strategies\StrategyInterface')
            ->setMethods($methods)
            ->getMockForAbstractClass();
    }
}


1		<?php
2		/**
3		* main checks for airlock authenticator
4		*/
5
6		namespace Graviton\SecurityBundle\Authentication;
7
8		use Graviton\SecurityBundle\Authentication\Strategies\StrategyInterface;
9		use Symfony\Component\HttpFoundation\Request;
10		use Symfony\Component\Security\Core\Authentication\Token\PreAuthenticatedToken;
11
12		/**
13		* Class AirlockAuthenticationKeyAuthenticatorTest
14		*
15		* @author List of contributors <https://github.com/libgraviton/graviton/graphs/contributors>
16		* @license http://opensource.org/licenses/gpl-license.php GNU Public License
17		* @link http://swisscom.ch
18		*/
19		class SecurityAuthenticatorTest extends \PHPUnit_Framework_TestCase
20		{
21		/** @var \Psr\Log\LoggerInterface\|\PHPUnit_Framework_MockObject_MockObject logger */
22		private $logger;
23
24		/**
25		* @return void
26		*/
27		protected function setUp()
28		{
29		/** @var \Psr\Log\LoggerInterface\|\PHPUnit_Framework_MockObject_MockObject logger */
30		$this->logger = $this->getMockBuilder('\Psr\Log\LoggerInterface')
31		->setMethods(array('warning', 'info'))
32		->getMockForAbstractClass();
33		}
34
35
36		/**
37		* @dataProvider stringProvider
38		*
39		* @param string $headerFieldValue value to check with
40		*
41		* @return void
42		*/
43		public function testCreateToken($headerFieldValue)
44		{
45		$userProviderMock = $this
46		->getMockBuilder('Graviton\SecurityBundle\Authentication\Provider\AuthenticationProvider')
47		->disableOriginalConstructor()
48		->setMethods(array('loadUserByUsername'))
49		->getMock();
50
51		$strategy = $this->getMockBuilder('\Graviton\SecurityBundle\Authentication\Strategies\StrategyInterface')
52		->setMethods(array('apply'))
53		->getMockForAbstractClass();
54		$strategy
55		->expects($this->once())
56		->method('apply')
57		->will($this->returnValue($headerFieldValue));
58
59		$authenticator = new SecurityAuthenticator(true, true, true, $userProviderMock, $strategy, $this->logger);
		0 ignored issues – show Documentation introduced 2016-01-26 15:51 UTC by Report Bug Copy Issue Report `true` is of type `boolean`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
60
61		$server = array(
62		'HTTP_X_IDP_USERNAME' => $headerFieldValue, //"example-authentication-header",
63		);
64
65		$request = new Request(array(), array(), array(), array(), array(), $server);
66
67		$token = $authenticator->createToken($request, 'AirlockProviderKey');
68
69		$this->assertInstanceOf(
70		'\Symfony\Component\Security\Core\Authentication\Token\PreAuthenticatedToken',
71		$token
72		);
73
74		$this->assertFalse($token->isAuthenticated());
75		}
76
77		/**
78		* @return array<string>
79		*/
80	View Code Duplication	public function stringProvider()
		0 ignored issues – show Duplication introduced 2016-01-26 09:25 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
81		{
82		return array(
83		'plain string, no special chars' => array('exampleAuthenticationHeader'),
84		'string with special chars' => array("$-_.+!*'(),{}\|\\^~[]`<>#%;/?:@&=."),
85		'string with octal chars' => array("a: \141, A: \101"),
86		'string with hex chars' => array("a: \x61, A: \x41"),
87		'live example' => array("10N0000188"),
88		);
89		}
90
91		/**
92		* @return void
93		*/
94		public function testAuthenticateToken()
95		{
96		$providerKey = 'some providerKey';
97		$apiKey = 'exampleAuthenticationHeader';
98
99		$securityUserMock = $this->getMockBuilder('Symfony\Component\Security\Core\User\UserInterface')
100		->setMethods(array('getRoles'))
101		->getMockForAbstractClass();
102		$securityUserMock
103		->expects($this->never())
104		->method('getRoles')
105		->will($this->returnValue(array('ROLE_GRAVITON_USER')));
106
107		$userProviderMock = $this->getProviderMock(array('loadUserByUsername'));
108		$userProviderMock
109		->expects($this->once())
110		->method('loadUserByUsername')
111		->will($this->returnValue($securityUserMock));
112
113		$anonymousToken = new PreAuthenticatedToken(
114		'anon.',
115		$apiKey,
116		$providerKey
117		);
118
119		$authenticator = new SecurityAuthenticator(
120		true,
121		true,
		0 ignored issues – show Documentation introduced 2016-01-27 11:04 UTC by Report Bug Copy Issue Report `true` is of type `boolean`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
122		true,
123		$userProviderMock,
		0 ignored issues – show Documentation introduced 2016-01-27 11:04 UTC by Report Bug Copy Issue Report `$userProviderMock` is of type `object<PHPUnit_Framework...\SecurityAuthenticator>`, but the function expects a `object<Graviton\Security...AuthenticationProvider>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
124		$this->getStrategyMock(),
125		$this->logger
126		);
127
128		$token = $authenticator->authenticateToken($anonymousToken, $userProviderMock, $providerKey);
		0 ignored issues – show Documentation introduced 2016-01-26 09:25 UTC by Report Bug Copy Issue Report `$userProviderMock` is of type `object<PHPUnit_Framework...\SecurityAuthenticator>`, but the function expects a `object<Symfony\Component...\UserProviderInterface>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
129
130		$this->assertInstanceOf(
131		'\Symfony\Component\Security\Core\Authentication\Token\PreAuthenticatedToken',
132		$token
133		);
134
135		$this->assertTrue($token->isAuthenticated());
136		}
137
138		/**
139		* @return void
140		*/
141		public function testAuthenticateTokenExpectingException()
142		{
143		$providerKey = 'some providerKey';
144		$apiKey = 'exampleAuthenticationHeader';
145
146		$userProviderMock = $this->getProviderMock(array('loadUserByUsername'));
147		$userProviderMock
148		->expects($this->once())
149		->method('loadUserByUsername')
150		->with($this->equalTo($apiKey))
151		->will($this->returnValue(false));
152
153		$anonymousToken = new PreAuthenticatedToken(
154		'anon.',
155		$apiKey,
156		$providerKey
157		);
158
159		$authenticator = new SecurityAuthenticator(
160		true,
161		false,
		0 ignored issues – show Documentation introduced 2016-01-27 11:04 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
162		false,
163		$userProviderMock,
		0 ignored issues – show Documentation introduced 2016-01-27 11:04 UTC by Report Bug Copy Issue Report `$userProviderMock` is of type `object<PHPUnit_Framework...\SecurityAuthenticator>`, but the function expects a `object<Graviton\Security...AuthenticationProvider>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
164		$this->getStrategyMock(),
165		$this->logger
166		);
167
168		$this->setExpectedException('\Symfony\Component\Security\Core\Exception\AuthenticationException');
169
170		$authenticator->authenticateToken($anonymousToken, $userProviderMock, $providerKey);
		0 ignored issues – show Documentation introduced 2016-01-26 09:25 UTC by Report Bug Copy Issue Report `$userProviderMock` is of type `object<PHPUnit_Framework...\SecurityAuthenticator>`, but the function expects a `object<Symfony\Component...\UserProviderInterface>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
171		}
172
173		/**
174		* @return void
175		*/
176		public function testSupportsToken()
177		{
178		$providerKey = 'some providerKey';
179		$apiKey = 'exampleAuthenticationHeader';
180
181		$anonymousToken = new PreAuthenticatedToken(
182		'anon.',
183		$apiKey,
184		$providerKey
185		);
186
187		$authenticator = new SecurityAuthenticator(
188		true,
189		true,
		0 ignored issues – show Documentation introduced 2016-01-27 11:04 UTC by Report Bug Copy Issue Report `true` is of type `boolean`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
190		true,
191		$this->getProviderMock(),
		0 ignored issues – show Documentation introduced 2016-01-27 11:04 UTC by Report Bug Copy Issue Report `$this->getProviderMock()` is of type `object<PHPUnit_Framework...\SecurityAuthenticator>`, but the function expects a `object<Graviton\Security...AuthenticationProvider>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
192		$this->getStrategyMock(),
193		$this->logger
194		);
195
196		$this->assertTrue($authenticator->supportsToken($anonymousToken, $providerKey));
197		}
198
199		/**
200		* @return void
201		*/
202		public function testOnAuthenticationFailure()
203		{
204		$exceptionDouble = $this->getMockBuilder('\Symfony\Component\Security\Core\Exception\AuthenticationException')
205		->disableOriginalConstructor()
206		->setMethods(array('getMessageKey'))
207		->getMock();
208		$exceptionDouble
209		->expects($this->once())
210		->method('getMessageKey')
211		->will($this->returnValue('test_message'));
212
213		$authenticator = new SecurityAuthenticator(
214		true,
215		true,
		0 ignored issues – show Documentation introduced 2016-01-27 11:04 UTC by Report Bug Copy Issue Report `true` is of type `boolean`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
216		true,
217		$this->getProviderMock(),
		0 ignored issues – show Documentation introduced 2016-01-27 11:04 UTC by Report Bug Copy Issue Report `$this->getProviderMock()` is of type `object<PHPUnit_Framework...\SecurityAuthenticator>`, but the function expects a `object<Graviton\Security...AuthenticationProvider>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
218		$this->getStrategyMock(),
219		$this->logger
220		);
221
222		$response = $authenticator->onAuthenticationFailure(new Request(), $exceptionDouble);
223
224		$this->assertEquals('test_message', $response->getContent());
225		$this->assertEquals(511, $response->getStatusCode());
226		}
227
228		/**
229		* @param string[] $methods methods to mock
230		*
231		* @return \PHPUnit_Framework_MockObject_MockObject\|SecurityAuthenticator
232		*/
233		private function getProviderMock(array $methods = array())
234		{
235		$userProviderMock = $this
236		->getMockBuilder('Graviton\SecurityBundle\Authentication\Provider\AuthenticationProvider')
237		->disableOriginalConstructor()
238		->setMethods($methods)
239		->getMock();
240		return $userProviderMock;
241		}
242
243		/**
244		* @param array $methods methods to mock
245		*
246		* @return StrategyInterface\|\PHPUnit_Framework_MockObject_MockObject
247		*/
248		private function getStrategyMock(array $methods = array('apply'))
249		{
250		return $this->getMockBuilder('\Graviton\SecurityBundle\Authentication\Strategies\StrategyInterface')
251		->setMethods($methods)
252		->getMockForAbstractClass();
253		}
254		}
255

libgraviton / graviton

Pull Request — develop (#353)

SecurityAuthenticatorTest A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

9 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like