PrePushToolHandlerTest::itShouldNotExecuteHook() - Code Metrics - Inspection of "Remove pre push tool service and rename command an..." - bruli/php-git-hooks - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( efc44e...835840 )

by Pablo

created 2017-06-17 19:36 UTC

PrePushToolHandlerTest::itShouldNotExecuteHook() A

↳ Parent: PrePushToolHandlerTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	9
rs	9.6666
cc	1
eloc	5
nc	1
nop	0

<?php

namespace PhpGitHooks\Module\Git\Tests\Behaviour;

use PhpGitHooks\Module\Configuration\Contract\Query\ConfigurationDataFinder;
use PhpGitHooks\Module\Configuration\Tests\Stub\ConfigurationDataResponseStub;
use PhpGitHooks\Module\Git\Contract\Command\PrePushTool;
use PhpGitHooks\Module\Git\Contract\Command\PrePushToolHandler;
use PhpGitHooks\Module\Git\Contract\Exception\InvalidPushException;
use PhpGitHooks\Module\Git\Contract\Response\BadJobLogoResponse;
use PhpGitHooks\Module\Git\Contract\Response\GoodJobLogoResponse;
use PhpGitHooks\Module\Git\Tests\Infrastructure\GitUnitTestCase;
use PhpGitHooks\Module\PhpUnit\Contract\Command\GuardCoverageCommand;
use PhpGitHooks\Module\PhpUnit\Contract\Command\PhpUnitToolCommand;
use PhpGitHooks\Module\PhpUnit\Contract\Command\StrictCoverageCommand;

class PrePushToolHandlerTest extends GitUnitTestCase
{
    private $remote = 'origin';
    private $url = '[email protected]';
    /**
     * @var PrePushToolHandler
     */
    private $prePushToolCommandHandler;

    protected function setUp()
    {
        $this->prePushToolCommandHandler = new PrePushToolHandler(
            $this->getQueryBus(),

            $this->getPrePushOriginalExecutor(),

            $this->getOutputInterface(),
            $this->getCommandBus()

        );
    }

    /**
     * @test
     */
    public function itShouldNotExecuteHook()
    {
        $this->shouldHandleQuery(
            new ConfigurationDataFinder(),
            ConfigurationDataResponseStub::createCustom(false, false, false)
        );

        $this->prePushToolCommandHandler->handle(new PrePushTool($this->remote, $this->url));
    }

    /**
     * @test
     */
    public function itShouldThrowsExceptionFromOriginalScript()
    {
        $this->expectException(InvalidPushException::class);

        $configurationDataResponse = ConfigurationDataResponseStub::createCustom(false, false, true);

        $this->shouldHandleQuery(
            new ConfigurationDataFinder(),
            $configurationDataResponse
        );
        $this->shouldWriteLnOutput(PrePushToolHandler::PRE_PUSH_HOOK);
        $this->shouldExecutePrePushOriginal($this->remote, $this->url, 'error');
        $this->shouldWriteLnOutput(
            BadJobLogoResponse::paint($configurationDataResponse->getPrePush()->getErrorMessage())
        );

        $this->prePushToolCommandHandler->handle(new PrePushTool($this->remote, $this->url));
    }

    /**
     * @test
     */
    public function itShouldWorksFine()
    {
        $configurationDataResponse = ConfigurationDataResponseStub::createCustom(false, false, true);

        $this->shouldHandleQuery(
            new ConfigurationDataFinder(),
            $configurationDataResponse
        );
        $this->shouldWriteLnOutput(PrePushToolHandler::PRE_PUSH_HOOK);
        $this->shouldExecutePrePushOriginal($this->remote, $this->url, '');
        $this->shouldHandleCommand(
            new PhpUnitToolCommand(
                $configurationDataResponse->getPrePush()->getPhpUnit()->isPhpunitRandomMode(),
                $configurationDataResponse->getPrePush()->getPhpUnit()->getPhpunitOptions(),
                $configurationDataResponse->getPrePush()->getErrorMessage()
            )
        );
        $this->shouldHandleCommand(
            new StrictCoverageCommand(
                $configurationDataResponse->getPrePush()->getPhpUnitStrictCoverage()->getMinimum(),
                $configurationDataResponse->getPrePush()->getErrorMessage()
            )
        );

        $this->shouldHandleCommand(
            new GuardCoverageCommand(
                $configurationDataResponse->getPreCommit()->getPhpUnitGuardCoverage()->getWarningMessage()
            )
        );

        $this->shouldWriteLnOutput(
            GoodJobLogoResponse::paint($configurationDataResponse->getPrePush()->getRightMessage())
        );

        $this->prePushToolCommandHandler->handle(new PrePushTool($this->remote, $this->url));
    }
}


1			<?php
2
3			namespace PhpGitHooks\Module\Git\Tests\Behaviour;
4
5			use PhpGitHooks\Module\Configuration\Contract\Query\ConfigurationDataFinder;
6			use PhpGitHooks\Module\Configuration\Tests\Stub\ConfigurationDataResponseStub;
7			use PhpGitHooks\Module\Git\Contract\Command\PrePushTool;
8			use PhpGitHooks\Module\Git\Contract\Command\PrePushToolHandler;
9			use PhpGitHooks\Module\Git\Contract\Exception\InvalidPushException;
10			use PhpGitHooks\Module\Git\Contract\Response\BadJobLogoResponse;
11			use PhpGitHooks\Module\Git\Contract\Response\GoodJobLogoResponse;
12			use PhpGitHooks\Module\Git\Tests\Infrastructure\GitUnitTestCase;
13			use PhpGitHooks\Module\PhpUnit\Contract\Command\GuardCoverageCommand;
14			use PhpGitHooks\Module\PhpUnit\Contract\Command\PhpUnitToolCommand;
15			use PhpGitHooks\Module\PhpUnit\Contract\Command\StrictCoverageCommand;
16
17			class PrePushToolHandlerTest extends GitUnitTestCase
18			{
19			private $remote = 'origin';
20			private $url = '[email protected]';
21			/**
22			* @var PrePushToolHandler
23			*/
24			private $prePushToolCommandHandler;
25
26			protected function setUp()
27			{
28			$this->prePushToolCommandHandler = new PrePushToolHandler(
29			$this->getQueryBus(),
			0 ignored issues – show Bug introduced 2017-06-17 19:39 UTC by Report Bug Copy Issue Report It seems like `$this->getQueryBus()` targeting `PhpGitHooks\Module\Share...BusTrait::getQueryBus()` can also be of type `object<Mockery\MockInterface>`; however, `PhpGitHooks\Module\Git\C...lHandler::__construct()` does only seem to accept `object<Bruli\EventBusBundle\QueryBus\QueryBus>`, 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...
30			$this->getPrePushOriginalExecutor(),
			0 ignored issues – show Bug introduced 2017-06-17 19:39 UTC by Report Bug Copy Issue Report It seems like `$this->getPrePushOriginalExecutor()` targeting `PhpGitHooks\Module\Git\T...ePushOriginalExecutor()` can also be of type `object<Mockery\MockInterface>`; however, `PhpGitHooks\Module\Git\C...lHandler::__construct()` does only seem to accept `object<PhpGitHooks\Modul...ginalExecutorInterface>`, 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->getOutputInterface(),
32			$this->getCommandBus()
			0 ignored issues – show Bug introduced 2017-06-17 19:39 UTC by Report Bug Copy Issue Report It seems like `$this->getCommandBus()` targeting `PhpGitHooks\Module\Share...sTrait::getCommandBus()` can also be of type `object<Mockery\MockInterface>`; however, `PhpGitHooks\Module\Git\C...lHandler::__construct()` does only seem to accept `object<Bruli\EventBusBun...\CommandBus\CommandBus>`, 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...
33			);
34			}
35
36			/**
37			* @test
38			*/
39			public function itShouldNotExecuteHook()
40			{
41			$this->shouldHandleQuery(
42			new ConfigurationDataFinder(),
43			ConfigurationDataResponseStub::createCustom(false, false, false)
44			);
45
46			$this->prePushToolCommandHandler->handle(new PrePushTool($this->remote, $this->url));
47			}
48
49			/**
50			* @test
51			*/
52			public function itShouldThrowsExceptionFromOriginalScript()
53			{
54			$this->expectException(InvalidPushException::class);
55
56			$configurationDataResponse = ConfigurationDataResponseStub::createCustom(false, false, true);
57
58			$this->shouldHandleQuery(
59			new ConfigurationDataFinder(),
60			$configurationDataResponse
61			);
62			$this->shouldWriteLnOutput(PrePushToolHandler::PRE_PUSH_HOOK);
63			$this->shouldExecutePrePushOriginal($this->remote, $this->url, 'error');
64			$this->shouldWriteLnOutput(
65			BadJobLogoResponse::paint($configurationDataResponse->getPrePush()->getErrorMessage())
66			);
67
68			$this->prePushToolCommandHandler->handle(new PrePushTool($this->remote, $this->url));
69			}
70
71			/**
72			* @test
73			*/
74			public function itShouldWorksFine()
75			{
76			$configurationDataResponse = ConfigurationDataResponseStub::createCustom(false, false, true);
77
78			$this->shouldHandleQuery(
79			new ConfigurationDataFinder(),
80			$configurationDataResponse
81			);
82			$this->shouldWriteLnOutput(PrePushToolHandler::PRE_PUSH_HOOK);
83			$this->shouldExecutePrePushOriginal($this->remote, $this->url, '');
84			$this->shouldHandleCommand(
85			new PhpUnitToolCommand(
86			$configurationDataResponse->getPrePush()->getPhpUnit()->isPhpunitRandomMode(),
87			$configurationDataResponse->getPrePush()->getPhpUnit()->getPhpunitOptions(),
88			$configurationDataResponse->getPrePush()->getErrorMessage()
89			)
90			);
91			$this->shouldHandleCommand(
92			new StrictCoverageCommand(
93			$configurationDataResponse->getPrePush()->getPhpUnitStrictCoverage()->getMinimum(),
94			$configurationDataResponse->getPrePush()->getErrorMessage()
95			)
96			);
97
98			$this->shouldHandleCommand(
99			new GuardCoverageCommand(
100			$configurationDataResponse->getPreCommit()->getPhpUnitGuardCoverage()->getWarningMessage()
101			)
102			);
103
104			$this->shouldWriteLnOutput(
105			GoodJobLogoResponse::paint($configurationDataResponse->getPrePush()->getRightMessage())
106			);
107
108			$this->prePushToolCommandHandler->handle(new PrePushTool($this->remote, $this->url));
109			}
110			}
111

bruli / php-git-hooks

Push — master ( efc44e...835840 )

PrePushToolHandlerTest::itShouldNotExecuteHook() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like