Inspection of "Translation update" - aimeos/ai-controller-frontend - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 866fb8...8bb600 )

by Aimeos

created 2018-08-09 13:33 UTC

Status

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

@@ -15,31 +15,31 @@
 block discarded – undo
 	{
 		$target = '\\Aimeos\\Controller\\Frontend\\Stock\\Iface';
 
-		$controller = \Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext() );
-		$this->assertInstanceOf( $target, $controller );
+		$controller = \Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext());
+		$this->assertInstanceOf($target, $controller);
 
-		$controller = \Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
-		$this->assertInstanceOf( $target, $controller );
+		$controller = \Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
+		$this->assertInstanceOf($target, $controller);
 	}
 
 
 	public function testCreateControllerInvalidImplementation()
 	{
-		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
-		\Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
+		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
+		\Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
 	}
 
 
 	public function testCreateControllerInvalidName()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext(), '%^');
 	}
 
 
 	public function testCreateControllerNotExisting()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
 	}
 }

Please login to merge, or discard this patch.

frontend/tests/Controller/Frontend/Supplier/Decorator/BaseTest.php 1 patch

Spacing +35 added lines, -35 removed lines patch added patch discarded remove patch

@@ -20,106 +20,106 @@
 block discarded – undo
 	{
 		$this->context = \TestHelperFrontend::getContext();
 
-		$this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Supplier\Standard' )
+		$this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Supplier\Standard')
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Supplier\Decorator\Base' )
-			->setConstructorArgs( [$this->stub, $this->context] )
+		$this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Supplier\Decorator\Base')
+			->setConstructorArgs([$this->stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->context, $this->object, $this->stub );
+		unset($this->context, $this->object, $this->stub);
 	}
 
 
 	public function testConstructException()
 	{
-		$stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Iface' )->getMock();
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock();
 
-		$this->setExpectedException( '\Aimeos\MW\Common\Exception' );
+		$this->setExpectedException('\Aimeos\MW\Common\Exception');
 
-		$this->getMockBuilder( '\Aimeos\Controller\Frontend\Supplier\Decorator\Base' )
-			->setConstructorArgs( [$stub, $this->context] )
+		$this->getMockBuilder('\Aimeos\Controller\Frontend\Supplier\Decorator\Base')
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	public function testCall()
 	{
-		$stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Supplier\Standard' )
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Supplier\Standard')
 			->disableOriginalConstructor()
-			->setMethods( ['invalid'] )
+			->setMethods(['invalid'])
 			->getMock();
 
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Supplier\Decorator\Base' )
-			->setConstructorArgs( [$stub, $this->context] )
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Supplier\Decorator\Base')
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 
-		$stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
+		$stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
 
-		$this->assertTrue( $object->invalid() );
+		$this->assertTrue($object->invalid());
 	}
 
 
 	public function testCreateFilter()
 	{
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'supplier' )->createSearch();
+		$search = \Aimeos\MShop\Factory::createManager($this->context, 'supplier')->createSearch();
 
-		$this->stub->expects( $this->once() )->method( 'createFilter' )
-			->will( $this->returnValue( $search ) );
+		$this->stub->expects($this->once())->method('createFilter')
+			->will($this->returnValue($search));
 
-		$this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->createFilter() );
+		$this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->createFilter());
 	}
 
 
 	public function testGetItem()
 	{
-		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'supplier' )->createItem();
+		$item = \Aimeos\MShop\Factory::createManager($this->context, 'supplier')->createItem();
 
-		$this->stub->expects( $this->once() )->method( 'getItem' )
-			->will( $this->returnValue( $item ) );
+		$this->stub->expects($this->once())->method('getItem')
+			->will($this->returnValue($item));
 
-		$this->assertInstanceOf( '\Aimeos\MShop\Supplier\Item\Iface', $this->object->getItem( -1 ) );
+		$this->assertInstanceOf('\Aimeos\MShop\Supplier\Item\Iface', $this->object->getItem( -1 ));
 	}
 
 
 	public function testGetItems()
 	{
-		$this->stub->expects( $this->once() )->method( 'getItems' )
-			->will( $this->returnValue( [] ) );
+		$this->stub->expects($this->once())->method('getItems')
+			->will($this->returnValue([]));
 
-		$this->assertEquals( [], $this->object->getItems( [-1], ['media'] ) );
+		$this->assertEquals([], $this->object->getItems([-1], ['media']));
 	}
 
 
 	public function testSearchItems()
 	{
-		$filter = \Aimeos\MShop\Factory::createManager( $this->context, 'supplier' )->createSearch();
+		$filter = \Aimeos\MShop\Factory::createManager($this->context, 'supplier')->createSearch();
 
-		$this->stub->expects( $this->once() )->method( 'searchItems' )
-			->will( $this->returnValue( [] ) );
+		$this->stub->expects($this->once())->method('searchItems')
+			->will($this->returnValue([]));
 
-		$this->assertEquals( [], $this->object->searchItems( $filter, ['media'] ) );
+		$this->assertEquals([], $this->object->searchItems($filter, ['media']));
 	}
 
 
 	public function testGetController()
 	{
-		$result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
+		$result = $this->access('getController')->invokeArgs($this->object, []);
 
-		$this->assertSame( $this->stub, $result );
+		$this->assertSame($this->stub, $result);
 	}
 
 
-	protected function access( $name )
+	protected function access($name)
 	{
-		$class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Supplier\Decorator\Base' );
-		$method = $class->getMethod( $name );
-		$method->setAccessible( true );
+		$class = new \ReflectionClass('\Aimeos\Controller\Frontend\Supplier\Decorator\Base');
+		$method = $class->getMethod($name);
+		$method->setAccessible(true);
 
 		return $method;
 	}

Please login to merge, or discard this patch.

controller/frontend/tests/Controller/Frontend/Supplier/FactoryTest.php 1 patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

@@ -15,31 +15,31 @@
 block discarded – undo
 	{
 		$target = '\\Aimeos\\Controller\\Frontend\\Supplier\\Iface';
 
-		$controller = \Aimeos\Controller\Frontend\Supplier\Factory::createController( \TestHelperFrontend::getContext() );
-		$this->assertInstanceOf( $target, $controller );
+		$controller = \Aimeos\Controller\Frontend\Supplier\Factory::createController(\TestHelperFrontend::getContext());
+		$this->assertInstanceOf($target, $controller);
 
-		$controller = \Aimeos\Controller\Frontend\Supplier\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
-		$this->assertInstanceOf( $target, $controller );
+		$controller = \Aimeos\Controller\Frontend\Supplier\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
+		$this->assertInstanceOf($target, $controller);
 	}
 
 
 	public function testCreateControllerInvalidImplementation()
 	{
-		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
-		\Aimeos\Controller\Frontend\Supplier\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
+		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
+		\Aimeos\Controller\Frontend\Supplier\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
 	}
 
 
 	public function testCreateControllerInvalidName()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend\Supplier\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend\Supplier\Factory::createController(\TestHelperFrontend::getContext(), '%^');
 	}
 
 
 	public function testCreateControllerNotExisting()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend\Supplier\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend\Supplier\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
 	}
 }

Please login to merge, or discard this patch.

frontend/tests/Controller/Frontend/Catalog/Decorator/BaseTest.php 1 patch

Spacing +31 added lines, -31 removed lines patch added patch discarded remove patch

@@ -20,95 +20,95 @@
 block discarded – undo
 	{
 		$this->context = \TestHelperFrontend::getContext();
 
-		$this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Catalog\Standard' )
+		$this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Catalog\Standard')
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Catalog\Decorator\Base' )
-			->setConstructorArgs( [$this->stub, $this->context] )
+		$this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Catalog\Decorator\Base')
+			->setConstructorArgs([$this->stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->context, $this->object, $this->stub );
+		unset($this->context, $this->object, $this->stub);
 	}
 
 
 	public function testConstructException()
 	{
-		$stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Iface' )->getMock();
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock();
 
-		$this->setExpectedException( '\Aimeos\MW\Common\Exception' );
+		$this->setExpectedException('\Aimeos\MW\Common\Exception');
 
-		$this->getMockBuilder( '\Aimeos\Controller\Frontend\Catalog\Decorator\Base' )
-			->setConstructorArgs( [$stub, $this->context] )
+		$this->getMockBuilder('\Aimeos\Controller\Frontend\Catalog\Decorator\Base')
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	public function testCall()
 	{
-		$stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Catalog\Standard' )
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Catalog\Standard')
 			->disableOriginalConstructor()
-			->setMethods( ['invalid'] )
+			->setMethods(['invalid'])
 			->getMock();
 
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Catalog\Decorator\Base' )
-			->setConstructorArgs( [$stub, $this->context] )
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Catalog\Decorator\Base')
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 
-		$stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
+		$stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
 
-		$this->assertTrue( $object->invalid() );
+		$this->assertTrue($object->invalid());
 	}
 
 
 	public function testCreateFilter()
 	{
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'catalog' )->createSearch();
+		$search = \Aimeos\MShop\Factory::createManager($this->context, 'catalog')->createSearch();
 
-		$this->stub->expects( $this->once() )->method( 'createFilter' )
-			->will( $this->returnValue( $search ) );
+		$this->stub->expects($this->once())->method('createFilter')
+			->will($this->returnValue($search));
 
-		$this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->createFilter() );
+		$this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->createFilter());
 	}
 
 
 	public function testGetPath()
 	{
-		$this->stub->expects( $this->once() )->method( 'getPath' )
-			->will( $this->returnValue( [] ) );
+		$this->stub->expects($this->once())->method('getPath')
+			->will($this->returnValue([]));
 
-		$this->assertEquals( [], $this->object->getPath( -1 ) );
+		$this->assertEquals([], $this->object->getPath( -1 ));
 	}
 
 
 	public function testGetTree()
 	{
-		$catItem = \Aimeos\MShop\Factory::createManager( $this->context, 'catalog' )->createItem();
+		$catItem = \Aimeos\MShop\Factory::createManager($this->context, 'catalog')->createItem();
 
-		$this->stub->expects( $this->once() )->method( 'getTree' )
-			->will( $this->returnValue( $catItem ) );
+		$this->stub->expects($this->once())->method('getTree')
+			->will($this->returnValue($catItem));
 
-		$this->assertInstanceOf( '\Aimeos\MShop\Catalog\Item\Iface', $this->object->getTree() );
+		$this->assertInstanceOf('\Aimeos\MShop\Catalog\Item\Iface', $this->object->getTree());
 	}
 
 
 	public function testGetController()
 	{
-		$result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
+		$result = $this->access('getController')->invokeArgs($this->object, []);
 
-		$this->assertSame( $this->stub, $result );
+		$this->assertSame($this->stub, $result);
 	}
 
 
-	protected function access( $name )
+	protected function access($name)
 	{
-		$class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Catalog\Decorator\Base' );
-		$method = $class->getMethod( $name );
-		$method->setAccessible( true );
+		$class = new \ReflectionClass('\Aimeos\Controller\Frontend\Catalog\Decorator\Base');
+		$method = $class->getMethod($name);
+		$method->setAccessible(true);
 
 		return $method;
 	}

Please login to merge, or discard this patch.

controller/frontend/tests/Controller/Frontend/Catalog/FactoryTest.php 1 patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

@@ -14,31 +14,31 @@
 block discarded – undo
 	{
 		$target = '\\Aimeos\\Controller\\Frontend\\Catalog\\Iface';
 
-		$controller = \Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext() );
-		$this->assertInstanceOf( $target, $controller );
+		$controller = \Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext());
+		$this->assertInstanceOf($target, $controller);
 
-		$controller = \Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
-		$this->assertInstanceOf( $target, $controller );
+		$controller = \Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
+		$this->assertInstanceOf($target, $controller);
 	}
 
 
 	public function testCreateControllerInvalidImplementation()
 	{
-		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
-		\Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
+		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
+		\Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
 	}
 
 
 	public function testCreateControllerInvalidName()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext(), '%^');
 	}
 
 
 	public function testCreateControllerNotExisting()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
 	}
 }

Please login to merge, or discard this patch.

frontend/tests/Controller/Frontend/Attribute/Decorator/BaseTest.php 1 patch

Spacing +39 added lines, -39 removed lines patch added patch discarded remove patch

@@ -20,117 +20,117 @@
 block discarded – undo
 	{
 		$this->context = \TestHelperFrontend::getContext();
 
-		$this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Standard' )
+		$this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Standard')
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' )
-			->setConstructorArgs( [$this->stub, $this->context] )
+		$this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Decorator\Base')
+			->setConstructorArgs([$this->stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->context, $this->object, $this->stub );
+		unset($this->context, $this->object, $this->stub);
 	}
 
 
 	public function testConstructException()
 	{
-		$stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Iface' )->getMock();
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock();
 
-		$this->setExpectedException( '\Aimeos\MW\Common\Exception' );
+		$this->setExpectedException('\Aimeos\MW\Common\Exception');
 
-		$this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' )
-			->setConstructorArgs( [$stub, $this->context] )
+		$this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Decorator\Base')
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	public function testCall()
 	{
-		$stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Standard' )
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Standard')
 			->disableOriginalConstructor()
-			->setMethods( ['invalid'] )
+			->setMethods(['invalid'])
 			->getMock();
 
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' )
-			->setConstructorArgs( [$stub, $this->context] )
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Decorator\Base')
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 
-		$stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
+		$stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
 
-		$this->assertTrue( $object->invalid() );
+		$this->assertTrue($object->invalid());
 	}
 
 
 	public function testAddFilterTypes()
 	{
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createSearch();
+		$search = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createSearch();
 
-		$this->stub->expects( $this->once() )->method( 'addFilterTypes' )
-			->will( $this->returnArgument( 0 ) );
+		$this->stub->expects($this->once())->method('addFilterTypes')
+			->will($this->returnArgument(0));
 
-		$this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->addFilterTypes( $search, [] ) );
+		$this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->addFilterTypes($search, []));
 	}
 
 
 	public function testCreateFilter()
 	{
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createSearch();
+		$search = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createSearch();
 
-		$this->stub->expects( $this->once() )->method( 'createFilter' )
-			->will( $this->returnValue( $search ) );
+		$this->stub->expects($this->once())->method('createFilter')
+			->will($this->returnValue($search));
 
-		$this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->createFilter() );
+		$this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->createFilter());
 	}
 
 
 	public function testGetItem()
 	{
-		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createItem();
+		$item = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createItem();
 
-		$this->stub->expects( $this->once() )->method( 'getItem' )
-			->will( $this->returnValue( $item ) );
+		$this->stub->expects($this->once())->method('getItem')
+			->will($this->returnValue($item));
 
-		$this->assertInstanceOf( '\Aimeos\MShop\Attribute\Item\Iface', $this->object->getItem( -1 ) );
+		$this->assertInstanceOf('\Aimeos\MShop\Attribute\Item\Iface', $this->object->getItem( -1 ));
 	}
 
 
 	public function testGetItems()
 	{
-		$this->stub->expects( $this->once() )->method( 'getItems' )
-			->will( $this->returnValue( [] ) );
+		$this->stub->expects($this->once())->method('getItems')
+			->will($this->returnValue([]));
 
-		$this->assertEquals( [], $this->object->getItems( [-1], ['media'] ) );
+		$this->assertEquals([], $this->object->getItems([-1], ['media']));
 	}
 
 
 	public function testSearchItems()
 	{
-		$filter = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createSearch();
+		$filter = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createSearch();
 
-		$this->stub->expects( $this->once() )->method( 'searchItems' )
-			->will( $this->returnValue( [] ) );
+		$this->stub->expects($this->once())->method('searchItems')
+			->will($this->returnValue([]));
 
-		$this->assertEquals( [], $this->object->searchItems( $filter, ['media'] ) );
+		$this->assertEquals([], $this->object->searchItems($filter, ['media']));
 	}
 
 
 	public function testGetController()
 	{
-		$result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
+		$result = $this->access('getController')->invokeArgs($this->object, []);
 
-		$this->assertSame( $this->stub, $result );
+		$this->assertSame($this->stub, $result);
 	}
 
 
-	protected function access( $name )
+	protected function access($name)
 	{
-		$class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' );
-		$method = $class->getMethod( $name );
-		$method->setAccessible( true );
+		$class = new \ReflectionClass('\Aimeos\Controller\Frontend\Attribute\Decorator\Base');
+		$method = $class->getMethod($name);
+		$method->setAccessible(true);
 
 		return $method;
 	}

Please login to merge, or discard this patch.

controller/frontend/tests/Controller/Frontend/Attribute/FactoryTest.php 1 patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

@@ -15,31 +15,31 @@
 block discarded – undo
 	{
 		$target = '\\Aimeos\\Controller\\Frontend\\Attribute\\Iface';
 
-		$controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext() );
-		$this->assertInstanceOf( $target, $controller );
+		$controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext());
+		$this->assertInstanceOf($target, $controller);
 
-		$controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
-		$this->assertInstanceOf( $target, $controller );
+		$controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
+		$this->assertInstanceOf($target, $controller);
 	}
 
 
 	public function testCreateControllerInvalidImplementation()
 	{
-		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
-		\Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
+		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
+		\Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
 	}
 
 
 	public function testCreateControllerInvalidName()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext(), '%^');
 	}
 
 
 	public function testCreateControllerNotExisting()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
 	}
 }

Please login to merge, or discard this patch.

controller/frontend/tests/Controller/Frontend/Basket/Decorator/BaseTest.php 1 patch

Spacing +45 added lines, -45 removed lines patch added patch discarded remove patch

@@ -20,54 +20,54 @@  discard block
 block discarded – undo
 	{
 		$this->context = \TestHelperFrontend::getContext();
 
-		$this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
+		$this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Decorator\Base' )
-			->setConstructorArgs( [$this->stub, $this->context] )
+		$this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Decorator\Base')
+			->setConstructorArgs([$this->stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->context, $this->object, $this->stub );
+		unset($this->context, $this->object, $this->stub);
 	}
 
 
 	public function testConstructException()
 	{
-		$stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Iface' )->getMock();
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock();
 
-		$this->setExpectedException( '\Aimeos\MW\Common\Exception' );
+		$this->setExpectedException('\Aimeos\MW\Common\Exception');
 
-		$this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Decorator\Base' )
-			->setConstructorArgs( [$stub, $this->context] )
+		$this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Decorator\Base')
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	public function testCall()
 	{
-		$stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
 			->disableOriginalConstructor()
-			->setMethods( ['invalid'] )
+			->setMethods(['invalid'])
 			->getMock();
 
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Decorator\Base' )
-			->setConstructorArgs( [$stub, $this->context] )
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Decorator\Base')
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 
-		$stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
+		$stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
 
-		$this->assertTrue( $object->invalid() );
+		$this->assertTrue($object->invalid());
 	}
 
 
 	public function testClear()
 	{
-		$this->stub->expects( $this->once() )->method( 'clear' );
+		$this->stub->expects($this->once())->method('clear');
 
 		$this->object->clear();
 	}
@@ -76,18 +76,18 @@  discard block
 block discarded – undo
 	public function testGet()
 	{
 		$context = \TestHelperFrontend::getContext();
-		$order = \Aimeos\MShop\Factory::createManager( $context, 'order/base' )->createItem();
+		$order = \Aimeos\MShop\Factory::createManager($context, 'order/base')->createItem();
 
-		$this->stub->expects( $this->once() )->method( 'get' )
-			->will( $this->returnValue( $order ) );
+		$this->stub->expects($this->once())->method('get')
+			->will($this->returnValue($order));
 
-		$this->assertInstanceOf( '\Aimeos\MShop\Order\Item\Base\Iface', $this->object->get() );
+		$this->assertInstanceOf('\Aimeos\MShop\Order\Item\Base\Iface', $this->object->get());
 	}
 
 
 	public function testSave()
 	{
-		$this->stub->expects( $this->once() )->method( 'save' );
+		$this->stub->expects($this->once())->method('save');
 
 		$this->object->save();
 	}
@@ -95,15 +95,15 @@  discard block
 block discarded – undo
 
 	public function testSetType()
 	{
-		$this->stub->expects( $this->once() )->method( 'setType' );
+		$this->stub->expects($this->once())->method('setType');
 
-		$this->object->setType( 'test' );
+		$this->object->setType('test');
 	}
 
 
 	public function testStore()
 	{
-		$this->stub->expects( $this->once() )->method( 'store' );
+		$this->stub->expects($this->once())->method('store');
 
 		$this->object->store();
 	}
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
 
 	public function testLoad()
 	{
-		$this->stub->expects( $this->once() )->method( 'load' );
+		$this->stub->expects($this->once())->method('load');
 
 		$this->object->load( -1 );
 	}
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
 
 	public function testAddProduct()
 	{
-		$this->stub->expects( $this->once() )->method( 'addProduct' );
+		$this->stub->expects($this->once())->method('addProduct');
 
 		$this->object->addProduct( -1 );
 	}
@@ -127,73 +127,73 @@  discard block
 block discarded – undo
 
 	public function testDeleteProduct()
 	{
-		$this->stub->expects( $this->once() )->method( 'deleteProduct' );
+		$this->stub->expects($this->once())->method('deleteProduct');
 
-		$this->object->deleteProduct( 0 );
+		$this->object->deleteProduct(0);
 	}
 
 
 	public function testEditProduct()
 	{
-		$this->stub->expects( $this->once() )->method( 'editProduct' );
+		$this->stub->expects($this->once())->method('editProduct');
 
-		$this->object->editProduct( 0, 1 );
+		$this->object->editProduct(0, 1);
 	}
 
 
 	public function testAddCoupon()
 	{
-		$this->stub->expects( $this->once() )->method( 'addCoupon' );
+		$this->stub->expects($this->once())->method('addCoupon');
 
-		$this->object->addCoupon( 'test' );
+		$this->object->addCoupon('test');
 	}
 
 
 	public function testDeleteCoupon()
 	{
-		$this->stub->expects( $this->once() )->method( 'deleteCoupon' );
+		$this->stub->expects($this->once())->method('deleteCoupon');
 
-		$this->object->deleteCoupon( 'test' );
+		$this->object->deleteCoupon('test');
 	}
 
 
 	public function testSetAddress()
 	{
-		$this->stub->expects( $this->once() )->method( 'setAddress' );
+		$this->stub->expects($this->once())->method('setAddress');
 
-		$this->object->setAddress( \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT, null );
+		$this->object->setAddress(\Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT, null);
 	}
 
 
 	public function testAddService()
 	{
-		$this->stub->expects( $this->once() )->method( 'addService' );
+		$this->stub->expects($this->once())->method('addService');
 
-		$this->object->addService( \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT, -1 );
+		$this->object->addService(\Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT, -1);
 	}
 
 
 	public function testDeleteService()
 	{
-		$this->stub->expects( $this->once() )->method( 'deleteService' );
+		$this->stub->expects($this->once())->method('deleteService');
 
-		$this->object->deleteService( \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT );
+		$this->object->deleteService(\Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT);
 	}
 
 
 	public function testGetController()
 	{
-		$result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
+		$result = $this->access('getController')->invokeArgs($this->object, []);
 
-		$this->assertSame( $this->stub, $result );
+		$this->assertSame($this->stub, $result);
 	}
 
 
-	protected function access( $name )
+	protected function access($name)
 	{
-		$class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Basket\Decorator\Base' );
-		$method = $class->getMethod( $name );
-		$method->setAccessible( true );
+		$class = new \ReflectionClass('\Aimeos\Controller\Frontend\Basket\Decorator\Base');
+		$method = $class->getMethod($name);
+		$method->setAccessible(true);
 
 		return $method;
 	}

Please login to merge, or discard this patch.

controller/frontend/tests/Controller/Frontend/Basket/FactoryTest.php 1 patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

@@ -14,31 +14,31 @@
 block discarded – undo
 	{
 		$target = '\\Aimeos\\Controller\\Frontend\\Basket\\Iface';
 
-		$controller = \Aimeos\Controller\Frontend\Basket\Factory::createController( \TestHelperFrontend::getContext() );
-		$this->assertInstanceOf( $target, $controller );
+		$controller = \Aimeos\Controller\Frontend\Basket\Factory::createController(\TestHelperFrontend::getContext());
+		$this->assertInstanceOf($target, $controller);
 
-		$controller = \Aimeos\Controller\Frontend\Basket\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
-		$this->assertInstanceOf( $target, $controller );
+		$controller = \Aimeos\Controller\Frontend\Basket\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
+		$this->assertInstanceOf($target, $controller);
 	}
 
 
 	public function testCreateControllerInvalidImplementation()
 	{
-		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
-		\Aimeos\Controller\Frontend\Basket\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
+		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
+		\Aimeos\Controller\Frontend\Basket\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
 	}
 
 
 	public function testCreateControllerInvalidName()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend\Basket\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend\Basket\Factory::createController(\TestHelperFrontend::getContext(), '%^');
 	}
 
 
 	public function testCreateControllerNotExisting()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend\Basket\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend\Basket\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
 	}
 }

Please login to merge, or discard this patch.

		@@ -20,106 +20,106 @@
		block discarded – undo
20	20	{
21	21	$this->context = \TestHelperFrontend::getContext();
22	22
23		- $this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Supplier\Standard' )
	23	+ $this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Supplier\Standard')
24	24	->disableOriginalConstructor()
25	25	->getMock();
26	26
27		- $this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Supplier\Decorator\Base' )
28		- ->setConstructorArgs( [$this->stub, $this->context] )
	27	+ $this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Supplier\Decorator\Base')
	28	+ ->setConstructorArgs([$this->stub, $this->context])
29	29	->getMockForAbstractClass();
30	30	}
31	31
32	32
33	33	protected function tearDown()
34	34	{
35		- unset( $this->context, $this->object, $this->stub );
	35	+ unset($this->context, $this->object, $this->stub);
36	36	}
37	37
38	38
39	39	public function testConstructException()
40	40	{
41		- $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Iface' )->getMock();
	41	+ $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock();
42	42
43		- $this->setExpectedException( '\Aimeos\MW\Common\Exception' );
	43	+ $this->setExpectedException('\Aimeos\MW\Common\Exception');
44	44
45		- $this->getMockBuilder( '\Aimeos\Controller\Frontend\Supplier\Decorator\Base' )
46		- ->setConstructorArgs( [$stub, $this->context] )
	45	+ $this->getMockBuilder('\Aimeos\Controller\Frontend\Supplier\Decorator\Base')
	46	+ ->setConstructorArgs([$stub, $this->context])
47	47	->getMockForAbstractClass();
48	48	}
49	49
50	50
51	51	public function testCall()
52	52	{
53		- $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Supplier\Standard' )
	53	+ $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Supplier\Standard')
54	54	->disableOriginalConstructor()
55		- ->setMethods( ['invalid'] )
	55	+ ->setMethods(['invalid'])
56	56	->getMock();
57	57
58		- $object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Supplier\Decorator\Base' )
59		- ->setConstructorArgs( [$stub, $this->context] )
	58	+ $object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Supplier\Decorator\Base')
	59	+ ->setConstructorArgs([$stub, $this->context])
60	60	->getMockForAbstractClass();
61	61
62		- $stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
	62	+ $stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
63	63
64		- $this->assertTrue( $object->invalid() );
	64	+ $this->assertTrue($object->invalid());
65	65	}
66	66
67	67
68	68	public function testCreateFilter()
69	69	{
70		- $search = \Aimeos\MShop\Factory::createManager( $this->context, 'supplier' )->createSearch();
	70	+ $search = \Aimeos\MShop\Factory::createManager($this->context, 'supplier')->createSearch();
71	71
72		- $this->stub->expects( $this->once() )->method( 'createFilter' )
73		- ->will( $this->returnValue( $search ) );
	72	+ $this->stub->expects($this->once())->method('createFilter')
	73	+ ->will($this->returnValue($search));
74	74
75		- $this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->createFilter() );
	75	+ $this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->createFilter());
76	76	}
77	77
78	78
79	79	public function testGetItem()
80	80	{
81		- $item = \Aimeos\MShop\Factory::createManager( $this->context, 'supplier' )->createItem();
	81	+ $item = \Aimeos\MShop\Factory::createManager($this->context, 'supplier')->createItem();
82	82
83		- $this->stub->expects( $this->once() )->method( 'getItem' )
84		- ->will( $this->returnValue( $item ) );
	83	+ $this->stub->expects($this->once())->method('getItem')
	84	+ ->will($this->returnValue($item));
85	85
86		- $this->assertInstanceOf( '\Aimeos\MShop\Supplier\Item\Iface', $this->object->getItem( -1 ) );
	86	+ $this->assertInstanceOf('\Aimeos\MShop\Supplier\Item\Iface', $this->object->getItem( -1 ));
87	87	}
88	88
89	89
90	90	public function testGetItems()
91	91	{
92		- $this->stub->expects( $this->once() )->method( 'getItems' )
93		- ->will( $this->returnValue( [] ) );
	92	+ $this->stub->expects($this->once())->method('getItems')
	93	+ ->will($this->returnValue([]));
94	94
95		- $this->assertEquals( [], $this->object->getItems( [-1], ['media'] ) );
	95	+ $this->assertEquals([], $this->object->getItems([-1], ['media']));
96	96	}
97	97
98	98
99	99	public function testSearchItems()
100	100	{
101		- $filter = \Aimeos\MShop\Factory::createManager( $this->context, 'supplier' )->createSearch();
	101	+ $filter = \Aimeos\MShop\Factory::createManager($this->context, 'supplier')->createSearch();
102	102
103		- $this->stub->expects( $this->once() )->method( 'searchItems' )
104		- ->will( $this->returnValue( [] ) );
	103	+ $this->stub->expects($this->once())->method('searchItems')
	104	+ ->will($this->returnValue([]));
105	105
106		- $this->assertEquals( [], $this->object->searchItems( $filter, ['media'] ) );
	106	+ $this->assertEquals([], $this->object->searchItems($filter, ['media']));
107	107	}
108	108
109	109
110	110	public function testGetController()
111	111	{
112		- $result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
	112	+ $result = $this->access('getController')->invokeArgs($this->object, []);
113	113
114		- $this->assertSame( $this->stub, $result );
	114	+ $this->assertSame($this->stub, $result);
115	115	}
116	116
117	117
118		- protected function access( $name )
	118	+ protected function access($name)
119	119	{
120		- $class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Supplier\Decorator\Base' );
121		- $method = $class->getMethod( $name );
122		- $method->setAccessible( true );
	120	+ $class = new \ReflectionClass('\Aimeos\Controller\Frontend\Supplier\Decorator\Base');
	121	+ $method = $class->getMethod($name);
	122	+ $method->setAccessible(true);
123	123
124	124	return $method;
125	125	}

		@@ -20,95 +20,95 @@
		block discarded – undo
20	20	{
21	21	$this->context = \TestHelperFrontend::getContext();
22	22
23		- $this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Catalog\Standard' )
	23	+ $this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Catalog\Standard')
24	24	->disableOriginalConstructor()
25	25	->getMock();
26	26
27		- $this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Catalog\Decorator\Base' )
28		- ->setConstructorArgs( [$this->stub, $this->context] )
	27	+ $this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Catalog\Decorator\Base')
	28	+ ->setConstructorArgs([$this->stub, $this->context])
29	29	->getMockForAbstractClass();
30	30	}
31	31
32	32
33	33	protected function tearDown()
34	34	{
35		- unset( $this->context, $this->object, $this->stub );
	35	+ unset($this->context, $this->object, $this->stub);
36	36	}
37	37
38	38
39	39	public function testConstructException()
40	40	{
41		- $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Iface' )->getMock();
	41	+ $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock();
42	42
43		- $this->setExpectedException( '\Aimeos\MW\Common\Exception' );
	43	+ $this->setExpectedException('\Aimeos\MW\Common\Exception');
44	44
45		- $this->getMockBuilder( '\Aimeos\Controller\Frontend\Catalog\Decorator\Base' )
46		- ->setConstructorArgs( [$stub, $this->context] )
	45	+ $this->getMockBuilder('\Aimeos\Controller\Frontend\Catalog\Decorator\Base')
	46	+ ->setConstructorArgs([$stub, $this->context])
47	47	->getMockForAbstractClass();
48	48	}
49	49
50	50
51	51	public function testCall()
52	52	{
53		- $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Catalog\Standard' )
	53	+ $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Catalog\Standard')
54	54	->disableOriginalConstructor()
55		- ->setMethods( ['invalid'] )
	55	+ ->setMethods(['invalid'])
56	56	->getMock();
57	57
58		- $object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Catalog\Decorator\Base' )
59		- ->setConstructorArgs( [$stub, $this->context] )
	58	+ $object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Catalog\Decorator\Base')
	59	+ ->setConstructorArgs([$stub, $this->context])
60	60	->getMockForAbstractClass();
61	61
62		- $stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
	62	+ $stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
63	63
64		- $this->assertTrue( $object->invalid() );
	64	+ $this->assertTrue($object->invalid());
65	65	}
66	66
67	67
68	68	public function testCreateFilter()
69	69	{
70		- $search = \Aimeos\MShop\Factory::createManager( $this->context, 'catalog' )->createSearch();
	70	+ $search = \Aimeos\MShop\Factory::createManager($this->context, 'catalog')->createSearch();
71	71
72		- $this->stub->expects( $this->once() )->method( 'createFilter' )
73		- ->will( $this->returnValue( $search ) );
	72	+ $this->stub->expects($this->once())->method('createFilter')
	73	+ ->will($this->returnValue($search));
74	74
75		- $this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->createFilter() );
	75	+ $this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->createFilter());
76	76	}
77	77
78	78
79	79	public function testGetPath()
80	80	{
81		- $this->stub->expects( $this->once() )->method( 'getPath' )
82		- ->will( $this->returnValue( [] ) );
	81	+ $this->stub->expects($this->once())->method('getPath')
	82	+ ->will($this->returnValue([]));
83	83
84		- $this->assertEquals( [], $this->object->getPath( -1 ) );
	84	+ $this->assertEquals([], $this->object->getPath( -1 ));
85	85	}
86	86
87	87
88	88	public function testGetTree()
89	89	{
90		- $catItem = \Aimeos\MShop\Factory::createManager( $this->context, 'catalog' )->createItem();
	90	+ $catItem = \Aimeos\MShop\Factory::createManager($this->context, 'catalog')->createItem();
91	91
92		- $this->stub->expects( $this->once() )->method( 'getTree' )
93		- ->will( $this->returnValue( $catItem ) );
	92	+ $this->stub->expects($this->once())->method('getTree')
	93	+ ->will($this->returnValue($catItem));
94	94
95		- $this->assertInstanceOf( '\Aimeos\MShop\Catalog\Item\Iface', $this->object->getTree() );
	95	+ $this->assertInstanceOf('\Aimeos\MShop\Catalog\Item\Iface', $this->object->getTree());
96	96	}
97	97
98	98
99	99	public function testGetController()
100	100	{
101		- $result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
	101	+ $result = $this->access('getController')->invokeArgs($this->object, []);
102	102
103		- $this->assertSame( $this->stub, $result );
	103	+ $this->assertSame($this->stub, $result);
104	104	}
105	105
106	106
107		- protected function access( $name )
	107	+ protected function access($name)
108	108	{
109		- $class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Catalog\Decorator\Base' );
110		- $method = $class->getMethod( $name );
111		- $method->setAccessible( true );
	109	+ $class = new \ReflectionClass('\Aimeos\Controller\Frontend\Catalog\Decorator\Base');
	110	+ $method = $class->getMethod($name);
	111	+ $method->setAccessible(true);
112	112
113	113	return $method;
114	114	}

		@@ -20,117 +20,117 @@
		block discarded – undo
20	20	{
21	21	$this->context = \TestHelperFrontend::getContext();
22	22
23		- $this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Standard' )
	23	+ $this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Standard')
24	24	->disableOriginalConstructor()
25	25	->getMock();
26	26
27		- $this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' )
28		- ->setConstructorArgs( [$this->stub, $this->context] )
	27	+ $this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Decorator\Base')
	28	+ ->setConstructorArgs([$this->stub, $this->context])
29	29	->getMockForAbstractClass();
30	30	}
31	31
32	32
33	33	protected function tearDown()
34	34	{
35		- unset( $this->context, $this->object, $this->stub );
	35	+ unset($this->context, $this->object, $this->stub);
36	36	}
37	37
38	38
39	39	public function testConstructException()
40	40	{
41		- $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Iface' )->getMock();
	41	+ $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock();
42	42
43		- $this->setExpectedException( '\Aimeos\MW\Common\Exception' );
	43	+ $this->setExpectedException('\Aimeos\MW\Common\Exception');
44	44
45		- $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' )
46		- ->setConstructorArgs( [$stub, $this->context] )
	45	+ $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Decorator\Base')
	46	+ ->setConstructorArgs([$stub, $this->context])
47	47	->getMockForAbstractClass();
48	48	}
49	49
50	50
51	51	public function testCall()
52	52	{
53		- $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Standard' )
	53	+ $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Standard')
54	54	->disableOriginalConstructor()
55		- ->setMethods( ['invalid'] )
	55	+ ->setMethods(['invalid'])
56	56	->getMock();
57	57
58		- $object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' )
59		- ->setConstructorArgs( [$stub, $this->context] )
	58	+ $object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Decorator\Base')
	59	+ ->setConstructorArgs([$stub, $this->context])
60	60	->getMockForAbstractClass();
61	61
62		- $stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
	62	+ $stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
63	63
64		- $this->assertTrue( $object->invalid() );
	64	+ $this->assertTrue($object->invalid());
65	65	}
66	66
67	67
68	68	public function testAddFilterTypes()
69	69	{
70		- $search = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createSearch();
	70	+ $search = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createSearch();
71	71
72		- $this->stub->expects( $this->once() )->method( 'addFilterTypes' )
73		- ->will( $this->returnArgument( 0 ) );
	72	+ $this->stub->expects($this->once())->method('addFilterTypes')
	73	+ ->will($this->returnArgument(0));
74	74
75		- $this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->addFilterTypes( $search, [] ) );
	75	+ $this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->addFilterTypes($search, []));
76	76	}
77	77
78	78
79	79	public function testCreateFilter()
80	80	{
81		- $search = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createSearch();
	81	+ $search = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createSearch();
82	82
83		- $this->stub->expects( $this->once() )->method( 'createFilter' )
84		- ->will( $this->returnValue( $search ) );
	83	+ $this->stub->expects($this->once())->method('createFilter')
	84	+ ->will($this->returnValue($search));
85	85
86		- $this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->createFilter() );
	86	+ $this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->createFilter());
87	87	}
88	88
89	89
90	90	public function testGetItem()
91	91	{
92		- $item = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createItem();
	92	+ $item = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createItem();
93	93
94		- $this->stub->expects( $this->once() )->method( 'getItem' )
95		- ->will( $this->returnValue( $item ) );
	94	+ $this->stub->expects($this->once())->method('getItem')
	95	+ ->will($this->returnValue($item));
96	96
97		- $this->assertInstanceOf( '\Aimeos\MShop\Attribute\Item\Iface', $this->object->getItem( -1 ) );
	97	+ $this->assertInstanceOf('\Aimeos\MShop\Attribute\Item\Iface', $this->object->getItem( -1 ));
98	98	}
99	99
100	100
101	101	public function testGetItems()
102	102	{
103		- $this->stub->expects( $this->once() )->method( 'getItems' )
104		- ->will( $this->returnValue( [] ) );
	103	+ $this->stub->expects($this->once())->method('getItems')
	104	+ ->will($this->returnValue([]));
105	105
106		- $this->assertEquals( [], $this->object->getItems( [-1], ['media'] ) );
	106	+ $this->assertEquals([], $this->object->getItems([-1], ['media']));
107	107	}
108	108
109	109
110	110	public function testSearchItems()
111	111	{
112		- $filter = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createSearch();
	112	+ $filter = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createSearch();
113	113
114		- $this->stub->expects( $this->once() )->method( 'searchItems' )
115		- ->will( $this->returnValue( [] ) );
	114	+ $this->stub->expects($this->once())->method('searchItems')
	115	+ ->will($this->returnValue([]));
116	116
117		- $this->assertEquals( [], $this->object->searchItems( $filter, ['media'] ) );
	117	+ $this->assertEquals([], $this->object->searchItems($filter, ['media']));
118	118	}
119	119
120	120
121	121	public function testGetController()
122	122	{
123		- $result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
	123	+ $result = $this->access('getController')->invokeArgs($this->object, []);
124	124
125		- $this->assertSame( $this->stub, $result );
	125	+ $this->assertSame($this->stub, $result);
126	126	}
127	127
128	128
129		- protected function access( $name )
	129	+ protected function access($name)
130	130	{
131		- $class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' );
132		- $method = $class->getMethod( $name );
133		- $method->setAccessible( true );
	131	+ $class = new \ReflectionClass('\Aimeos\Controller\Frontend\Attribute\Decorator\Base');
	132	+ $method = $class->getMethod($name);
	133	+ $method->setAccessible(true);
134	134
135	135	return $method;
136	136	}

		@@ -15,31 +15,31 @@
		block discarded – undo
15	15	{
16	16	$target = '\\Aimeos\\Controller\\Frontend\\Stock\\Iface';
17	17
18		- $controller = \Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext() );
19		- $this->assertInstanceOf( $target, $controller );
	18	+ $controller = \Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext());
	19	+ $this->assertInstanceOf($target, $controller);
20	20
21		- $controller = \Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
22		- $this->assertInstanceOf( $target, $controller );
	21	+ $controller = \Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
	22	+ $this->assertInstanceOf($target, $controller);
23	23	}
24	24
25	25
26	26	public function testCreateControllerInvalidImplementation()
27	27	{
28		- $this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
29		- \Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
	28	+ $this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
	29	+ \Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
30	30	}
31	31
32	32
33	33	public function testCreateControllerInvalidName()
34	34	{
35		- $this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
36		- \Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
	35	+ $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
	36	+ \Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext(), '%^');
37	37	}
38	38
39	39
40	40	public function testCreateControllerNotExisting()
41	41	{
42		- $this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
43		- \Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
	42	+ $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
	43	+ \Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
44	44	}
45	45	}

		@@ -14,31 +14,31 @@
		block discarded – undo
14	14	{
15	15	$target = '\\Aimeos\\Controller\\Frontend\\Catalog\\Iface';
16	16
17		- $controller = \Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext() );
18		- $this->assertInstanceOf( $target, $controller );
	17	+ $controller = \Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext());
	18	+ $this->assertInstanceOf($target, $controller);
19	19
20		- $controller = \Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
21		- $this->assertInstanceOf( $target, $controller );
	20	+ $controller = \Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
	21	+ $this->assertInstanceOf($target, $controller);
22	22	}
23	23
24	24
25	25	public function testCreateControllerInvalidImplementation()
26	26	{
27		- $this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
28		- \Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
	27	+ $this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
	28	+ \Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
29	29	}
30	30
31	31
32	32	public function testCreateControllerInvalidName()
33	33	{
34		- $this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
35		- \Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
	34	+ $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
	35	+ \Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext(), '%^');
36	36	}
37	37
38	38
39	39	public function testCreateControllerNotExisting()
40	40	{
41		- $this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
42		- \Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
	41	+ $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
	42	+ \Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
43	43	}
44	44	}

		@@ -20,54 +20,54 @@ discard block
		block discarded – undo
20	20	{
21	21	$this->context = \TestHelperFrontend::getContext();
22	22
23		- $this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
	23	+ $this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
24	24	->disableOriginalConstructor()
25	25	->getMock();
26	26
27		- $this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Decorator\Base' )
28		- ->setConstructorArgs( [$this->stub, $this->context] )
	27	+ $this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Decorator\Base')
	28	+ ->setConstructorArgs([$this->stub, $this->context])
29	29	->getMockForAbstractClass();
30	30	}
31	31
32	32
33	33	protected function tearDown()
34	34	{
35		- unset( $this->context, $this->object, $this->stub );
	35	+ unset($this->context, $this->object, $this->stub);
36	36	}
37	37
38	38
39	39	public function testConstructException()
40	40	{
41		- $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Iface' )->getMock();
	41	+ $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock();
42	42
43		- $this->setExpectedException( '\Aimeos\MW\Common\Exception' );
	43	+ $this->setExpectedException('\Aimeos\MW\Common\Exception');
44	44
45		- $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Decorator\Base' )
46		- ->setConstructorArgs( [$stub, $this->context] )
	45	+ $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Decorator\Base')
	46	+ ->setConstructorArgs([$stub, $this->context])
47	47	->getMockForAbstractClass();
48	48	}
49	49
50	50
51	51	public function testCall()
52	52	{
53		- $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
	53	+ $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
54	54	->disableOriginalConstructor()
55		- ->setMethods( ['invalid'] )
	55	+ ->setMethods(['invalid'])
56	56	->getMock();
57	57
58		- $object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Decorator\Base' )
59		- ->setConstructorArgs( [$stub, $this->context] )
	58	+ $object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Decorator\Base')
	59	+ ->setConstructorArgs([$stub, $this->context])
60	60	->getMockForAbstractClass();
61	61
62		- $stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
	62	+ $stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
63	63
64		- $this->assertTrue( $object->invalid() );
	64	+ $this->assertTrue($object->invalid());
65	65	}
66	66
67	67
68	68	public function testClear()
69	69	{
70		- $this->stub->expects( $this->once() )->method( 'clear' );
	70	+ $this->stub->expects($this->once())->method('clear');
71	71
72	72	$this->object->clear();
73	73	}
		@@ -76,18 +76,18 @@ discard block
		block discarded – undo
76	76	public function testGet()
77	77	{
78	78	$context = \TestHelperFrontend::getContext();
79		- $order = \Aimeos\MShop\Factory::createManager( $context, 'order/base' )->createItem();
	79	+ $order = \Aimeos\MShop\Factory::createManager($context, 'order/base')->createItem();
80	80
81		- $this->stub->expects( $this->once() )->method( 'get' )
82		- ->will( $this->returnValue( $order ) );
	81	+ $this->stub->expects($this->once())->method('get')
	82	+ ->will($this->returnValue($order));
83	83
84		- $this->assertInstanceOf( '\Aimeos\MShop\Order\Item\Base\Iface', $this->object->get() );
	84	+ $this->assertInstanceOf('\Aimeos\MShop\Order\Item\Base\Iface', $this->object->get());
85	85	}
86	86
87	87
88	88	public function testSave()
89	89	{
90		- $this->stub->expects( $this->once() )->method( 'save' );
	90	+ $this->stub->expects($this->once())->method('save');
91	91
92	92	$this->object->save();
93	93	}
		@@ -95,15 +95,15 @@ discard block
		block discarded – undo
95	95
96	96	public function testSetType()
97	97	{
98		- $this->stub->expects( $this->once() )->method( 'setType' );
	98	+ $this->stub->expects($this->once())->method('setType');
99	99
100		- $this->object->setType( 'test' );
	100	+ $this->object->setType('test');
101	101	}
102	102
103	103
104	104	public function testStore()
105	105	{
106		- $this->stub->expects( $this->once() )->method( 'store' );
	106	+ $this->stub->expects($this->once())->method('store');
107	107
108	108	$this->object->store();
109	109	}
		@@ -111,7 +111,7 @@ discard block
		block discarded – undo
111	111
112	112	public function testLoad()
113	113	{
114		- $this->stub->expects( $this->once() )->method( 'load' );
	114	+ $this->stub->expects($this->once())->method('load');
115	115
116	116	$this->object->load( -1 );
117	117	}
		@@ -119,7 +119,7 @@ discard block
		block discarded – undo
119	119
120	120	public function testAddProduct()
121	121	{
122		- $this->stub->expects( $this->once() )->method( 'addProduct' );
	122	+ $this->stub->expects($this->once())->method('addProduct');
123	123
124	124	$this->object->addProduct( -1 );
125	125	}
		@@ -127,73 +127,73 @@ discard block
		block discarded – undo
127	127
128	128	public function testDeleteProduct()
129	129	{
130		- $this->stub->expects( $this->once() )->method( 'deleteProduct' );
	130	+ $this->stub->expects($this->once())->method('deleteProduct');
131	131
132		- $this->object->deleteProduct( 0 );
	132	+ $this->object->deleteProduct(0);
133	133	}
134	134
135	135
136	136	public function testEditProduct()
137	137	{
138		- $this->stub->expects( $this->once() )->method( 'editProduct' );
	138	+ $this->stub->expects($this->once())->method('editProduct');
139	139
140		- $this->object->editProduct( 0, 1 );
	140	+ $this->object->editProduct(0, 1);
141	141	}
142	142
143	143
144	144	public function testAddCoupon()
145	145	{
146		- $this->stub->expects( $this->once() )->method( 'addCoupon' );
	146	+ $this->stub->expects($this->once())->method('addCoupon');
147	147
148		- $this->object->addCoupon( 'test' );
	148	+ $this->object->addCoupon('test');
149	149	}
150	150
151	151
152	152	public function testDeleteCoupon()
153	153	{
154		- $this->stub->expects( $this->once() )->method( 'deleteCoupon' );
	154	+ $this->stub->expects($this->once())->method('deleteCoupon');
155	155
156		- $this->object->deleteCoupon( 'test' );
	156	+ $this->object->deleteCoupon('test');
157	157	}
158	158
159	159
160	160	public function testSetAddress()
161	161	{
162		- $this->stub->expects( $this->once() )->method( 'setAddress' );
	162	+ $this->stub->expects($this->once())->method('setAddress');
163	163
164		- $this->object->setAddress( \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT, null );
	164	+ $this->object->setAddress(\Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT, null);
165	165	}
166	166
167	167
168	168	public function testAddService()
169	169	{
170		- $this->stub->expects( $this->once() )->method( 'addService' );
	170	+ $this->stub->expects($this->once())->method('addService');
171	171
172		- $this->object->addService( \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT, -1 );
	172	+ $this->object->addService(\Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT, -1);
173	173	}
174	174
175	175
176	176	public function testDeleteService()
177	177	{
178		- $this->stub->expects( $this->once() )->method( 'deleteService' );
	178	+ $this->stub->expects($this->once())->method('deleteService');
179	179
180		- $this->object->deleteService( \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT );
	180	+ $this->object->deleteService(\Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT);
181	181	}
182	182
183	183
184	184	public function testGetController()
185	185	{
186		- $result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
	186	+ $result = $this->access('getController')->invokeArgs($this->object, []);
187	187
188		- $this->assertSame( $this->stub, $result );
	188	+ $this->assertSame($this->stub, $result);
189	189	}
190	190
191	191
192		- protected function access( $name )
	192	+ protected function access($name)
193	193	{
194		- $class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Basket\Decorator\Base' );
195		- $method = $class->getMethod( $name );
196		- $method->setAccessible( true );
	194	+ $class = new \ReflectionClass('\Aimeos\Controller\Frontend\Basket\Decorator\Base');
	195	+ $method = $class->getMethod($name);
	196	+ $method->setAccessible(true);
197	197
198	198	return $method;
199	199	}

aimeos / ai-controller-frontend

Push — master ( 866fb8...8bb600 )

Status

Category

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

Spacing +35 added lines, -35 removed lines patch added patch discarded remove patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

Spacing +31 added lines, -31 removed lines patch added patch discarded remove patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

Spacing +39 added lines, -39 removed lines patch added patch discarded remove patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

Spacing +45 added lines, -45 removed lines patch added patch discarded remove patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch