CacheTest::testDelete() - Code Metrics - Inspection of "Update index and cache when deleting items" - aimeos/ai-admin-jqadm - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 3b83d2...39a0ee )

by Aimeos

created 2016-06-16 21:30 UTC

CacheTest::testDelete() A

↳ Parent: CacheTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	15
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	15
rs	9.4285
cc	1
eloc	10
nc	1
nop	0

<?php

/**
 * @license LGPLv3, http://opensource.org/licenses/LGPL-3.0
 * @copyright Aimeos (aimeos.org), 2016
 */


namespace Aimeos\Admin\JQAdm\Common\Decorator;


class CacheTest extends \PHPUnit_Framework_TestCase
{
	private $context;
	private $object;
	private $mock;
	private $cache;


	protected function setUp()
	{
		$this->cache = $this->getMockBuilder( 'Aimeos\MW\Cache\None' )
			->setMethods( array( 'deleteByTags' ) )
			->disableOriginalConstructor()
			->getMock();

		$this->mock = $this->getMockBuilder( 'Aimeos\Admin\JQAdm\Product\Standard' )
			->setMethods( array( 'delete', 'save' ) )
			->disableOriginalConstructor()
			->getMock();

		$templatePaths = \TestHelperJqadm::getTemplatePaths();
		$this->context = \TestHelperJqadm::getContext();
		$this->context->setCache( $this->cache );

		$this->object = new \Aimeos\Admin\JQAdm\Common\Decorator\Cache( $this->mock, $this->context, $templatePaths );
	}


	protected function tearDown()
	{
		unset( $this->object, $this->mock, $this->context, $this->cache );
	}


	public function testDelete()
	{
		$view = \TestHelperJqadm::getView();
		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $view, array( 'id' => 1 ) );
		$view->addHelper( 'param', $helper );

		$tags = array( 'product', 'product-1' );
		$this->cache->expects( $this->once() )->method( 'deleteByTags' )->with( $this->equalTo( $tags ) );
		$this->mock->expects( $this->once() )->method( 'delete' )->will( $this->returnValue( 'test' ) );

		$this->object->setView( $view );
		$result = $this->object->delete();

		$this->assertEquals( 'test', $result );
	}


	public function testSave()
	{
		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'CNC' );
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}

		$tags = array( 'product', 'product-' . $item->getId() );
		$view = \TestHelperJqadm::getView();
		$view->item = $item;

		$this->cache->expects( $this->once() )->method( 'deleteByTags' )->with( $this->equalTo( $tags ) );
		$this->mock->expects( $this->once() )->method( 'save' )->will( $this->returnValue( 'test' ) );

		$this->object->setView( $view );
		$result = $this->object->save();

		$this->assertEquals( 'test', $result );
	}
}


1			<?php
2
3			/**
4			* @license LGPLv3, http://opensource.org/licenses/LGPL-3.0
5			* @copyright Aimeos (aimeos.org), 2016
6			*/
7
8
9			namespace Aimeos\Admin\JQAdm\Common\Decorator;
10
11
12			class CacheTest extends \PHPUnit_Framework_TestCase
13			{
14			private $context;
15			private $object;
16			private $mock;
17			private $cache;
18
19
20			protected function setUp()
21			{
22			$this->cache = $this->getMockBuilder( 'Aimeos\MW\Cache\None' )
23			->setMethods( array( 'deleteByTags' ) )
24			->disableOriginalConstructor()
25			->getMock();
26
27			$this->mock = $this->getMockBuilder( 'Aimeos\Admin\JQAdm\Product\Standard' )
28			->setMethods( array( 'delete', 'save' ) )
29			->disableOriginalConstructor()
30			->getMock();
31
32			$templatePaths = \TestHelperJqadm::getTemplatePaths();
33			$this->context = \TestHelperJqadm::getContext();
34			$this->context->setCache( $this->cache );
35
36			$this->object = new \Aimeos\Admin\JQAdm\Common\Decorator\Cache( $this->mock, $this->context, $templatePaths );
37			}
38
39
40			protected function tearDown()
41			{
42			unset( $this->object, $this->mock, $this->context, $this->cache );
43			}
44
45
46			public function testDelete()
47			{
48			$view = \TestHelperJqadm::getView();
49			$helper = new \Aimeos\MW\View\Helper\Param\Standard( $view, array( 'id' => 1 ) );
50			$view->addHelper( 'param', $helper );
51
52			$tags = array( 'product', 'product-1' );
53			$this->cache->expects( $this->once() )->method( 'deleteByTags' )->with( $this->equalTo( $tags ) );
54			$this->mock->expects( $this->once() )->method( 'delete' )->will( $this->returnValue( 'test' ) );
55
56			$this->object->setView( $view );
57			$result = $this->object->delete();
58
59			$this->assertEquals( 'test', $result );
60			}
61
62
63			public function testSave()
64			{
65			$item = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'CNC' );
			1 ignored issue – show Bug introduced 2016-06-07 22:23 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Aimeos\MShop\Common\Manager\Iface` as the method `findItem()` does only exist in the following implementations of said interface: `Aimeos\MShop\Attribute\Manager\Lists\Type\Standard`, `Aimeos\MShop\Attribute\Manager\Standard`, `Aimeos\MShop\Attribute\Manager\Type\Standard`, `Aimeos\MShop\Catalog\Manager\Decorator\Base`, `Aimeos\MShop\Catalog\Manager\Decorator\Sitecheck`, `Aimeos\MShop\Catalog\Manager\Lists\Type\Standard`, `Aimeos\MShop\Catalog\Manager\Standard`, `Aimeos\MShop\Common\Manager\Decorator\Base`, `Aimeos\MShop\Common\Manager\Decorator\Changelog`, `Aimeos\MShop\Common\Manager\Decorator\Sitecheck`, `Aimeos\MShop\Common\Manager\Type\Base`, `Aimeos\MShop\Coupon\Manager\Code\Standard`, `Aimeos\MShop\Customer\Manager\Base`, `Aimeos\MShop\Customer\Manager\Group\Standard`, `Aimeos\MShop\Customer\Manager\Lists\Type\Standard`, `Aimeos\MShop\Customer\Manager\Standard`, `Aimeos\MShop\Locale\Manager\Site\Standard`, `Aimeos\MShop\Media\Manager\Lists\Type\Standard`, `Aimeos\MShop\Media\Manager\Type\Standard`, `Aimeos\MShop\Plugin\Manager\Type\Standard`, `Aimeos\MShop\Price\Manager\Lists\Type\Standard`, `Aimeos\MShop\Price\Manager\Type\Standard`, `Aimeos\MShop\Product\Manager\Lists\Type\Standard`, `Aimeos\MShop\Product\Man...\Property\Type\Standard`, `Aimeos\MShop\Product\Manager\Standard`, `Aimeos\MShop\Product\Man...tock\Warehouse\Standard`, `Aimeos\MShop\Product\Manager\Type\Standard`, `Aimeos\MShop\Service\Manager\Lists\Type\Standard`, `Aimeos\MShop\Service\Manager\Standard`, `Aimeos\MShop\Service\Manager\Type\Standard`, `Aimeos\MShop\Supplier\Manager\Lists\Type\Standard`, `Aimeos\MShop\Supplier\Manager\Standard`, `Aimeos\MShop\Tag\Manager\Type\Standard`, `Aimeos\MShop\Text\Manager\Lists\Type\Standard`, `Aimeos\MShop\Text\Manager\Type\Standard`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
66
67			$tags = array( 'product', 'product-' . $item->getId() );
68			$view = \TestHelperJqadm::getView();
69			$view->item = $item;
70
71			$this->cache->expects( $this->once() )->method( 'deleteByTags' )->with( $this->equalTo( $tags ) );
72			$this->mock->expects( $this->once() )->method( 'save' )->will( $this->returnValue( 'test' ) );
73
74			$this->object->setView( $view );
75			$result = $this->object->save();
76
77			$this->assertEquals( 'test', $result );
78			}
79			}
80

aimeos / ai-admin-jqadm

Push — master ( 3b83d2...39a0ee )

CacheTest::testDelete() A

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like