@@ -20,122 +20,122 @@ |
||
| 20 | 20 | { |
| 21 | 21 | $this->context = \TestHelperFrontend::getContext(); |
| 22 | 22 | |
| 23 | - $this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Subscription\Standard' ) |
|
| 23 | + $this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Subscription\Standard') |
|
| 24 | 24 | ->disableOriginalConstructor() |
| 25 | 25 | ->getMock(); |
| 26 | 26 | |
| 27 | - $this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Subscription\Decorator\Base' ) |
|
| 28 | - ->setConstructorArgs( [$this->stub, $this->context] ) |
|
| 27 | + $this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Subscription\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\Controller\Frontend\Exception' ); |
|
| 43 | + $this->setExpectedException('\Aimeos\Controller\Frontend\Exception'); |
|
| 44 | 44 | |
| 45 | - $this->getMockBuilder( '\Aimeos\Controller\Frontend\Subscription\Decorator\Base' ) |
|
| 46 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
| 45 | + $this->getMockBuilder('\Aimeos\Controller\Frontend\Subscription\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\Subscription\Standard' ) |
|
| 53 | + $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Subscription\Standard') |
|
| 54 | 54 | ->disableOriginalConstructor() |
| 55 | - ->setMethods( ['invalid'] ) |
|
| 55 | + ->setMethods(['invalid']) |
|
| 56 | 56 | ->getMock(); |
| 57 | 57 | |
| 58 | - $object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Subscription\Decorator\Base' ) |
|
| 59 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
| 58 | + $object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Subscription\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 testCancel() |
| 69 | 69 | { |
| 70 | - $item = \Aimeos\MShop\Factory::createManager( $this->context, 'subscription' )->createItem(); |
|
| 70 | + $item = \Aimeos\MShop\Factory::createManager($this->context, 'subscription')->createItem(); |
|
| 71 | 71 | |
| 72 | - $this->stub->expects( $this->once() )->method( 'cancel' )->will( $this->returnValue( $item ) ); |
|
| 72 | + $this->stub->expects($this->once())->method('cancel')->will($this->returnValue($item)); |
|
| 73 | 73 | |
| 74 | - $this->assertInstanceOf( '\Aimeos\MShop\Subscription\Item\Iface', $this->object->cancel( -1, '' ) ); |
|
| 74 | + $this->assertInstanceOf('\Aimeos\MShop\Subscription\Item\Iface', $this->object->cancel( -1, '' )); |
|
| 75 | 75 | } |
| 76 | 76 | |
| 77 | 77 | |
| 78 | 78 | public function testCreateFilter() |
| 79 | 79 | { |
| 80 | - $search = \Aimeos\MShop\Factory::createManager( $this->context, 'subscription' )->createSearch(); |
|
| 80 | + $search = \Aimeos\MShop\Factory::createManager($this->context, 'subscription')->createSearch(); |
|
| 81 | 81 | |
| 82 | - $this->stub->expects( $this->once() )->method( 'createFilter' )->will( $this->returnValue( $search ) ); |
|
| 82 | + $this->stub->expects($this->once())->method('createFilter')->will($this->returnValue($search)); |
|
| 83 | 83 | |
| 84 | - $this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->createFilter() ); |
|
| 84 | + $this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->createFilter()); |
|
| 85 | 85 | } |
| 86 | 86 | |
| 87 | 87 | |
| 88 | 88 | public function testGetItem() |
| 89 | 89 | { |
| 90 | - $item = \Aimeos\MShop\Factory::createManager( $this->context, 'subscription' )->createItem(); |
|
| 90 | + $item = \Aimeos\MShop\Factory::createManager($this->context, 'subscription')->createItem(); |
|
| 91 | 91 | |
| 92 | - $this->stub->expects( $this->once() )->method( 'getItem' )->will( $this->returnValue( $item ) ); |
|
| 92 | + $this->stub->expects($this->once())->method('getItem')->will($this->returnValue($item)); |
|
| 93 | 93 | |
| 94 | - $this->assertInstanceOf( '\Aimeos\MShop\Subscription\Item\Iface', $this->object->getItem( -1 ) ); |
|
| 94 | + $this->assertInstanceOf('\Aimeos\MShop\Subscription\Item\Iface', $this->object->getItem( -1 )); |
|
| 95 | 95 | } |
| 96 | 96 | |
| 97 | 97 | |
| 98 | 98 | public function testSaveItem() |
| 99 | 99 | { |
| 100 | - $item = \Aimeos\MShop\Factory::createManager( $this->context, 'subscription' )->createItem(); |
|
| 100 | + $item = \Aimeos\MShop\Factory::createManager($this->context, 'subscription')->createItem(); |
|
| 101 | 101 | |
| 102 | - $this->stub->expects( $this->once() )->method( 'saveItem' ); |
|
| 102 | + $this->stub->expects($this->once())->method('saveItem'); |
|
| 103 | 103 | |
| 104 | - $this->object->saveItem( $item ); |
|
| 104 | + $this->object->saveItem($item); |
|
| 105 | 105 | } |
| 106 | 106 | |
| 107 | 107 | |
| 108 | 108 | public function testSearchItems() |
| 109 | 109 | { |
| 110 | - $search = $this->getMockBuilder( '\Aimeos\MW\Criteria\Iface' )->getMock(); |
|
| 110 | + $search = $this->getMockBuilder('\Aimeos\MW\Criteria\Iface')->getMock(); |
|
| 111 | 111 | |
| 112 | - $this->stub->expects( $this->once() )->method( 'searchItems' )->will( $this->returnValue( [] ) ); |
|
| 112 | + $this->stub->expects($this->once())->method('searchItems')->will($this->returnValue([])); |
|
| 113 | 113 | |
| 114 | - $this->assertEquals( [], $this->object->searchItems( $search ) ); |
|
| 114 | + $this->assertEquals([], $this->object->searchItems($search)); |
|
| 115 | 115 | } |
| 116 | 116 | |
| 117 | 117 | |
| 118 | 118 | public function testGetIntervals() |
| 119 | 119 | { |
| 120 | - $this->stub->expects( $this->once() )->method( 'getIntervals' )->will( $this->returnValue( [] ) ); |
|
| 120 | + $this->stub->expects($this->once())->method('getIntervals')->will($this->returnValue([])); |
|
| 121 | 121 | |
| 122 | - $this->assertEquals( [], $this->object->getIntervals() ); |
|
| 122 | + $this->assertEquals([], $this->object->getIntervals()); |
|
| 123 | 123 | } |
| 124 | 124 | |
| 125 | 125 | |
| 126 | 126 | public function testGetController() |
| 127 | 127 | { |
| 128 | - $result = $this->access( 'getController' )->invokeArgs( $this->object, [] ); |
|
| 128 | + $result = $this->access('getController')->invokeArgs($this->object, []); |
|
| 129 | 129 | |
| 130 | - $this->assertSame( $this->stub, $result ); |
|
| 130 | + $this->assertSame($this->stub, $result); |
|
| 131 | 131 | } |
| 132 | 132 | |
| 133 | 133 | |
| 134 | - protected function access( $name ) |
|
| 134 | + protected function access($name) |
|
| 135 | 135 | { |
| 136 | - $class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Subscription\Decorator\Base' ); |
|
| 137 | - $method = $class->getMethod( $name ); |
|
| 138 | - $method->setAccessible( true ); |
|
| 136 | + $class = new \ReflectionClass('\Aimeos\Controller\Frontend\Subscription\Decorator\Base'); |
|
| 137 | + $method = $class->getMethod($name); |
|
| 138 | + $method->setAccessible(true); |
|
| 139 | 139 | |
| 140 | 140 | return $method; |
| 141 | 141 | } |
@@ -17,104 +17,104 @@ |
||
| 17 | 17 | |
| 18 | 18 | protected function setUp() |
| 19 | 19 | { |
| 20 | - \Aimeos\MShop\Factory::setCache( true ); |
|
| 20 | + \Aimeos\MShop\Factory::setCache(true); |
|
| 21 | 21 | |
| 22 | 22 | $this->context = \TestHelperFrontend::getContext(); |
| 23 | - $this->object = new \Aimeos\Controller\Frontend\Subscription\Standard( $this->context ); |
|
| 23 | + $this->object = new \Aimeos\Controller\Frontend\Subscription\Standard($this->context); |
|
| 24 | 24 | } |
| 25 | 25 | |
| 26 | 26 | |
| 27 | 27 | protected function tearDown() |
| 28 | 28 | { |
| 29 | - unset( $this->object, $this->context ); |
|
| 29 | + unset($this->object, $this->context); |
|
| 30 | 30 | |
| 31 | - \Aimeos\MShop\Factory::setCache( false ); |
|
| 31 | + \Aimeos\MShop\Factory::setCache(false); |
|
| 32 | 32 | \Aimeos\MShop\Factory::clear(); |
| 33 | 33 | } |
| 34 | 34 | |
| 35 | 35 | |
| 36 | 36 | public function testCancel() |
| 37 | 37 | { |
| 38 | - $this->context->setUserId( $this->getCustomerId() ); |
|
| 39 | - $item = \Aimeos\MShop\Factory::createManager( $this->context, 'subscription' )->createItem(); |
|
| 38 | + $this->context->setUserId($this->getCustomerId()); |
|
| 39 | + $item = \Aimeos\MShop\Factory::createManager($this->context, 'subscription')->createItem(); |
|
| 40 | 40 | |
| 41 | - $cntl = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Subscription\Standard' ) |
|
| 42 | - ->setConstructorArgs( [$this->context] ) |
|
| 43 | - ->setMethods( ['saveItem'] ) |
|
| 41 | + $cntl = $this->getMockBuilder('\Aimeos\Controller\Frontend\Subscription\Standard') |
|
| 42 | + ->setConstructorArgs([$this->context]) |
|
| 43 | + ->setMethods(['saveItem']) |
|
| 44 | 44 | ->getMock(); |
| 45 | 45 | |
| 46 | - $cntl->expects( $this->once() )->method( 'saveItem' ) |
|
| 47 | - ->will( $this->returnValue( $item ) ); |
|
| 46 | + $cntl->expects($this->once())->method('saveItem') |
|
| 47 | + ->will($this->returnValue($item)); |
|
| 48 | 48 | |
| 49 | - $this->assertInstanceOf( '\Aimeos\MShop\Subscription\Item\Iface', $cntl->cancel( $this->getSubscriptionId() ) ); |
|
| 49 | + $this->assertInstanceOf('\Aimeos\MShop\Subscription\Item\Iface', $cntl->cancel($this->getSubscriptionId())); |
|
| 50 | 50 | } |
| 51 | 51 | |
| 52 | 52 | |
| 53 | 53 | public function testCreateFilter() |
| 54 | 54 | { |
| 55 | - $this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->createFilter() ); |
|
| 55 | + $this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->createFilter()); |
|
| 56 | 56 | } |
| 57 | 57 | |
| 58 | 58 | |
| 59 | 59 | public function testGetIntervals() |
| 60 | 60 | { |
| 61 | - $this->assertGreaterThan( 0, count( $this->object->getIntervals() ) ); |
|
| 61 | + $this->assertGreaterThan(0, count($this->object->getIntervals())); |
|
| 62 | 62 | } |
| 63 | 63 | |
| 64 | 64 | |
| 65 | 65 | public function testGetItem() |
| 66 | 66 | { |
| 67 | 67 | $id = $this->getSubscriptionId(); |
| 68 | - $this->context->setUserId( $this->getCustomerId() ); |
|
| 68 | + $this->context->setUserId($this->getCustomerId()); |
|
| 69 | 69 | |
| 70 | - $this->assertInstanceOf( '\Aimeos\MShop\Subscription\Item\Iface', $this->object->getItem( $id ) ); |
|
| 70 | + $this->assertInstanceOf('\Aimeos\MShop\Subscription\Item\Iface', $this->object->getItem($id)); |
|
| 71 | 71 | } |
| 72 | 72 | |
| 73 | 73 | |
| 74 | 74 | public function testGetItemException() |
| 75 | 75 | { |
| 76 | - $this->setExpectedException( '\Aimeos\Controller\Frontend\Subscription\Exception' ); |
|
| 76 | + $this->setExpectedException('\Aimeos\Controller\Frontend\Subscription\Exception'); |
|
| 77 | 77 | $this->object->getItem( -1 ); |
| 78 | 78 | } |
| 79 | 79 | |
| 80 | 80 | |
| 81 | 81 | public function testSaveItem() |
| 82 | 82 | { |
| 83 | - $manager = $this->getMockBuilder( '\\Aimeos\\MShop\\Subscription\\Manager\\Standard' ) |
|
| 84 | - ->setConstructorArgs( [$this->context] ) |
|
| 85 | - ->setMethods( ['saveItem'] ) |
|
| 83 | + $manager = $this->getMockBuilder('\\Aimeos\\MShop\\Subscription\\Manager\\Standard') |
|
| 84 | + ->setConstructorArgs([$this->context]) |
|
| 85 | + ->setMethods(['saveItem']) |
|
| 86 | 86 | ->getMock(); |
| 87 | 87 | |
| 88 | - \Aimeos\MShop\Factory::injectManager( $this->context, 'subscription', $manager ); |
|
| 88 | + \Aimeos\MShop\Factory::injectManager($this->context, 'subscription', $manager); |
|
| 89 | 89 | |
| 90 | - $manager->expects( $this->once() )->method( 'saveItem' ) |
|
| 91 | - ->will( $this->returnValue( $manager->createItem() ) ); |
|
| 90 | + $manager->expects($this->once())->method('saveItem') |
|
| 91 | + ->will($this->returnValue($manager->createItem())); |
|
| 92 | 92 | |
| 93 | - $this->assertInstanceOf( '\Aimeos\MShop\Subscription\Item\Iface', $this->object->saveItem( $manager->createItem() ) ); |
|
| 93 | + $this->assertInstanceOf('\Aimeos\MShop\Subscription\Item\Iface', $this->object->saveItem($manager->createItem())); |
|
| 94 | 94 | } |
| 95 | 95 | |
| 96 | 96 | |
| 97 | 97 | public function testSearchItems() |
| 98 | 98 | { |
| 99 | - $this->assertGreaterThan( 1, $this->object->searchItems( $this->object->createFilter() ) ); |
|
| 99 | + $this->assertGreaterThan(1, $this->object->searchItems($this->object->createFilter())); |
|
| 100 | 100 | } |
| 101 | 101 | |
| 102 | 102 | protected function getCustomerId() |
| 103 | 103 | { |
| 104 | - $manager = \Aimeos\MShop\Factory::createManager( $this->context, 'customer' ); |
|
| 105 | - return $manager->findItem( 'UTC001' )->getId(); |
|
| 104 | + $manager = \Aimeos\MShop\Factory::createManager($this->context, 'customer'); |
|
| 105 | + return $manager->findItem('UTC001')->getId(); |
|
| 106 | 106 | } |
| 107 | 107 | |
| 108 | 108 | |
| 109 | 109 | protected function getSubscriptionId() |
| 110 | 110 | { |
| 111 | - $manager = \Aimeos\MShop\Factory::createManager( $this->context, 'subscription' ); |
|
| 112 | - $search = $manager->createSearch()->setSlice( 0, 1 ); |
|
| 113 | - $search->setConditions( $search->compare( '==', 'order.base.customerid', $this->getCustomerId() ) ); |
|
| 114 | - $result = $manager->searchItems( $search ); |
|
| 111 | + $manager = \Aimeos\MShop\Factory::createManager($this->context, 'subscription'); |
|
| 112 | + $search = $manager->createSearch()->setSlice(0, 1); |
|
| 113 | + $search->setConditions($search->compare('==', 'order.base.customerid', $this->getCustomerId())); |
|
| 114 | + $result = $manager->searchItems($search); |
|
| 115 | 115 | |
| 116 | - if( ( $item = reset( $result ) ) === false ) { |
|
| 117 | - throw new \RuntimeException( 'No subscription item found' ); |
|
| 116 | + if (($item = reset($result)) === false) { |
|
| 117 | + throw new \RuntimeException('No subscription item found'); |
|
| 118 | 118 | } |
| 119 | 119 | |
| 120 | 120 | return $item->getId(); |
@@ -15,31 +15,31 @@ |
||
| 15 | 15 | { |
| 16 | 16 | $target = '\\Aimeos\\Controller\\Frontend\\Subscription\\Iface'; |
| 17 | 17 | |
| 18 | - $controller = \Aimeos\Controller\Frontend\Subscription\Factory::createController( \TestHelperFrontend::getContext() ); |
|
| 19 | - $this->assertInstanceOf( $target, $controller ); |
|
| 18 | + $controller = \Aimeos\Controller\Frontend\Subscription\Factory::createController(\TestHelperFrontend::getContext()); |
|
| 19 | + $this->assertInstanceOf($target, $controller); |
|
| 20 | 20 | |
| 21 | - $controller = \Aimeos\Controller\Frontend\Subscription\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' ); |
|
| 22 | - $this->assertInstanceOf( $target, $controller ); |
|
| 21 | + $controller = \Aimeos\Controller\Frontend\Subscription\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\\Controller\\Frontend\\Exception' ); |
|
| 29 | - \Aimeos\Controller\Frontend\Subscription\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' ); |
|
| 28 | + $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception'); |
|
| 29 | + \Aimeos\Controller\Frontend\Subscription\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\Subscription\Factory::createController( \TestHelperFrontend::getContext(), '%^' ); |
|
| 35 | + $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception'); |
|
| 36 | + \Aimeos\Controller\Frontend\Subscription\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\Subscription\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' ); |
|
| 42 | + $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception'); |
|
| 43 | + \Aimeos\Controller\Frontend\Subscription\Factory::createController(\TestHelperFrontend::getContext(), 'notexist'); |
|
| 44 | 44 | } |
| 45 | 45 | } |