SyncTerminalsCommandTest::testExecute() - Code Metrics - loevgaard/dandomain-altapay-bundle - Measure and Improve Code Quality continuously with Scrutinizer

SyncTerminalsCommandTest::testExecute() A
last analyzed 2017-12-13 14:39 UTC

↳ Parent: SyncTerminalsCommandTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	20
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	20
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	13
nc	1
nop	0

<?php

namespace Loevgaard\DandomainAltapayBundle\Tests\Command;

use Loevgaard\AltaPay\Client\Client;
use Loevgaard\DandomainAltapayBundle\Command\SyncTerminalsCommand;
use Loevgaard\DandomainAltapayBundle\Entity\TerminalRepository;
use Loevgaard\DandomainAltapayBundle\Synchronizer\TerminalSynchronizer;
use PHPUnit\Framework\TestCase;
use Symfony\Component\Console\Application;
use Symfony\Component\Console\Tester\CommandTester;

class SyncTerminalsCommandTest extends TestCase
{
    public function testExecute()
    {
        $terminalRepository = $this->createMock(TerminalRepository::class);
        $altapayClient = $this->createMock(Client::class);
        $terminalSynchronizer = new TerminalSynchronizer($terminalRepository, $altapayClient);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $command = new SyncTerminalsCommand($terminalSynchronizer);

        $application = new Application();
        $application->setAutoExit(false);
        $application->add($command);

        $command = $application->find('loevgaard:dandomain:altapay:sync-terminals');
        $commandTester = new CommandTester($command);
        $exitCode = $commandTester->execute([
            'command' => $command->getName(),
        ]);

        $this->assertSame(0, $exitCode, 'Returns 0 in case of success');
    }
}


1			<?php
2
3			namespace Loevgaard\DandomainAltapayBundle\Tests\Command;
4
5			use Loevgaard\AltaPay\Client\Client;
6			use Loevgaard\DandomainAltapayBundle\Command\SyncTerminalsCommand;
7			use Loevgaard\DandomainAltapayBundle\Entity\TerminalRepository;
8			use Loevgaard\DandomainAltapayBundle\Synchronizer\TerminalSynchronizer;
9			use PHPUnit\Framework\TestCase;
10			use Symfony\Component\Console\Application;
11			use Symfony\Component\Console\Tester\CommandTester;
12
13			class SyncTerminalsCommandTest extends TestCase
14			{
15			public function testExecute()
16			{
17			$terminalRepository = $this->createMock(TerminalRepository::class);
18			$altapayClient = $this->createMock(Client::class);
19			$terminalSynchronizer = new TerminalSynchronizer($terminalRepository, $altapayClient);
			0 ignored issues – show Documentation introduced 2017-12-06 07:55 UTC by Report Bug Copy Issue Report `$terminalRepository` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<Loevgaard\Dandoma...ity\TerminalRepository>`. 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... Documentation introduced 2017-12-06 12:25 UTC by Report Bug Copy Issue Report `$altapayClient` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<Loevgaard\AltaPay\Client\Client>`. 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...
20
21			$command = new SyncTerminalsCommand($terminalSynchronizer);
22
23			$application = new Application();
24			$application->setAutoExit(false);
25			$application->add($command);
26
27			$command = $application->find('loevgaard:dandomain:altapay:sync-terminals');
28			$commandTester = new CommandTester($command);
29			$exitCode = $commandTester->execute([
30			'command' => $command->getName(),
31			]);
32
33			$this->assertSame(0, $exitCode, 'Returns 0 in case of success');
34			}
35			}
36

loevgaard / dandomain-altapay-bundle

SyncTerminalsCommandTest::testExecute() A last analyzed 2017-12-13 14:39 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

SyncTerminalsCommandTest::testExecute() A
last analyzed 2017-12-13 14:39 UTC