@@ -26,7 +26,7 @@ discard block |
||
| 26 | 26 | * @return \Aimeos\Controller\Frontend\Attribute\Iface Attribute controller for fluent interface |
| 27 | 27 | * @since 2019.04 |
| 28 | 28 | */ |
| 29 | - public function attribute( $attrIds ); |
|
| 29 | + public function attribute($attrIds); |
|
| 30 | 30 | |
| 31 | 31 | /** |
| 32 | 32 | * Adds generic condition for filtering attributes |
@@ -37,7 +37,7 @@ discard block |
||
| 37 | 37 | * @return \Aimeos\Controller\Frontend\Attribute\Iface Attribute controller for fluent interface |
| 38 | 38 | * @since 2019.04 |
| 39 | 39 | */ |
| 40 | - public function compare( $operator, $key, $value ); |
|
| 40 | + public function compare($operator, $key, $value); |
|
| 41 | 41 | |
| 42 | 42 | /** |
| 43 | 43 | * Adds the domain of the attributes for filtering |
@@ -46,7 +46,7 @@ discard block |
||
| 46 | 46 | * @return \Aimeos\Controller\Frontend\Attribute\Iface Attribute controller for fluent interface |
| 47 | 47 | * @since 2019.04 |
| 48 | 48 | */ |
| 49 | - public function domain( $domain ); |
|
| 49 | + public function domain($domain); |
|
| 50 | 50 | |
| 51 | 51 | /** |
| 52 | 52 | * Returns the attribute for the given attribute ID |
@@ -56,7 +56,7 @@ discard block |
||
| 56 | 56 | * @return \Aimeos\MShop\Attribute\Item\Iface Attribute item including the referenced domains items |
| 57 | 57 | * @since 2019.04 |
| 58 | 58 | */ |
| 59 | - public function get( $id, $domains = ['media', 'price', 'text'] ); |
|
| 59 | + public function get($id, $domains = ['media', 'price', 'text']); |
|
| 60 | 60 | |
| 61 | 61 | /** |
| 62 | 62 | * Returns the attribute for the given attribute code |
@@ -67,7 +67,7 @@ discard block |
||
| 67 | 67 | * @return \Aimeos\MShop\Attribute\Item\Iface Attribute item including the referenced domains items |
| 68 | 68 | * @since 2019.04 |
| 69 | 69 | */ |
| 70 | - public function find( $code, $domains = ['media', 'price', 'text'], $type ); |
|
| 70 | + public function find($code, $domains = ['media', 'price', 'text'], $type); |
|
| 71 | 71 | |
| 72 | 72 | /** |
| 73 | 73 | * Parses the given array and adds the conditions to the list of conditions |
@@ -76,7 +76,7 @@ discard block |
||
| 76 | 76 | * @return \Aimeos\Controller\Frontend\Attribute\Iface Attribute controller for fluent interface |
| 77 | 77 | * @since 2019.04 |
| 78 | 78 | */ |
| 79 | - public function parse( array $conditions ); |
|
| 79 | + public function parse(array $conditions); |
|
| 80 | 80 | |
| 81 | 81 | /** |
| 82 | 82 | * Returns the attributes filtered by the previously assigned conditions |
@@ -86,7 +86,7 @@ discard block |
||
| 86 | 86 | * @return \Aimeos\MShop\Attribute\Item\Iface[] Ordered list of attribute items |
| 87 | 87 | * @since 2019.04 |
| 88 | 88 | */ |
| 89 | - public function search( $domains = ['media', 'price', 'text'], &$total = null ); |
|
| 89 | + public function search($domains = ['media', 'price', 'text'], &$total = null); |
|
| 90 | 90 | |
| 91 | 91 | /** |
| 92 | 92 | * Sets the start value and the number of returned attributes for slicing the list of found attributes |
@@ -96,7 +96,7 @@ discard block |
||
| 96 | 96 | * @return \Aimeos\Controller\Frontend\Attribute\Iface Attribute controller for fluent interface |
| 97 | 97 | * @since 2019.04 |
| 98 | 98 | */ |
| 99 | - public function slice( $start, $limit ); |
|
| 99 | + public function slice($start, $limit); |
|
| 100 | 100 | |
| 101 | 101 | /** |
| 102 | 102 | * Sets the sorting of the result list |
@@ -105,7 +105,7 @@ discard block |
||
| 105 | 105 | * @return \Aimeos\Controller\Frontend\Attribute\Iface Attribute controller for fluent interface |
| 106 | 106 | * @since 2019.04 |
| 107 | 107 | */ |
| 108 | - public function sort( $key = null ); |
|
| 108 | + public function sort($key = null); |
|
| 109 | 109 | |
| 110 | 110 | /** |
| 111 | 111 | * Adds attribute types for filtering |
@@ -114,5 +114,5 @@ discard block |
||
| 114 | 114 | * @return \Aimeos\Controller\Frontend\Attribute\Iface Attribute controller for fluent interface |
| 115 | 115 | * @since 2019.04 |
| 116 | 116 | */ |
| 117 | - public function type( $codes ); |
|
| 117 | + public function type($codes); |
|
| 118 | 118 | } |
@@ -20,59 +20,59 @@ discard block |
||
| 20 | 20 | { |
| 21 | 21 | $this->context = \TestHelperFrontend::getContext(); |
| 22 | 22 | |
| 23 | - $this->stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Subscription\Standard::class ) |
|
| 23 | + $this->stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Subscription\Standard::class) |
|
| 24 | 24 | ->disableOriginalConstructor() |
| 25 | 25 | ->getMock(); |
| 26 | 26 | |
| 27 | - $this->object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Subscription\Decorator\Base::class ) |
|
| 28 | - ->setConstructorArgs( [$this->stub, $this->context] ) |
|
| 27 | + $this->object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Subscription\Decorator\Base::class) |
|
| 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::class )->getMock(); |
|
| 41 | + $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Iface::class)->getMock(); |
|
| 42 | 42 | |
| 43 | - $this->setExpectedException( \Aimeos\MW\Common\Exception::class ); |
|
| 43 | + $this->setExpectedException(\Aimeos\MW\Common\Exception::class); |
|
| 44 | 44 | |
| 45 | - $this->getMockBuilder( \Aimeos\Controller\Frontend\Subscription\Decorator\Base::class ) |
|
| 46 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
| 45 | + $this->getMockBuilder(\Aimeos\Controller\Frontend\Subscription\Decorator\Base::class) |
|
| 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\Subscription\Standard::class ) |
|
| 53 | + $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Subscription\Standard::class) |
|
| 54 | 54 | ->disableOriginalConstructor() |
| 55 | - ->setMethods( ['invalid'] ) |
|
| 55 | + ->setMethods(['invalid']) |
|
| 56 | 56 | ->getMock(); |
| 57 | 57 | |
| 58 | - $object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Subscription\Decorator\Base::class ) |
|
| 59 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
| 58 | + $object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Subscription\Decorator\Base::class) |
|
| 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 testCancel() |
| 69 | 69 | { |
| 70 | - $item = \Aimeos\MShop::create( $this->context, 'subscription' )->createItem(); |
|
| 70 | + $item = \Aimeos\MShop::create($this->context, 'subscription')->createItem(); |
|
| 71 | 71 | |
| 72 | - $this->stub->expects( $this->once() )->method( 'cancel' ) |
|
| 73 | - ->will( $this->returnValue( $item ) ); |
|
| 72 | + $this->stub->expects($this->once())->method('cancel') |
|
| 73 | + ->will($this->returnValue($item)); |
|
| 74 | 74 | |
| 75 | - $this->assertInstanceOf( \Aimeos\MShop\Subscription\Item\Iface::class, $this->object->cancel( -1 ) ); |
|
| 75 | + $this->assertInstanceOf(\Aimeos\MShop\Subscription\Item\Iface::class, $this->object->cancel( -1 )); |
|
| 76 | 76 | } |
| 77 | 77 | |
| 78 | 78 | |
@@ -80,30 +80,30 @@ discard block |
||
| 80 | 80 | { |
| 81 | 81 | $expected = \Aimeos\Controller\Frontend\Subscription\Iface::class; |
| 82 | 82 | |
| 83 | - $this->stub->expects( $this->once() )->method( 'compare' ) |
|
| 84 | - ->will( $this->returnValue( $this->stub ) ); |
|
| 83 | + $this->stub->expects($this->once())->method('compare') |
|
| 84 | + ->will($this->returnValue($this->stub)); |
|
| 85 | 85 | |
| 86 | - $this->assertInstanceOf( $expected, $this->object->compare( '==', 'supplier.status', 1 ) ); |
|
| 86 | + $this->assertInstanceOf($expected, $this->object->compare('==', 'supplier.status', 1)); |
|
| 87 | 87 | } |
| 88 | 88 | |
| 89 | 89 | |
| 90 | 90 | public function testGet() |
| 91 | 91 | { |
| 92 | - $item = \Aimeos\MShop::create( $this->context, 'subscription' )->createItem(); |
|
| 92 | + $item = \Aimeos\MShop::create($this->context, 'subscription')->createItem(); |
|
| 93 | 93 | |
| 94 | - $this->stub->expects( $this->once() )->method( 'get' ) |
|
| 95 | - ->will( $this->returnValue( $item ) ); |
|
| 94 | + $this->stub->expects($this->once())->method('get') |
|
| 95 | + ->will($this->returnValue($item)); |
|
| 96 | 96 | |
| 97 | - $this->assertInstanceOf( \Aimeos\MShop\Subscription\Item\Iface::class, $this->object->get( -1 ) ); |
|
| 97 | + $this->assertInstanceOf(\Aimeos\MShop\Subscription\Item\Iface::class, $this->object->get( -1 )); |
|
| 98 | 98 | } |
| 99 | 99 | |
| 100 | 100 | |
| 101 | 101 | public function testGetIntervals() |
| 102 | 102 | { |
| 103 | - $this->stub->expects( $this->once() )->method( 'getIntervals' ) |
|
| 104 | - ->will( $this->returnValue( [] ) ); |
|
| 103 | + $this->stub->expects($this->once())->method('getIntervals') |
|
| 104 | + ->will($this->returnValue([])); |
|
| 105 | 105 | |
| 106 | - $this->assertEquals( [], $this->object->getIntervals() ); |
|
| 106 | + $this->assertEquals([], $this->object->getIntervals()); |
|
| 107 | 107 | } |
| 108 | 108 | |
| 109 | 109 | |
@@ -111,32 +111,32 @@ discard block |
||
| 111 | 111 | { |
| 112 | 112 | $expected = \Aimeos\Controller\Frontend\Subscription\Iface::class; |
| 113 | 113 | |
| 114 | - $this->stub->expects( $this->once() )->method( 'parse' ) |
|
| 115 | - ->will( $this->returnValue( $this->stub ) ); |
|
| 114 | + $this->stub->expects($this->once())->method('parse') |
|
| 115 | + ->will($this->returnValue($this->stub)); |
|
| 116 | 116 | |
| 117 | - $this->assertInstanceOf( $expected, $this->object->parse( [] ) ); |
|
| 117 | + $this->assertInstanceOf($expected, $this->object->parse([])); |
|
| 118 | 118 | } |
| 119 | 119 | |
| 120 | 120 | |
| 121 | 121 | public function testSave() |
| 122 | 122 | { |
| 123 | - $item = \Aimeos\MShop::create( $this->context, 'subscription' )->createItem(); |
|
| 123 | + $item = \Aimeos\MShop::create($this->context, 'subscription')->createItem(); |
|
| 124 | 124 | |
| 125 | - $this->stub->expects( $this->once() )->method( 'save' ) |
|
| 126 | - ->will( $this->returnValue( $item ) ); |
|
| 125 | + $this->stub->expects($this->once())->method('save') |
|
| 126 | + ->will($this->returnValue($item)); |
|
| 127 | 127 | |
| 128 | - $this->assertInstanceOf( \Aimeos\MShop\Subscription\Item\Iface::class, $this->object->save( $item ) ); |
|
| 128 | + $this->assertInstanceOf(\Aimeos\MShop\Subscription\Item\Iface::class, $this->object->save($item)); |
|
| 129 | 129 | } |
| 130 | 130 | |
| 131 | 131 | |
| 132 | 132 | public function testSearch() |
| 133 | 133 | { |
| 134 | - $item = \Aimeos\MShop::create( $this->context, 'subscription' )->createItem(); |
|
| 134 | + $item = \Aimeos\MShop::create($this->context, 'subscription')->createItem(); |
|
| 135 | 135 | |
| 136 | - $this->stub->expects( $this->once() )->method( 'search' ) |
|
| 137 | - ->will( $this->returnValue( [$item] ) ); |
|
| 136 | + $this->stub->expects($this->once())->method('search') |
|
| 137 | + ->will($this->returnValue([$item])); |
|
| 138 | 138 | |
| 139 | - $this->assertEquals( [$item], $this->object->search() ); |
|
| 139 | + $this->assertEquals([$item], $this->object->search()); |
|
| 140 | 140 | } |
| 141 | 141 | |
| 142 | 142 | |
@@ -144,10 +144,10 @@ discard block |
||
| 144 | 144 | { |
| 145 | 145 | $expected = \Aimeos\Controller\Frontend\Subscription\Iface::class; |
| 146 | 146 | |
| 147 | - $this->stub->expects( $this->once() )->method( 'slice' ) |
|
| 148 | - ->will( $this->returnValue( $this->stub ) ); |
|
| 147 | + $this->stub->expects($this->once())->method('slice') |
|
| 148 | + ->will($this->returnValue($this->stub)); |
|
| 149 | 149 | |
| 150 | - $this->assertInstanceOf( $expected, $this->object->slice( 0, 100 ) ); |
|
| 150 | + $this->assertInstanceOf($expected, $this->object->slice(0, 100)); |
|
| 151 | 151 | } |
| 152 | 152 | |
| 153 | 153 | |
@@ -155,26 +155,26 @@ discard block |
||
| 155 | 155 | { |
| 156 | 156 | $expected = \Aimeos\Controller\Frontend\Subscription\Iface::class; |
| 157 | 157 | |
| 158 | - $this->stub->expects( $this->once() )->method( 'sort' ) |
|
| 159 | - ->will( $this->returnValue( $this->stub ) ); |
|
| 158 | + $this->stub->expects($this->once())->method('sort') |
|
| 159 | + ->will($this->returnValue($this->stub)); |
|
| 160 | 160 | |
| 161 | - $this->assertInstanceOf( $expected, $this->object->sort( 'interval' ) ); |
|
| 161 | + $this->assertInstanceOf($expected, $this->object->sort('interval')); |
|
| 162 | 162 | } |
| 163 | 163 | |
| 164 | 164 | |
| 165 | 165 | public function testGetController() |
| 166 | 166 | { |
| 167 | - $result = $this->access( 'getController' )->invokeArgs( $this->object, [] ); |
|
| 167 | + $result = $this->access('getController')->invokeArgs($this->object, []); |
|
| 168 | 168 | |
| 169 | - $this->assertSame( $this->stub, $result ); |
|
| 169 | + $this->assertSame($this->stub, $result); |
|
| 170 | 170 | } |
| 171 | 171 | |
| 172 | 172 | |
| 173 | - protected function access( $name ) |
|
| 173 | + protected function access($name) |
|
| 174 | 174 | { |
| 175 | - $class = new \ReflectionClass( \Aimeos\Controller\Frontend\Subscription\Decorator\Base::class ); |
|
| 176 | - $method = $class->getMethod( $name ); |
|
| 177 | - $method->setAccessible( true ); |
|
| 175 | + $class = new \ReflectionClass(\Aimeos\Controller\Frontend\Subscription\Decorator\Base::class); |
|
| 176 | + $method = $class->getMethod($name); |
|
| 177 | + $method->setAccessible(true); |
|
| 178 | 178 | |
| 179 | 179 | return $method; |
| 180 | 180 | } |
@@ -19,69 +19,69 @@ discard block |
||
| 19 | 19 | protected function setUp() |
| 20 | 20 | { |
| 21 | 21 | $this->context = \TestHelperFrontend::getContext(); |
| 22 | - $this->context->setUserId( $this->getCustomerId() ); |
|
| 22 | + $this->context->setUserId($this->getCustomerId()); |
|
| 23 | 23 | |
| 24 | - $this->manager = $this->getMockBuilder( '\\Aimeos\\MShop\\Subscription\\Manager\\Standard' ) |
|
| 25 | - ->setConstructorArgs( [$this->context] ) |
|
| 26 | - ->setMethods( ['saveItem'] ) |
|
| 24 | + $this->manager = $this->getMockBuilder('\\Aimeos\\MShop\\Subscription\\Manager\\Standard') |
|
| 25 | + ->setConstructorArgs([$this->context]) |
|
| 26 | + ->setMethods(['saveItem']) |
|
| 27 | 27 | ->getMock(); |
| 28 | 28 | |
| 29 | - \Aimeos\MShop::cache( true ); |
|
| 30 | - \Aimeos\MShop::inject( 'subscription', $this->manager ); |
|
| 29 | + \Aimeos\MShop::cache(true); |
|
| 30 | + \Aimeos\MShop::inject('subscription', $this->manager); |
|
| 31 | 31 | |
| 32 | - $this->object = new \Aimeos\Controller\Frontend\Subscription\Standard( $this->context ); |
|
| 32 | + $this->object = new \Aimeos\Controller\Frontend\Subscription\Standard($this->context); |
|
| 33 | 33 | } |
| 34 | 34 | |
| 35 | 35 | |
| 36 | 36 | protected function tearDown() |
| 37 | 37 | { |
| 38 | - \Aimeos\MShop::cache( false ); |
|
| 39 | - unset( $this->object, $this->manager, $this->context ); |
|
| 38 | + \Aimeos\MShop::cache(false); |
|
| 39 | + unset($this->object, $this->manager, $this->context); |
|
| 40 | 40 | } |
| 41 | 41 | |
| 42 | 42 | |
| 43 | 43 | public function testCancel() |
| 44 | 44 | { |
| 45 | 45 | $expected = \Aimeos\MShop\Subscription\Item\Iface::class; |
| 46 | - $item = \Aimeos\MShop::create( $this->context, 'subscription' )->createItem(); |
|
| 46 | + $item = \Aimeos\MShop::create($this->context, 'subscription')->createItem(); |
|
| 47 | 47 | |
| 48 | - $this->manager->expects( $this->once() )->method( 'saveItem' ) |
|
| 49 | - ->will( $this->returnValue( $item ) ); |
|
| 48 | + $this->manager->expects($this->once())->method('saveItem') |
|
| 49 | + ->will($this->returnValue($item)); |
|
| 50 | 50 | |
| 51 | - $this->assertInstanceOf( $expected, $this->object->cancel( $this->getSubscriptionId() ) ); |
|
| 51 | + $this->assertInstanceOf($expected, $this->object->cancel($this->getSubscriptionId())); |
|
| 52 | 52 | } |
| 53 | 53 | |
| 54 | 54 | |
| 55 | 55 | public function testCompare() |
| 56 | 56 | { |
| 57 | - $this->assertEquals( 2, count( $this->object->compare( '>=', 'subscription.datenext', '2000-01-01' )->search() ) ); |
|
| 57 | + $this->assertEquals(2, count($this->object->compare('>=', 'subscription.datenext', '2000-01-01')->search())); |
|
| 58 | 58 | } |
| 59 | 59 | |
| 60 | 60 | |
| 61 | 61 | public function testGet() |
| 62 | 62 | { |
| 63 | 63 | $expected = \Aimeos\MShop\Subscription\Item\Iface::class; |
| 64 | - $this->assertInstanceOf( $expected, $this->object->get( $this->getSubscriptionId() ) ); |
|
| 64 | + $this->assertInstanceOf($expected, $this->object->get($this->getSubscriptionId())); |
|
| 65 | 65 | } |
| 66 | 66 | |
| 67 | 67 | |
| 68 | 68 | public function testGetException() |
| 69 | 69 | { |
| 70 | - $this->setExpectedException( \Aimeos\Controller\Frontend\Subscription\Exception::class ); |
|
| 70 | + $this->setExpectedException(\Aimeos\Controller\Frontend\Subscription\Exception::class); |
|
| 71 | 71 | $this->object->get( -1 ); |
| 72 | 72 | } |
| 73 | 73 | |
| 74 | 74 | |
| 75 | 75 | public function testGetIntervals() |
| 76 | 76 | { |
| 77 | - $this->assertGreaterThan( 0, count( $this->object->getIntervals() ) ); |
|
| 77 | + $this->assertGreaterThan(0, count($this->object->getIntervals())); |
|
| 78 | 78 | } |
| 79 | 79 | |
| 80 | 80 | |
| 81 | 81 | public function testParse() |
| 82 | 82 | { |
| 83 | 83 | $cond = ['&&' => [['==' => ['subscription.datenext' => '2000-01-01']], ['==' => ['subscription.status' => 1]]]]; |
| 84 | - $this->assertEquals( 1, count( $this->object->parse( $cond )->search() ) ); |
|
| 84 | + $this->assertEquals(1, count($this->object->parse($cond)->search())); |
|
| 85 | 85 | } |
| 86 | 86 | |
| 87 | 87 | |
@@ -90,60 +90,60 @@ discard block |
||
| 90 | 90 | $item = $this->manager->createItem(); |
| 91 | 91 | $expected = \Aimeos\MShop\Subscription\Item\Iface::class; |
| 92 | 92 | |
| 93 | - $this->manager->expects( $this->once() )->method( 'saveItem' ) |
|
| 94 | - ->will( $this->returnValue( $item ) ); |
|
| 93 | + $this->manager->expects($this->once())->method('saveItem') |
|
| 94 | + ->will($this->returnValue($item)); |
|
| 95 | 95 | |
| 96 | - $this->assertInstanceOf( $expected, $this->object->save( $item ) ); |
|
| 96 | + $this->assertInstanceOf($expected, $this->object->save($item)); |
|
| 97 | 97 | } |
| 98 | 98 | |
| 99 | 99 | |
| 100 | 100 | public function testSearch() |
| 101 | 101 | { |
| 102 | 102 | $total = 0; |
| 103 | - $this->assertGreaterThanOrEqual( 2, count( $this->object->search( $total ) ) ); |
|
| 104 | - $this->assertGreaterThanOrEqual( 2, $total ); |
|
| 103 | + $this->assertGreaterThanOrEqual(2, count($this->object->search($total))); |
|
| 104 | + $this->assertGreaterThanOrEqual(2, $total); |
|
| 105 | 105 | } |
| 106 | 106 | |
| 107 | 107 | |
| 108 | 108 | public function testSlice() |
| 109 | 109 | { |
| 110 | - $this->assertEquals( 1, count( $this->object->slice( 0, 1 )->search() ) ); |
|
| 110 | + $this->assertEquals(1, count($this->object->slice(0, 1)->search())); |
|
| 111 | 111 | } |
| 112 | 112 | |
| 113 | 113 | |
| 114 | 114 | public function testSort() |
| 115 | 115 | { |
| 116 | - $this->assertEquals( 2, count( $this->object->sort()->search() ) ); |
|
| 116 | + $this->assertEquals(2, count($this->object->sort()->search())); |
|
| 117 | 117 | } |
| 118 | 118 | |
| 119 | 119 | |
| 120 | 120 | public function testSortInterval() |
| 121 | 121 | { |
| 122 | - $this->assertEquals( 2, count( $this->object->sort( 'interval' )->search() ) ); |
|
| 122 | + $this->assertEquals(2, count($this->object->sort('interval')->search())); |
|
| 123 | 123 | } |
| 124 | 124 | |
| 125 | 125 | |
| 126 | 126 | public function testSortGeneric() |
| 127 | 127 | { |
| 128 | - $this->assertEquals( 2, count( $this->object->sort( 'subscription.dateend' )->search() ) ); |
|
| 128 | + $this->assertEquals(2, count($this->object->sort('subscription.dateend')->search())); |
|
| 129 | 129 | } |
| 130 | 130 | |
| 131 | 131 | |
| 132 | 132 | protected function getCustomerId() |
| 133 | 133 | { |
| 134 | - return \Aimeos\MShop::create( $this->context, 'customer' )->findItem( 'UTC001' )->getId(); |
|
| 134 | + return \Aimeos\MShop::create($this->context, 'customer')->findItem('UTC001')->getId(); |
|
| 135 | 135 | } |
| 136 | 136 | |
| 137 | 137 | |
| 138 | 138 | protected function getSubscriptionId() |
| 139 | 139 | { |
| 140 | - $manager = \Aimeos\MShop::create( $this->context, 'subscription' ); |
|
| 141 | - $search = $manager->createSearch()->setSlice( 0, 1 ); |
|
| 142 | - $search->setConditions( $search->compare( '==', 'order.base.customerid', $this->context->getUserId() ) ); |
|
| 143 | - $result = $manager->searchItems( $search ); |
|
| 140 | + $manager = \Aimeos\MShop::create($this->context, 'subscription'); |
|
| 141 | + $search = $manager->createSearch()->setSlice(0, 1); |
|
| 142 | + $search->setConditions($search->compare('==', 'order.base.customerid', $this->context->getUserId())); |
|
| 143 | + $result = $manager->searchItems($search); |
|
| 144 | 144 | |
| 145 | - if( ( $item = reset( $result ) ) === false ) { |
|
| 146 | - throw new \RuntimeException( 'No subscription item found' ); |
|
| 145 | + if (($item = reset($result)) === false) { |
|
| 146 | + throw new \RuntimeException('No subscription item found'); |
|
| 147 | 147 | } |
| 148 | 148 | |
| 149 | 149 | return $item->getId(); |
@@ -20,48 +20,48 @@ discard block |
||
| 20 | 20 | { |
| 21 | 21 | $this->context = \TestHelperFrontend::getContext(); |
| 22 | 22 | |
| 23 | - $this->stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Standard::class ) |
|
| 23 | + $this->stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Standard::class) |
|
| 24 | 24 | ->disableOriginalConstructor() |
| 25 | 25 | ->getMock(); |
| 26 | 26 | |
| 27 | - $this->object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Decorator\Base::class ) |
|
| 28 | - ->setConstructorArgs( [$this->stub, $this->context] ) |
|
| 27 | + $this->object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Decorator\Base::class) |
|
| 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::class )->getMock(); |
|
| 41 | + $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Iface::class)->getMock(); |
|
| 42 | 42 | |
| 43 | - $this->setExpectedException( \Aimeos\MW\Common\Exception::class ); |
|
| 43 | + $this->setExpectedException(\Aimeos\MW\Common\Exception::class); |
|
| 44 | 44 | |
| 45 | - $this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Decorator\Base::class ) |
|
| 46 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
| 45 | + $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Decorator\Base::class) |
|
| 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\Stock\Standard::class ) |
|
| 53 | + $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Standard::class) |
|
| 54 | 54 | ->disableOriginalConstructor() |
| 55 | - ->setMethods( ['invalid'] ) |
|
| 55 | + ->setMethods(['invalid']) |
|
| 56 | 56 | ->getMock(); |
| 57 | 57 | |
| 58 | - $object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Decorator\Base::class ) |
|
| 59 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
| 58 | + $object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Decorator\Base::class) |
|
| 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 | |
@@ -69,10 +69,10 @@ discard block |
||
| 69 | 69 | { |
| 70 | 70 | $expected = \Aimeos\Controller\Frontend\Stock\Iface::class; |
| 71 | 71 | |
| 72 | - $this->stub->expects( $this->once() )->method( 'code' ) |
|
| 73 | - ->will( $this->returnValue( $this->stub ) ); |
|
| 72 | + $this->stub->expects($this->once())->method('code') |
|
| 73 | + ->will($this->returnValue($this->stub)); |
|
| 74 | 74 | |
| 75 | - $this->assertInstanceOf( $expected, $this->object->code( ['CNC', 'CNE'] ) ); |
|
| 75 | + $this->assertInstanceOf($expected, $this->object->code(['CNC', 'CNE'])); |
|
| 76 | 76 | } |
| 77 | 77 | |
| 78 | 78 | |
@@ -80,34 +80,34 @@ discard block |
||
| 80 | 80 | { |
| 81 | 81 | $expected = \Aimeos\Controller\Frontend\Stock\Iface::class; |
| 82 | 82 | |
| 83 | - $this->stub->expects( $this->once() )->method( 'compare' ) |
|
| 84 | - ->will( $this->returnValue( $this->stub ) ); |
|
| 83 | + $this->stub->expects($this->once())->method('compare') |
|
| 84 | + ->will($this->returnValue($this->stub)); |
|
| 85 | 85 | |
| 86 | - $this->assertInstanceOf( $expected, $this->object->compare( '==', 'stock.dateback', '2000-01-01 00:00:00' ) ); |
|
| 86 | + $this->assertInstanceOf($expected, $this->object->compare('==', 'stock.dateback', '2000-01-01 00:00:00')); |
|
| 87 | 87 | } |
| 88 | 88 | |
| 89 | 89 | |
| 90 | 90 | public function testGet() |
| 91 | 91 | { |
| 92 | - $item = \Aimeos\MShop::create( $this->context, 'stock' )->createItem(); |
|
| 92 | + $item = \Aimeos\MShop::create($this->context, 'stock')->createItem(); |
|
| 93 | 93 | $expected = \Aimeos\MShop\Stock\Item\Iface::class; |
| 94 | 94 | |
| 95 | - $this->stub->expects( $this->once() )->method( 'get' ) |
|
| 96 | - ->will( $this->returnValue( $item ) ); |
|
| 95 | + $this->stub->expects($this->once())->method('get') |
|
| 96 | + ->will($this->returnValue($item)); |
|
| 97 | 97 | |
| 98 | - $this->assertInstanceOf( $expected, $this->object->get( 1 ) ); |
|
| 98 | + $this->assertInstanceOf($expected, $this->object->get(1)); |
|
| 99 | 99 | } |
| 100 | 100 | |
| 101 | 101 | |
| 102 | 102 | public function testFind() |
| 103 | 103 | { |
| 104 | - $item = \Aimeos\MShop::create( $this->context, 'stock' )->createItem(); |
|
| 104 | + $item = \Aimeos\MShop::create($this->context, 'stock')->createItem(); |
|
| 105 | 105 | $expected = \Aimeos\MShop\Stock\Item\Iface::class; |
| 106 | 106 | |
| 107 | - $this->stub->expects( $this->once() )->method( 'find' ) |
|
| 108 | - ->will( $this->returnValue( $item ) ); |
|
| 107 | + $this->stub->expects($this->once())->method('find') |
|
| 108 | + ->will($this->returnValue($item)); |
|
| 109 | 109 | |
| 110 | - $this->assertInstanceOf( $expected, $this->object->find( 'test', 'default' ) ); |
|
| 110 | + $this->assertInstanceOf($expected, $this->object->find('test', 'default')); |
|
| 111 | 111 | } |
| 112 | 112 | |
| 113 | 113 | |
@@ -115,21 +115,21 @@ discard block |
||
| 115 | 115 | { |
| 116 | 116 | $expected = \Aimeos\Controller\Frontend\Stock\Iface::class; |
| 117 | 117 | |
| 118 | - $this->stub->expects( $this->once() )->method( 'parse' ) |
|
| 119 | - ->will( $this->returnValue( $this->stub ) ); |
|
| 118 | + $this->stub->expects($this->once())->method('parse') |
|
| 119 | + ->will($this->returnValue($this->stub)); |
|
| 120 | 120 | |
| 121 | - $this->assertInstanceOf( $expected, $this->object->parse( [] ) ); |
|
| 121 | + $this->assertInstanceOf($expected, $this->object->parse([])); |
|
| 122 | 122 | } |
| 123 | 123 | |
| 124 | 124 | |
| 125 | 125 | public function testSearch() |
| 126 | 126 | { |
| 127 | - $item = \Aimeos\MShop::create( $this->context, 'stock' )->createItem(); |
|
| 127 | + $item = \Aimeos\MShop::create($this->context, 'stock')->createItem(); |
|
| 128 | 128 | |
| 129 | - $this->stub->expects( $this->once() )->method( 'search' ) |
|
| 130 | - ->will( $this->returnValue( [$item] ) ); |
|
| 129 | + $this->stub->expects($this->once())->method('search') |
|
| 130 | + ->will($this->returnValue([$item])); |
|
| 131 | 131 | |
| 132 | - $this->assertEquals( [$item], $this->object->search() ); |
|
| 132 | + $this->assertEquals([$item], $this->object->search()); |
|
| 133 | 133 | } |
| 134 | 134 | |
| 135 | 135 | |
@@ -137,10 +137,10 @@ discard block |
||
| 137 | 137 | { |
| 138 | 138 | $expected = \Aimeos\Controller\Frontend\Stock\Iface::class; |
| 139 | 139 | |
| 140 | - $this->stub->expects( $this->once() )->method( 'slice' ) |
|
| 141 | - ->will( $this->returnValue( $this->stub ) ); |
|
| 140 | + $this->stub->expects($this->once())->method('slice') |
|
| 141 | + ->will($this->returnValue($this->stub)); |
|
| 142 | 142 | |
| 143 | - $this->assertInstanceOf( $expected, $this->object->slice( 0, 100 ) ); |
|
| 143 | + $this->assertInstanceOf($expected, $this->object->slice(0, 100)); |
|
| 144 | 144 | } |
| 145 | 145 | |
| 146 | 146 | |
@@ -148,26 +148,26 @@ discard block |
||
| 148 | 148 | { |
| 149 | 149 | $expected = \Aimeos\Controller\Frontend\Stock\Iface::class; |
| 150 | 150 | |
| 151 | - $this->stub->expects( $this->once() )->method( 'sort' ) |
|
| 152 | - ->will( $this->returnValue( $this->stub ) ); |
|
| 151 | + $this->stub->expects($this->once())->method('sort') |
|
| 152 | + ->will($this->returnValue($this->stub)); |
|
| 153 | 153 | |
| 154 | - $this->assertInstanceOf( $expected, $this->object->sort( 'stock' ) ); |
|
| 154 | + $this->assertInstanceOf($expected, $this->object->sort('stock')); |
|
| 155 | 155 | } |
| 156 | 156 | |
| 157 | 157 | |
| 158 | 158 | public function testGetController() |
| 159 | 159 | { |
| 160 | - $result = $this->access( 'getController' )->invokeArgs( $this->object, [] ); |
|
| 160 | + $result = $this->access('getController')->invokeArgs($this->object, []); |
|
| 161 | 161 | |
| 162 | - $this->assertSame( $this->stub, $result ); |
|
| 162 | + $this->assertSame($this->stub, $result); |
|
| 163 | 163 | } |
| 164 | 164 | |
| 165 | 165 | |
| 166 | - protected function access( $name ) |
|
| 166 | + protected function access($name) |
|
| 167 | 167 | { |
| 168 | - $class = new \ReflectionClass( \Aimeos\Controller\Frontend\Stock\Decorator\Base::class ); |
|
| 169 | - $method = $class->getMethod( $name ); |
|
| 170 | - $method->setAccessible( true ); |
|
| 168 | + $class = new \ReflectionClass(\Aimeos\Controller\Frontend\Stock\Decorator\Base::class); |
|
| 169 | + $method = $class->getMethod($name); |
|
| 170 | + $method->setAccessible(true); |
|
| 171 | 171 | |
| 172 | 172 | return $method; |
| 173 | 173 | } |
@@ -18,93 +18,93 @@ |
||
| 18 | 18 | protected function setUp() |
| 19 | 19 | { |
| 20 | 20 | $this->context = \TestHelperFrontend::getContext(); |
| 21 | - $this->object = new \Aimeos\Controller\Frontend\Stock\Standard( $this->context ); |
|
| 21 | + $this->object = new \Aimeos\Controller\Frontend\Stock\Standard($this->context); |
|
| 22 | 22 | } |
| 23 | 23 | |
| 24 | 24 | |
| 25 | 25 | protected function tearDown() |
| 26 | 26 | { |
| 27 | - unset( $this->object, $this->context ); |
|
| 27 | + unset($this->object, $this->context); |
|
| 28 | 28 | } |
| 29 | 29 | |
| 30 | 30 | |
| 31 | 31 | public function testCode() |
| 32 | 32 | { |
| 33 | - $this->assertEquals( 2, count( $this->object->code( ['CNC', 'CNE'] )->search() ) ); |
|
| 33 | + $this->assertEquals(2, count($this->object->code(['CNC', 'CNE'])->search())); |
|
| 34 | 34 | } |
| 35 | 35 | |
| 36 | 36 | |
| 37 | 37 | public function testCompare() |
| 38 | 38 | { |
| 39 | - $this->assertEquals( 1, count( $this->object->compare( '==', 'stock.stocklevel', null )->search() ) ); |
|
| 39 | + $this->assertEquals(1, count($this->object->compare('==', 'stock.stocklevel', null)->search())); |
|
| 40 | 40 | } |
| 41 | 41 | |
| 42 | 42 | |
| 43 | 43 | public function testGet() |
| 44 | 44 | { |
| 45 | 45 | $iface = \Aimeos\MShop\Stock\Item\Iface::class; |
| 46 | - $item = \Aimeos\MShop::create( $this->context, 'stock' )->findItem( 'CNC', [], 'product', 'default' ); |
|
| 46 | + $item = \Aimeos\MShop::create($this->context, 'stock')->findItem('CNC', [], 'product', 'default'); |
|
| 47 | 47 | |
| 48 | - $this->assertInstanceOf( $iface, $this->object->get( $item->getId() ) ); |
|
| 48 | + $this->assertInstanceOf($iface, $this->object->get($item->getId())); |
|
| 49 | 49 | } |
| 50 | 50 | |
| 51 | 51 | |
| 52 | 52 | public function testFind() |
| 53 | 53 | { |
| 54 | 54 | $iface = \Aimeos\MShop\Stock\Item\Iface::class; |
| 55 | - $this->assertInstanceOf( $iface, $this->object->find( 'CNC', 'default' ) ); |
|
| 55 | + $this->assertInstanceOf($iface, $this->object->find('CNC', 'default')); |
|
| 56 | 56 | } |
| 57 | 57 | |
| 58 | 58 | |
| 59 | 59 | public function testParse() |
| 60 | 60 | { |
| 61 | 61 | $cond = ['||' => [['==' => ['stock.dateback' => null]], ['>=' => ['stock.dateback' => '2010-01-01 00:00:00']]]]; |
| 62 | - $this->assertEquals( 13, count( $this->object->parse( $cond )->search() ) ); |
|
| 62 | + $this->assertEquals(13, count($this->object->parse($cond)->search())); |
|
| 63 | 63 | } |
| 64 | 64 | |
| 65 | 65 | |
| 66 | 66 | public function testSearch() |
| 67 | 67 | { |
| 68 | 68 | $total = 0; |
| 69 | - $this->assertGreaterThanOrEqual( 15, count( $this->object->search( $total ) ) ); |
|
| 70 | - $this->assertGreaterThanOrEqual( 15, $total ); |
|
| 69 | + $this->assertGreaterThanOrEqual(15, count($this->object->search($total))); |
|
| 70 | + $this->assertGreaterThanOrEqual(15, $total); |
|
| 71 | 71 | } |
| 72 | 72 | |
| 73 | 73 | |
| 74 | 74 | public function testSlice() |
| 75 | 75 | { |
| 76 | - $this->assertEquals( 2, count( $this->object->slice( 0, 2 )->search() ) ); |
|
| 76 | + $this->assertEquals(2, count($this->object->slice(0, 2)->search())); |
|
| 77 | 77 | } |
| 78 | 78 | |
| 79 | 79 | |
| 80 | 80 | public function testSort() |
| 81 | 81 | { |
| 82 | - $this->assertGreaterThanOrEqual( 15, count( $this->object->sort()->search() ) ); |
|
| 82 | + $this->assertGreaterThanOrEqual(15, count($this->object->sort()->search())); |
|
| 83 | 83 | } |
| 84 | 84 | |
| 85 | 85 | |
| 86 | 86 | public function testSortGeneric() |
| 87 | 87 | { |
| 88 | - $this->assertGreaterThanOrEqual( 15, count( $this->object->sort( 'stock.dateback' )->search() ) ); |
|
| 88 | + $this->assertGreaterThanOrEqual(15, count($this->object->sort('stock.dateback')->search())); |
|
| 89 | 89 | } |
| 90 | 90 | |
| 91 | 91 | |
| 92 | 92 | public function testSortStock() |
| 93 | 93 | { |
| 94 | - $result = $this->object->sort( 'stock' )->search(); |
|
| 95 | - $this->assertStringStartsWith( 'U:TEST', reset( $result )->getProductCode() ); |
|
| 94 | + $result = $this->object->sort('stock')->search(); |
|
| 95 | + $this->assertStringStartsWith('U:TEST', reset($result)->getProductCode()); |
|
| 96 | 96 | } |
| 97 | 97 | |
| 98 | 98 | |
| 99 | 99 | public function testSortStockDesc() |
| 100 | 100 | { |
| 101 | - $result = $this->object->sort( '-stock' )->search(); |
|
| 102 | - $this->assertStringStartsWith( 'U:TEST', end( $result )->getProductCode() ); |
|
| 101 | + $result = $this->object->sort('-stock')->search(); |
|
| 102 | + $this->assertStringStartsWith('U:TEST', end($result)->getProductCode()); |
|
| 103 | 103 | } |
| 104 | 104 | |
| 105 | 105 | |
| 106 | 106 | public function testType() |
| 107 | 107 | { |
| 108 | - $this->assertEquals( 8, count( $this->object->type( 'default' )->search() ) ); |
|
| 108 | + $this->assertEquals(8, count($this->object->type('default')->search())); |
|
| 109 | 109 | } |
| 110 | 110 | } |
@@ -20,48 +20,48 @@ discard block |
||
| 20 | 20 | { |
| 21 | 21 | $this->context = \TestHelperFrontend::getContext(); |
| 22 | 22 | |
| 23 | - $this->stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Standard::class ) |
|
| 23 | + $this->stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Standard::class) |
|
| 24 | 24 | ->disableOriginalConstructor() |
| 25 | 25 | ->getMock(); |
| 26 | 26 | |
| 27 | - $this->object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Decorator\Base::class ) |
|
| 28 | - ->setConstructorArgs( [$this->stub, $this->context] ) |
|
| 27 | + $this->object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Decorator\Base::class) |
|
| 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::class )->getMock(); |
|
| 41 | + $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Iface::class)->getMock(); |
|
| 42 | 42 | |
| 43 | - $this->setExpectedException( \Aimeos\MW\Common\Exception::class ); |
|
| 43 | + $this->setExpectedException(\Aimeos\MW\Common\Exception::class); |
|
| 44 | 44 | |
| 45 | - $this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Decorator\Base::class ) |
|
| 46 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
| 45 | + $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Decorator\Base::class) |
|
| 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::class ) |
|
| 53 | + $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Standard::class) |
|
| 54 | 54 | ->disableOriginalConstructor() |
| 55 | - ->setMethods( ['invalid'] ) |
|
| 55 | + ->setMethods(['invalid']) |
|
| 56 | 56 | ->getMock(); |
| 57 | 57 | |
| 58 | - $object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Decorator\Base::class ) |
|
| 59 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
| 58 | + $object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Decorator\Base::class) |
|
| 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 | |
@@ -69,34 +69,34 @@ discard block |
||
| 69 | 69 | { |
| 70 | 70 | $expected = \Aimeos\Controller\Frontend\Supplier\Iface::class; |
| 71 | 71 | |
| 72 | - $this->stub->expects( $this->once() )->method( 'compare' ) |
|
| 73 | - ->will( $this->returnValue( $this->stub ) ); |
|
| 72 | + $this->stub->expects($this->once())->method('compare') |
|
| 73 | + ->will($this->returnValue($this->stub)); |
|
| 74 | 74 | |
| 75 | - $this->assertInstanceOf( $expected, $this->object->compare( '==', 'supplier.status', 1 ) ); |
|
| 75 | + $this->assertInstanceOf($expected, $this->object->compare('==', 'supplier.status', 1)); |
|
| 76 | 76 | } |
| 77 | 77 | |
| 78 | 78 | |
| 79 | 79 | public function testGet() |
| 80 | 80 | { |
| 81 | - $item = \Aimeos\MShop::create( $this->context, 'supplier' )->createItem(); |
|
| 81 | + $item = \Aimeos\MShop::create($this->context, 'supplier')->createItem(); |
|
| 82 | 82 | $expected = \Aimeos\MShop\Supplier\Item\Iface::class; |
| 83 | 83 | |
| 84 | - $this->stub->expects( $this->once() )->method( 'get' ) |
|
| 85 | - ->will( $this->returnValue( $item ) ); |
|
| 84 | + $this->stub->expects($this->once())->method('get') |
|
| 85 | + ->will($this->returnValue($item)); |
|
| 86 | 86 | |
| 87 | - $this->assertInstanceOf( $expected, $this->object->get( 1 ) ); |
|
| 87 | + $this->assertInstanceOf($expected, $this->object->get(1)); |
|
| 88 | 88 | } |
| 89 | 89 | |
| 90 | 90 | |
| 91 | 91 | public function testFind() |
| 92 | 92 | { |
| 93 | - $item = \Aimeos\MShop::create( $this->context, 'supplier' )->createItem(); |
|
| 93 | + $item = \Aimeos\MShop::create($this->context, 'supplier')->createItem(); |
|
| 94 | 94 | $expected = \Aimeos\MShop\Supplier\Item\Iface::class; |
| 95 | 95 | |
| 96 | - $this->stub->expects( $this->once() )->method( 'find' ) |
|
| 97 | - ->will( $this->returnValue( $item ) ); |
|
| 96 | + $this->stub->expects($this->once())->method('find') |
|
| 97 | + ->will($this->returnValue($item)); |
|
| 98 | 98 | |
| 99 | - $this->assertInstanceOf( $expected, $this->object->find( 'test' ) ); |
|
| 99 | + $this->assertInstanceOf($expected, $this->object->find('test')); |
|
| 100 | 100 | } |
| 101 | 101 | |
| 102 | 102 | |
@@ -104,21 +104,21 @@ discard block |
||
| 104 | 104 | { |
| 105 | 105 | $expected = \Aimeos\Controller\Frontend\Supplier\Iface::class; |
| 106 | 106 | |
| 107 | - $this->stub->expects( $this->once() )->method( 'parse' ) |
|
| 108 | - ->will( $this->returnValue( $this->stub ) ); |
|
| 107 | + $this->stub->expects($this->once())->method('parse') |
|
| 108 | + ->will($this->returnValue($this->stub)); |
|
| 109 | 109 | |
| 110 | - $this->assertInstanceOf( $expected, $this->object->parse( [] ) ); |
|
| 110 | + $this->assertInstanceOf($expected, $this->object->parse([])); |
|
| 111 | 111 | } |
| 112 | 112 | |
| 113 | 113 | |
| 114 | 114 | public function testSearch() |
| 115 | 115 | { |
| 116 | - $item = \Aimeos\MShop::create( $this->context, 'supplier' )->createItem(); |
|
| 116 | + $item = \Aimeos\MShop::create($this->context, 'supplier')->createItem(); |
|
| 117 | 117 | |
| 118 | - $this->stub->expects( $this->once() )->method( 'search' ) |
|
| 119 | - ->will( $this->returnValue( [$item] ) ); |
|
| 118 | + $this->stub->expects($this->once())->method('search') |
|
| 119 | + ->will($this->returnValue([$item])); |
|
| 120 | 120 | |
| 121 | - $this->assertEquals( [$item], $this->object->search() ); |
|
| 121 | + $this->assertEquals([$item], $this->object->search()); |
|
| 122 | 122 | } |
| 123 | 123 | |
| 124 | 124 | |
@@ -126,10 +126,10 @@ discard block |
||
| 126 | 126 | { |
| 127 | 127 | $expected = \Aimeos\Controller\Frontend\Supplier\Iface::class; |
| 128 | 128 | |
| 129 | - $this->stub->expects( $this->once() )->method( 'slice' ) |
|
| 130 | - ->will( $this->returnValue( $this->stub ) ); |
|
| 129 | + $this->stub->expects($this->once())->method('slice') |
|
| 130 | + ->will($this->returnValue($this->stub)); |
|
| 131 | 131 | |
| 132 | - $this->assertInstanceOf( $expected, $this->object->slice( 0, 100 ) ); |
|
| 132 | + $this->assertInstanceOf($expected, $this->object->slice(0, 100)); |
|
| 133 | 133 | } |
| 134 | 134 | |
| 135 | 135 | |
@@ -137,26 +137,26 @@ discard block |
||
| 137 | 137 | { |
| 138 | 138 | $expected = \Aimeos\Controller\Frontend\Supplier\Iface::class; |
| 139 | 139 | |
| 140 | - $this->stub->expects( $this->once() )->method( 'sort' ) |
|
| 141 | - ->will( $this->returnValue( $this->stub ) ); |
|
| 140 | + $this->stub->expects($this->once())->method('sort') |
|
| 141 | + ->will($this->returnValue($this->stub)); |
|
| 142 | 142 | |
| 143 | - $this->assertInstanceOf( $expected, $this->object->sort( 'supplier.label' ) ); |
|
| 143 | + $this->assertInstanceOf($expected, $this->object->sort('supplier.label')); |
|
| 144 | 144 | } |
| 145 | 145 | |
| 146 | 146 | |
| 147 | 147 | public function testGetController() |
| 148 | 148 | { |
| 149 | - $result = $this->access( 'getController' )->invokeArgs( $this->object, [] ); |
|
| 149 | + $result = $this->access('getController')->invokeArgs($this->object, []); |
|
| 150 | 150 | |
| 151 | - $this->assertSame( $this->stub, $result ); |
|
| 151 | + $this->assertSame($this->stub, $result); |
|
| 152 | 152 | } |
| 153 | 153 | |
| 154 | 154 | |
| 155 | - protected function access( $name ) |
|
| 155 | + protected function access($name) |
|
| 156 | 156 | { |
| 157 | - $class = new \ReflectionClass( \Aimeos\Controller\Frontend\Supplier\Decorator\Base::class ); |
|
| 158 | - $method = $class->getMethod( $name ); |
|
| 159 | - $method->setAccessible( true ); |
|
| 157 | + $class = new \ReflectionClass(\Aimeos\Controller\Frontend\Supplier\Decorator\Base::class); |
|
| 158 | + $method = $class->getMethod($name); |
|
| 159 | + $method->setAccessible(true); |
|
| 160 | 160 | |
| 161 | 161 | return $method; |
| 162 | 162 | } |
@@ -18,67 +18,67 @@ |
||
| 18 | 18 | protected function setUp() |
| 19 | 19 | { |
| 20 | 20 | $this->context = \TestHelperFrontend::getContext(); |
| 21 | - $this->object = new \Aimeos\Controller\Frontend\Supplier\Standard( $this->context ); |
|
| 21 | + $this->object = new \Aimeos\Controller\Frontend\Supplier\Standard($this->context); |
|
| 22 | 22 | } |
| 23 | 23 | |
| 24 | 24 | |
| 25 | 25 | protected function tearDown() |
| 26 | 26 | { |
| 27 | - unset( $this->object, $this->context ); |
|
| 27 | + unset($this->object, $this->context); |
|
| 28 | 28 | } |
| 29 | 29 | |
| 30 | 30 | |
| 31 | 31 | public function testCompare() |
| 32 | 32 | { |
| 33 | - $this->assertEquals( 2, count( $this->object->compare( '=~', 'supplier.label', 'unit' )->search() ) ); |
|
| 33 | + $this->assertEquals(2, count($this->object->compare('=~', 'supplier.label', 'unit')->search())); |
|
| 34 | 34 | } |
| 35 | 35 | |
| 36 | 36 | |
| 37 | 37 | public function testGet() |
| 38 | 38 | { |
| 39 | 39 | $iface = \Aimeos\MShop\Supplier\Item\Iface::class; |
| 40 | - $item = \Aimeos\MShop::create( $this->context, 'supplier' )->findItem( 'unitCode001' ); |
|
| 40 | + $item = \Aimeos\MShop::create($this->context, 'supplier')->findItem('unitCode001'); |
|
| 41 | 41 | |
| 42 | - $this->assertInstanceOf( $iface, $this->object->get( $item->getId() ) ); |
|
| 42 | + $this->assertInstanceOf($iface, $this->object->get($item->getId())); |
|
| 43 | 43 | } |
| 44 | 44 | |
| 45 | 45 | |
| 46 | 46 | public function testFind() |
| 47 | 47 | { |
| 48 | 48 | $iface = \Aimeos\MShop\Supplier\Item\Iface::class; |
| 49 | - $this->assertInstanceOf( $iface, $this->object->find( 'unitCode001' ) ); |
|
| 49 | + $this->assertInstanceOf($iface, $this->object->find('unitCode001')); |
|
| 50 | 50 | } |
| 51 | 51 | |
| 52 | 52 | |
| 53 | 53 | public function testParse() |
| 54 | 54 | { |
| 55 | 55 | $cond = ['&&' => [['==' => ['supplier.status' => 1]], ['=~' => ['supplier.label' => 'unit']]]]; |
| 56 | - $this->assertEquals( 2, count( $this->object->parse( $cond )->search() ) ); |
|
| 56 | + $this->assertEquals(2, count($this->object->parse($cond)->search())); |
|
| 57 | 57 | } |
| 58 | 58 | |
| 59 | 59 | |
| 60 | 60 | public function testSearch() |
| 61 | 61 | { |
| 62 | 62 | $total = 0; |
| 63 | - $this->assertGreaterThanOrEqual( 2, count( $this->object->search( [], $total ) ) ); |
|
| 64 | - $this->assertGreaterThanOrEqual( 2, $total ); |
|
| 63 | + $this->assertGreaterThanOrEqual(2, count($this->object->search([], $total))); |
|
| 64 | + $this->assertGreaterThanOrEqual(2, $total); |
|
| 65 | 65 | } |
| 66 | 66 | |
| 67 | 67 | |
| 68 | 68 | public function testSlice() |
| 69 | 69 | { |
| 70 | - $this->assertEquals( 1, count( $this->object->slice( 0, 1 )->search() ) ); |
|
| 70 | + $this->assertEquals(1, count($this->object->slice(0, 1)->search())); |
|
| 71 | 71 | } |
| 72 | 72 | |
| 73 | 73 | |
| 74 | 74 | public function testSort() |
| 75 | 75 | { |
| 76 | - $this->assertGreaterThanOrEqual( 2, count( $this->object->sort()->search() ) ); |
|
| 76 | + $this->assertGreaterThanOrEqual(2, count($this->object->sort()->search())); |
|
| 77 | 77 | } |
| 78 | 78 | |
| 79 | 79 | |
| 80 | 80 | public function testSortGeneric() |
| 81 | 81 | { |
| 82 | - $this->assertGreaterThanOrEqual( 2, count( $this->object->sort( 'supplier.label' )->search() ) ); |
|
| 82 | + $this->assertGreaterThanOrEqual(2, count($this->object->sort('supplier.label')->search())); |
|
| 83 | 83 | } |
| 84 | 84 | } |