DompdfWrapperTest::testCreateDompdf() - Code Metrics - Inspection of "Removed unused imports" - core23/DompdfBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c317f4...56a5d3 )

by Christian

created 2017-07-19 16:27 UTC

DompdfWrapperTest::testCreateDompdf() A

↳ Parent: DompdfWrapperTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
c	0
b	0
f	0
rs	10
cc	1
eloc	2
nc	1
nop	0

<?php

/*
 * (c) Christian Gripp <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Core23\DompdfBundle\Tests\Wrapper;

use Core23\DompdfBundle\Factory\DompdfFactory;
use Core23\DompdfBundle\Factory\DompdfFactoryInterface;
use Core23\DompdfBundle\Wrapper\DompdfWrapper;
use Dompdf\Dompdf;
use PHPUnit\Framework\TestCase;

class DompdfWrapperTest extends TestCase
{
    /**
     * @var \PHPUnit_Framework_MockObject_MockObject|DompdfFactory
     */
    private $dompdfFactory;

    /**
     * @var DompdfWrapper
     */
    private $dompdfWrapper;

    /**
     * @var \PHPUnit_Framework_MockObject_MockObject|Dompdf
     */
    private $dompdf;

    /**
     * {@inheritdoc}
     */
    protected function setUp()
    {
        $this->dompdf        = $this->createMock(Dompdf::class);
        $this->dompdfFactory = $this->createMock(DompdfFactoryInterface::class);

        $this->dompdfWrapper = new DompdfWrapper($this->dompdfFactory);
    }

    public function testStreamHtml()
    {
        $input = "<h1>Foo</h1>Bar <b>baz</b><img src='img/foo'>";

        $this->dompdfFactory->expects($this->any())
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.
}
            ->method('create')
            ->will($this->returnValue($this->dompdf));

        $this->dompdf->expects($this->once())
            ->method('loadHtml')
            ->with($this->equalTo($input));
        $this->dompdf->expects($this->once())
            ->method('render');
        $this->dompdf->expects($this->once())
            ->method('stream')
            ->with($this->equalTo('file.pdf'));

        $this->dompdfWrapper->streamHtml($input, 'file.pdf');
    }

    public function testStreamHtmlWithImg()
    {
        $input  = "<h1>Foo</h1>Bar <b>baz</b><img src='img/foo'>";
        $output = "<h1>Foo</h1>Bar <b>baz</b><img src='img/foo'>";

        $this->dompdfFactory->expects($this->any())
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.
}
            ->method('create')
            ->with($this->equalTo(array('tempDir' => 'bar')))
            ->will($this->returnValue($this->dompdf));

        $this->dompdf->expects($this->once())
            ->method('loadHtml')
            ->with($this->equalTo($output));
        $this->dompdf->expects($this->once())
            ->method('render');
        $this->dompdf->expects($this->once())
            ->method('stream')
            ->with($this->equalTo('file.pdf'));

        $this->dompdfWrapper->streamHtml($input, 'file.pdf', array('tempDir' => 'bar'));
    }

    public function testGetPdf()
    {
        $input = "<h1>Foo</h1>Bar <b>baz</b><img src='img/foo'>";

        $this->dompdfFactory->expects($this->any())
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.
}
            ->method('create')
            ->will($this->returnValue($this->dompdf));

        $this->dompdf->expects($this->once())
            ->method('loadHtml')
            ->with($this->equalTo($input));
        $this->dompdf->expects($this->once())
            ->method('render');
        $this->dompdf->expects($this->once())
            ->method('output')
            ->willReturn('BINARY_CONTENT');

        $this->dompdfWrapper->getPdf($input, array('tempDir' => 'bar'));
    }
}


1		<?php
2
3		/*
4		* (c) Christian Gripp <[email protected]>
5		*
6		* For the full copyright and license information, please view the LICENSE
7		* file that was distributed with this source code.
8		*/
9
10		namespace Core23\DompdfBundle\Tests\Wrapper;
11
12		use Core23\DompdfBundle\Factory\DompdfFactory;
13		use Core23\DompdfBundle\Factory\DompdfFactoryInterface;
14		use Core23\DompdfBundle\Wrapper\DompdfWrapper;
15		use Dompdf\Dompdf;
16		use PHPUnit\Framework\TestCase;
17
18		class DompdfWrapperTest extends TestCase
19		{
20		/**
21		* @var \PHPUnit_Framework_MockObject_MockObject\|DompdfFactory
22		*/
23		private $dompdfFactory;
24
25		/**
26		* @var DompdfWrapper
27		*/
28		private $dompdfWrapper;
29
30		/**
31		* @var \PHPUnit_Framework_MockObject_MockObject\|Dompdf
32		*/
33		private $dompdf;
34
35		/**
36		* {@inheritdoc}
37		*/
38		protected function setUp()
39		{
40		$this->dompdf = $this->createMock(Dompdf::class);
41		$this->dompdfFactory = $this->createMock(DompdfFactoryInterface::class);
42
43		$this->dompdfWrapper = new DompdfWrapper($this->dompdfFactory);
44		}
45
46	View Code Duplication	public function testStreamHtml()
47		{
48		$input = "<h1>Foo</h1>Bar <b>baz</b><img src='img/foo'>";
49
50		$this->dompdfFactory->expects($this->any())
		1 ignored issue – show Bug introduced 2017-07-19 16:32 UTC by Report Bug Copy Issue Report The method `expects` does only exist in `PHPUnit_Framework_MockObject_MockObject`, but not in `Core23\DompdfBundle\Factory\DompdfFactory`. 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...
51		->method('create')
52		->will($this->returnValue($this->dompdf));
53
54		$this->dompdf->expects($this->once())
55		->method('loadHtml')
56		->with($this->equalTo($input));
57		$this->dompdf->expects($this->once())
58		->method('render');
59		$this->dompdf->expects($this->once())
60		->method('stream')
61		->with($this->equalTo('file.pdf'));
62
63		$this->dompdfWrapper->streamHtml($input, 'file.pdf');
64		}
65
66		public function testStreamHtmlWithImg()
67		{
68		$input = "<h1>Foo</h1>Bar <b>baz</b><img src='img/foo'>";
69		$output = "<h1>Foo</h1>Bar <b>baz</b><img src='img/foo'>";
70
71		$this->dompdfFactory->expects($this->any())
		1 ignored issue – show Bug introduced 2017-07-19 16:32 UTC by Report Bug Copy Issue Report The method `expects` does only exist in `PHPUnit_Framework_MockObject_MockObject`, but not in `Core23\DompdfBundle\Factory\DompdfFactory`. 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...
72		->method('create')
73		->with($this->equalTo(array('tempDir' => 'bar')))
74		->will($this->returnValue($this->dompdf));
75
76		$this->dompdf->expects($this->once())
77		->method('loadHtml')
78		->with($this->equalTo($output));
79		$this->dompdf->expects($this->once())
80		->method('render');
81		$this->dompdf->expects($this->once())
82		->method('stream')
83		->with($this->equalTo('file.pdf'));
84
85		$this->dompdfWrapper->streamHtml($input, 'file.pdf', array('tempDir' => 'bar'));
86		}
87
88	View Code Duplication	public function testGetPdf()
89		{
90		$input = "<h1>Foo</h1>Bar <b>baz</b><img src='img/foo'>";
91
92		$this->dompdfFactory->expects($this->any())
		1 ignored issue – show Bug introduced 2017-07-19 16:32 UTC by Report Bug Copy Issue Report The method `expects` does only exist in `PHPUnit_Framework_MockObject_MockObject`, but not in `Core23\DompdfBundle\Factory\DompdfFactory`. 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...
93		->method('create')
94		->will($this->returnValue($this->dompdf));
95
96		$this->dompdf->expects($this->once())
97		->method('loadHtml')
98		->with($this->equalTo($input));
99		$this->dompdf->expects($this->once())
100		->method('render');
101		$this->dompdf->expects($this->once())
102		->method('output')
103		->willReturn('BINARY_CONTENT');
104
105		$this->dompdfWrapper->getPdf($input, array('tempDir' => 'bar'));
106		}
107		}
108

core23 / DompdfBundle

GitHub Access Token became invalid

Push — master ( c317f4...56a5d3 )

DompdfWrapperTest::testCreateDompdf() A

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like