ComposerFilesExtractorQueryHandlerTrait::shouldHandleComposerFilesExtractorQuery() - Code Metrics - bruli/php-git-hooks - Measure and Improve Code Quality continuously with Scrutinizer

shouldHandleComposerFilesExtractorQuery() A
last analyzed 2019-12-13 13:07 UTC

↳ Parent: ComposerFilesExtractorQueryHandlerTrait

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	11
rs	9.9
c	0
b	0
f	0
cc	1
nc	1
nop	2

<?php

namespace PhpGitHooks\Module\Files\Tests\Infrastructure;

use PhpGitHooks\Module\Files\Contract\Query\ComposerFilesExtractor;
use PhpGitHooks\Module\Files\Contract\Query\ComposerFilesExtractorHandler;
use PhpGitHooks\Module\Files\Contract\Response\ComposerFilesResponse;
use PhpGitHooks\Module\Tests\Infrastructure\UnitTestCase\Mock;
use PhpGitHooks\Module\Tests\Infrastructure\UnitTestCase\SimilarTo;

trait ComposerFilesExtractorQueryHandlerTrait
{
    /**
     * @var ComposerFilesExtractorHandler
     */
    private $composerFilesExtractorQueryHandler;

    /**
     * @return \Mockery\MockInterface|ComposerFilesExtractorHandler
     */
    protected function getComposerFilesExtractorQueryHandler()
    {
        return $this->composerFilesExtractorQueryHandler = $this->composerFilesExtractorQueryHandler ?: Mock::get(
            ComposerFilesExtractorHandler::class
        );
    }

    /**
     * @param ComposerFilesExtractor $composerFilesFilesExtractorQuery
     * @param ComposerFilesResponse       $return
     */
    protected function shouldHandleComposerFilesExtractorQuery(
        ComposerFilesExtractor $composerFilesFilesExtractorQuery,
        ComposerFilesResponse $return
    ) {
        $this->getComposerFilesExtractorQueryHandler()
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.
}
             ->shouldReceive('handle')
             ->once()
             ->with((new SimilarTo())->compare($composerFilesFilesExtractorQuery))
             ->andReturn($return)
        ;
    }
}


1			<?php
2
3			namespace PhpGitHooks\Module\Files\Tests\Infrastructure;
4
5			use PhpGitHooks\Module\Files\Contract\Query\ComposerFilesExtractor;
6			use PhpGitHooks\Module\Files\Contract\Query\ComposerFilesExtractorHandler;
7			use PhpGitHooks\Module\Files\Contract\Response\ComposerFilesResponse;
8			use PhpGitHooks\Module\Tests\Infrastructure\UnitTestCase\Mock;
9			use PhpGitHooks\Module\Tests\Infrastructure\UnitTestCase\SimilarTo;
10
11			trait ComposerFilesExtractorQueryHandlerTrait
12			{
13			/**
14			* @var ComposerFilesExtractorHandler
15			*/
16			private $composerFilesExtractorQueryHandler;
17
18			/**
19			* @return \Mockery\MockInterface\|ComposerFilesExtractorHandler
20			*/
21			protected function getComposerFilesExtractorQueryHandler()
22			{
23			return $this->composerFilesExtractorQueryHandler = $this->composerFilesExtractorQueryHandler ?: Mock::get(
24			ComposerFilesExtractorHandler::class
25			);
26			}
27
28			/**
29			* @param ComposerFilesExtractor $composerFilesFilesExtractorQuery
30			* @param ComposerFilesResponse $return
31			*/
32			protected function shouldHandleComposerFilesExtractorQuery(
33			ComposerFilesExtractor $composerFilesFilesExtractorQuery,
34			ComposerFilesResponse $return
35			) {
36			$this->getComposerFilesExtractorQueryHandler()
			0 ignored issues – show Bug introduced 2017-06-17 16:47 UTC by Report Bug Copy Issue Report The method `shouldReceive` does only exist in `Mockery\MockInterface`, but not in `PhpGitHooks\Module\Files...erFilesExtractorHandler`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: 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. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
37			->shouldReceive('handle')
38			->once()
39			->with((new SimilarTo())->compare($composerFilesFilesExtractorQuery))
40			->andReturn($return)
41			;
42			}
43			}
44

bruli / php-git-hooks

shouldHandleComposerFilesExtractorQuery() A last analyzed 2019-12-13 13:07 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like

shouldHandleComposerFilesExtractorQuery() A
last analyzed 2019-12-13 13:07 UTC