@@ -20,117 +20,117 @@ |
||
| 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 @@ |
||
| 15 | 15 | { |
| 16 | 16 | $target = '\\Aimeos\\Controller\\Frontend\\Attribute\\Iface'; |
| 17 | 17 | |
| 18 | - $controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext() ); |
|
| 19 | - $this->assertInstanceOf( $target, $controller ); |
|
| 18 | + $controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext()); |
|
| 19 | + $this->assertInstanceOf($target, $controller); |
|
| 20 | 20 | |
| 21 | - $controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' ); |
|
| 22 | - $this->assertInstanceOf( $target, $controller ); |
|
| 21 | + $controller = \Aimeos\Controller\Frontend\Attribute\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\Attribute\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' ); |
|
| 28 | + $this->setExpectedException('\\Aimeos\\MW\\Common\\Exception'); |
|
| 29 | + \Aimeos\Controller\Frontend\Attribute\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\Attribute\Factory::createController( \TestHelperFrontend::getContext(), '%^' ); |
|
| 35 | + $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception'); |
|
| 36 | + \Aimeos\Controller\Frontend\Attribute\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\Attribute\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' ); |
|
| 42 | + $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception'); |
|
| 43 | + \Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext(), 'notexist'); |
|
| 44 | 44 | } |
| 45 | 45 | } |
@@ -20,54 +20,54 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 | } |
@@ -14,31 +14,31 @@ |
||
| 14 | 14 | { |
| 15 | 15 | $target = '\\Aimeos\\Controller\\Frontend\\Basket\\Iface'; |
| 16 | 16 | |
| 17 | - $controller = \Aimeos\Controller\Frontend\Basket\Factory::createController( \TestHelperFrontend::getContext() ); |
|
| 18 | - $this->assertInstanceOf( $target, $controller ); |
|
| 17 | + $controller = \Aimeos\Controller\Frontend\Basket\Factory::createController(\TestHelperFrontend::getContext()); |
|
| 18 | + $this->assertInstanceOf($target, $controller); |
|
| 19 | 19 | |
| 20 | - $controller = \Aimeos\Controller\Frontend\Basket\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' ); |
|
| 21 | - $this->assertInstanceOf( $target, $controller ); |
|
| 20 | + $controller = \Aimeos\Controller\Frontend\Basket\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\Basket\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' ); |
|
| 27 | + $this->setExpectedException('\\Aimeos\\MW\\Common\\Exception'); |
|
| 28 | + \Aimeos\Controller\Frontend\Basket\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\Basket\Factory::createController( \TestHelperFrontend::getContext(), '%^' ); |
|
| 34 | + $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception'); |
|
| 35 | + \Aimeos\Controller\Frontend\Basket\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\Basket\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' ); |
|
| 41 | + $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception'); |
|
| 42 | + \Aimeos\Controller\Frontend\Basket\Factory::createController(\TestHelperFrontend::getContext(), 'notexist'); |
|
| 43 | 43 | } |
| 44 | 44 | } |