Psr7UriExtensionTest::testGenerateAbsoluteUrl() - Code Metrics - Inspection of "Update travis config: remove tests for PHP 5.4" - AlexMasterov/twig-extension - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — master (294c1a)

by Alex

created 2016-04-17 05:28 UTC

Psr7UriExtensionTest::testGenerateAbsoluteUrl() A

↳ Parent: Psr7UriExtensionTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	12
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	12
rs	9.4285
cc	1
eloc	8
nc	1
nop	6

<?php

namespace Asmaster\TwigExtension\tests;

use PHPUnit_Framework_TestCase;
use Psr\Http\Message\UriInterface;
use Psr\Http\Message\ServerRequestInterface;
use Twig_ExtensionInterface;
use Asmaster\TwigExtension\Psr7UriExtension;

class Psr7UriExtensionTest extends PHPUnit_Framework_TestCase
{
    /**
     * @var ServerRequestInterface|\PHPUnit_Framework_MockObject_MockObject
     */
    protected $request;

    public function setUp()
    {
        $this->request = $this->getMockBuilder(ServerRequestInterface::class)
            ->disableOriginalConstructor()
            ->getMock();
    }

    public function testExtension()
    {
        $extension = new Psr7UriExtension($this->request);

        $this->assertSame('psr7_uri', $extension->getName());
    }

    /**
     * @dataProvider getFunctions()
     */
    public function testExtensionFunctions($function)
    {
        $extension = new Psr7UriExtension($this->request);

        $functions = array_map(
            function ($function) {
                return $function->getName();
            },
            $extension->getFunctions()
        );

        $this->assertContains($function, $functions);
    }

    public function getFunctions()
    {
        return [[
            'absolute_url',
            'relative_url'
       ]];
    }

    /**
     * @dataProvider getGenerateAbsoluteUrlData()
     */
    public function testGenerateAbsoluteUrl($path, $expected, $scheme, $host, $port, $basePath)
    {
        $request = $this->request
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('getUri')
            ->willReturn($this->createUri($scheme, $host, $port, $basePath));

        $extension = new Psr7UriExtension($this->request);
        $absoluteUrl = $extension->generateAbsoluteUrl($path);

        $this->assertEquals($expected, $absoluteUrl);
    }

    public function getGenerateAbsoluteUrlData()
    {
        return [
            ['/foo.png', '/foo.png', 'http', '', null, '/foo.png'],
            ['/foo.png', 'http://localhost/foo.png', 'http', 'localhost', null, '/'],
            ['/foo.png', 'http://localhost/foo.png', 'http', 'localhost', null, '/foo.png'],
            ['foo.png', 'http://localhost/foo/foo.png', 'http', 'localhost', null, '/foo'],
            ['foo.png', 'http://localhost/bar/foo.png', 'http', 'localhost', null, '/bar'],
            ['foo.png', 'http://localhost/foo/bar/foo.png', 'http', 'localhost', null, '/foo/bar'],
            ['/foo.png', 'http://localhost:8080/foo.png', 'http', 'localhost', 8080, '/'],
            ['/foo.png', 'https://localhost/foo.png', 'https', 'localhost', null, '/'],
            ['/', 'http://localhost/', 'http', 'localhost', null, '/'],
            ['//', '//', 'http', 'localhost', null, '/']
        ];
    }

    /**
     * @dataProvider getGenerateRelativeUrlData()
     */
    public function testGenerateRelativeUrl($path, $expected, $scheme, $host, $port, $basePath)
    {
        $request = $this->request
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('getUri')
            ->willReturn($this->createUri($scheme, $host, $port, $basePath));

        $extension = new Psr7UriExtension($this->request);
        $relativeUrl = $extension->generateRelativeUrl($path);

        $this->assertEquals($expected, $relativeUrl);
    }

    public function getGenerateRelativeUrlData()
    {
        return [
            ['/a/b/c/foo.png', 'foo.png', 'http', 'localhost', null, '/a/b/c/d'],
            ['/a/b/foo.png', '../foo.png', 'http', 'localhost', null, '/a/b/c/d'],
            ['/a/b/c/d', '', 'http', 'localhost', null, '/a/b/c/d'],
            ['/a/b/c/', './', 'http', 'localhost', null, '/a/b/c/d'],
            ['/a/b/c/other', 'other', 'http', 'localhost', null, '/a/b/c/d'],
            ['/a/b/z/foo.png', '../z/foo.png', 'http', 'localhost', null, '/a/b/c/d'],
            ['/a/b/c/this:that', './this:that', 'http', 'localhost', null, '/a/b/c/d'],
            ['/a/b/c/foo/this:that', 'foo/this:that', 'http', 'localhost', null, '/a/b/c/d'],
            ['/', '/', 'http', 'localhost', null, ''],
            ['//', '//', 'http', 'localhost', null, '']
        ];
    }

    /**
     * @param string $scheme
     * @param string $host
     * @param string $port
     * @param string $basePath
     *
     * @return UriInterface $uri New request URI to use.
     */
    protected function createUri($scheme, $host, $port, $basePath)
    {
        $uri = $this->getMockBuilder(UriInterface::class)
            ->disableOriginalConstructor()
            ->getMock();

        $uri->expects($this->any())->method('getScheme')->willReturn($scheme);
        $uri->expects($this->any())->method('getHost')->willReturn($host);
        $uri->expects($this->any())->method('getPort')->willReturn($port);
        $uri->expects($this->any())->method('getPath')->willReturn($basePath);

        return $uri;
    }
}


1			<?php
2
3			namespace Asmaster\TwigExtension\tests;
4
5			use PHPUnit_Framework_TestCase;
6			use Psr\Http\Message\UriInterface;
7			use Psr\Http\Message\ServerRequestInterface;
8			use Twig_ExtensionInterface;
9			use Asmaster\TwigExtension\Psr7UriExtension;
10
11			class Psr7UriExtensionTest extends PHPUnit_Framework_TestCase
12			{
13			/**
14			* @var ServerRequestInterface\|\PHPUnit_Framework_MockObject_MockObject
15			*/
16			protected $request;
17
18			public function setUp()
19			{
20			$this->request = $this->getMockBuilder(ServerRequestInterface::class)
21			->disableOriginalConstructor()
22			->getMock();
23			}
24
25			public function testExtension()
26			{
27			$extension = new Psr7UriExtension($this->request);
28
29			$this->assertSame('psr7_uri', $extension->getName());
30			}
31
32			/**
33			* @dataProvider getFunctions()
34			*/
35			public function testExtensionFunctions($function)
36			{
37			$extension = new Psr7UriExtension($this->request);
38
39			$functions = array_map(
40			function ($function) {
41			return $function->getName();
42			},
43			$extension->getFunctions()
44			);
45
46			$this->assertContains($function, $functions);
47			}
48
49			public function getFunctions()
50			{
51			return [[
52			'absolute_url',
53			'relative_url'
54			]];
55			}
56
57			/**
58			* @dataProvider getGenerateAbsoluteUrlData()
59			*/
60			public function testGenerateAbsoluteUrl($path, $expected, $scheme, $host, $port, $basePath)
61			{
62			$request = $this->request
			0 ignored issues – show Bug introduced 2016-04-17 05:30 UTC by Report Bug Copy Issue Report The method `expects` does only exist in `PHPUnit_Framework_MockObject_MockObject`, but not in `Psr\Http\Message\ServerRequestInterface`. 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... Unused Code introduced 2016-04-17 05:30 UTC by Report Bug Copy Issue Report `$request` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
63			->expects($this->any())
64			->method('getUri')
65			->willReturn($this->createUri($scheme, $host, $port, $basePath));
66
67			$extension = new Psr7UriExtension($this->request);
68			$absoluteUrl = $extension->generateAbsoluteUrl($path);
69
70			$this->assertEquals($expected, $absoluteUrl);
71			}
72
73			public function getGenerateAbsoluteUrlData()
74			{
75			return [
76			['/foo.png', '/foo.png', 'http', '', null, '/foo.png'],
77			['/foo.png', 'http://localhost/foo.png', 'http', 'localhost', null, '/'],
78			['/foo.png', 'http://localhost/foo.png', 'http', 'localhost', null, '/foo.png'],
79			['foo.png', 'http://localhost/foo/foo.png', 'http', 'localhost', null, '/foo'],
80			['foo.png', 'http://localhost/bar/foo.png', 'http', 'localhost', null, '/bar'],
81			['foo.png', 'http://localhost/foo/bar/foo.png', 'http', 'localhost', null, '/foo/bar'],
82			['/foo.png', 'http://localhost:8080/foo.png', 'http', 'localhost', 8080, '/'],
83			['/foo.png', 'https://localhost/foo.png', 'https', 'localhost', null, '/'],
84			['/', 'http://localhost/', 'http', 'localhost', null, '/'],
85			['//', '//', 'http', 'localhost', null, '/']
86			];
87			}
88
89			/**
90			* @dataProvider getGenerateRelativeUrlData()
91			*/
92			public function testGenerateRelativeUrl($path, $expected, $scheme, $host, $port, $basePath)
93			{
94			$request = $this->request
			0 ignored issues – show Bug introduced 2016-04-17 05:30 UTC by Report Bug Copy Issue Report The method `expects` does only exist in `PHPUnit_Framework_MockObject_MockObject`, but not in `Psr\Http\Message\ServerRequestInterface`. 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... Unused Code introduced 2016-04-17 05:30 UTC by Report Bug Copy Issue Report `$request` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
95			->expects($this->any())
96			->method('getUri')
97			->willReturn($this->createUri($scheme, $host, $port, $basePath));
98
99			$extension = new Psr7UriExtension($this->request);
100			$relativeUrl = $extension->generateRelativeUrl($path);
101
102			$this->assertEquals($expected, $relativeUrl);
103			}
104
105			public function getGenerateRelativeUrlData()
106			{
107			return [
108			['/a/b/c/foo.png', 'foo.png', 'http', 'localhost', null, '/a/b/c/d'],
109			['/a/b/foo.png', '../foo.png', 'http', 'localhost', null, '/a/b/c/d'],
110			['/a/b/c/d', '', 'http', 'localhost', null, '/a/b/c/d'],
111			['/a/b/c/', './', 'http', 'localhost', null, '/a/b/c/d'],
112			['/a/b/c/other', 'other', 'http', 'localhost', null, '/a/b/c/d'],
113			['/a/b/z/foo.png', '../z/foo.png', 'http', 'localhost', null, '/a/b/c/d'],
114			['/a/b/c/this:that', './this:that', 'http', 'localhost', null, '/a/b/c/d'],
115			['/a/b/c/foo/this:that', 'foo/this:that', 'http', 'localhost', null, '/a/b/c/d'],
116			['/', '/', 'http', 'localhost', null, ''],
117			['//', '//', 'http', 'localhost', null, '']
118			];
119			}
120
121			/**
122			* @param string $scheme
123			* @param string $host
124			* @param string $port
125			* @param string $basePath
126			*
127			* @return UriInterface $uri New request URI to use.
128			*/
129			protected function createUri($scheme, $host, $port, $basePath)
130			{
131			$uri = $this->getMockBuilder(UriInterface::class)
132			->disableOriginalConstructor()
133			->getMock();
134
135			$uri->expects($this->any())->method('getScheme')->willReturn($scheme);
136			$uri->expects($this->any())->method('getHost')->willReturn($host);
137			$uri->expects($this->any())->method('getPort')->willReturn($port);
138			$uri->expects($this->any())->method('getPath')->willReturn($basePath);
139
140			return $uri;
141			}
142			}
143

AlexMasterov / twig-extension

Branch — master (294c1a)

Psr7UriExtensionTest::testGenerateAbsoluteUrl() A

Complexity

Size

Duplication

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like