| Conditions | 1 | 
| Paths | 1 | 
| Total Lines | 75 | 
| Code Lines | 65 | 
| Lines | 0 | 
| Ratio | 0 % | 
| Changes | 1 | ||
| Bugs | 0 | Features | 1 | 
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php  | 
            ||
| 108 | public function testSaveUpdateDeleteItem()  | 
            ||
| 109 | 	{ | 
            ||
| 110 | $item = new \Aimeos\MShop\Common\Item\Address\Standard( 'customer.address.', $this->fixture );  | 
            ||
| 111 | $item->setId( null );  | 
            ||
| 112 | $this->object->saveItem( $item );  | 
            ||
| 113 | $itemSaved = $this->object->getItem( $item->getId() );  | 
            ||
| 114 | |||
| 115 | $itemExp = clone $itemSaved;  | 
            ||
| 116 | $itemExp->setPosition( 1 );  | 
            ||
| 117 | $itemExp->setCity( 'Berlin' );  | 
            ||
| 118 | $itemExp->setState( 'Berlin' );  | 
            ||
| 119 | $this->object->saveItem( $itemExp );  | 
            ||
| 120 | $itemUpd = $this->object->getItem( $itemExp->getId() );  | 
            ||
| 121 | |||
| 122 | $this->object->deleteItem( $itemSaved->getId() );  | 
            ||
| 123 | |||
| 124 | |||
| 125 | $this->assertTrue( $item->getId() !== null );  | 
            ||
| 126 | $this->assertEquals( $item->getId(), $itemSaved->getId() );  | 
            ||
| 127 | $this->assertEquals( $item->getParentId(), $itemSaved->getParentId());  | 
            ||
| 128 | $this->assertEquals( $item->getPosition(), $itemSaved->getPosition());  | 
            ||
| 129 | $this->assertEquals( $item->getCompany(), $itemSaved->getCompany());  | 
            ||
| 130 | $this->assertEquals( $item->getVatID(), $itemSaved->getVatID());  | 
            ||
| 131 | $this->assertEquals( $item->getSalutation(), $itemSaved->getSalutation());  | 
            ||
| 132 | $this->assertEquals( $item->getTitle(), $itemSaved->getTitle());  | 
            ||
| 133 | $this->assertEquals( $item->getFirstname(), $itemSaved->getFirstname());  | 
            ||
| 134 | $this->assertEquals( $item->getLastname(), $itemSaved->getLastname());  | 
            ||
| 135 | $this->assertEquals( $item->getAddress1(), $itemSaved->getAddress1());  | 
            ||
| 136 | $this->assertEquals( $item->getAddress2(), $itemSaved->getAddress2());  | 
            ||
| 137 | $this->assertEquals( $item->getAddress3(), $itemSaved->getAddress3());  | 
            ||
| 138 | $this->assertEquals( $item->getPostal(), $itemSaved->getPostal());  | 
            ||
| 139 | $this->assertEquals( $item->getCity(), $itemSaved->getCity());  | 
            ||
| 140 | $this->assertEquals( $item->getState(), $itemSaved->getState());  | 
            ||
| 141 | $this->assertEquals( $item->getCountryId(), $itemSaved->getCountryId());  | 
            ||
| 142 | $this->assertEquals( $item->getLanguageId(), $itemSaved->getLanguageId());  | 
            ||
| 143 | $this->assertEquals( $item->getTelephone(), $itemSaved->getTelephone());  | 
            ||
| 144 | $this->assertEquals( $item->getEmail(), $itemSaved->getEmail());  | 
            ||
| 145 | $this->assertEquals( $item->getTelefax(), $itemSaved->getTelefax());  | 
            ||
| 146 | $this->assertEquals( $item->getWebsite(), $itemSaved->getWebsite());  | 
            ||
| 147 | $this->assertEquals( $item->getFlag(), $itemSaved->getFlag());  | 
            ||
| 148 | |||
| 149 | $this->assertEquals( $this->editor, $itemSaved->getEditor() );  | 
            ||
| 150 | 		$this->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeCreated() ); | 
            ||
| 151 | 		$this->assertRegExp('/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeModified()); | 
            ||
| 152 | |||
| 153 | $this->assertEquals( $itemExp->getId(), $itemUpd->getId() );  | 
            ||
| 154 | $this->assertEquals( $itemExp->getParentId(), $itemUpd->getParentId());  | 
            ||
| 155 | $this->assertEquals( $itemExp->getPosition(), $itemUpd->getPosition());  | 
            ||
| 156 | $this->assertEquals( $itemExp->getCompany(), $itemUpd->getCompany());  | 
            ||
| 157 | $this->assertEquals( $itemExp->getVatID(), $itemUpd->getVatID());  | 
            ||
| 158 | $this->assertEquals( $itemExp->getSalutation(), $itemUpd->getSalutation());  | 
            ||
| 159 | $this->assertEquals( $itemExp->getTitle(), $itemUpd->getTitle());  | 
            ||
| 160 | $this->assertEquals( $itemExp->getFirstname(), $itemUpd->getFirstname());  | 
            ||
| 161 | $this->assertEquals( $itemExp->getLastname(), $itemUpd->getLastname());  | 
            ||
| 162 | $this->assertEquals( $itemExp->getAddress1(), $itemUpd->getAddress1());  | 
            ||
| 163 | $this->assertEquals( $itemExp->getAddress2(), $itemUpd->getAddress2());  | 
            ||
| 164 | $this->assertEquals( $itemExp->getAddress3(), $itemUpd->getAddress3());  | 
            ||
| 165 | $this->assertEquals( $itemExp->getPostal(), $itemUpd->getPostal());  | 
            ||
| 166 | $this->assertEquals( $itemExp->getCity(), $itemUpd->getCity());  | 
            ||
| 167 | $this->assertEquals( $itemExp->getState(), $itemUpd->getState());  | 
            ||
| 168 | $this->assertEquals( $itemExp->getCountryId(), $itemUpd->getCountryId());  | 
            ||
| 169 | $this->assertEquals( $itemExp->getLanguageId(), $itemUpd->getLanguageId());  | 
            ||
| 170 | $this->assertEquals( $itemExp->getTelephone(), $itemUpd->getTelephone());  | 
            ||
| 171 | $this->assertEquals( $itemExp->getEmail(), $itemUpd->getEmail());  | 
            ||
| 172 | $this->assertEquals( $itemExp->getTelefax(), $itemUpd->getTelefax());  | 
            ||
| 173 | $this->assertEquals( $itemExp->getWebsite(), $itemUpd->getWebsite());  | 
            ||
| 174 | $this->assertEquals( $itemExp->getFlag(), $itemUpd->getFlag());  | 
            ||
| 175 | |||
| 176 | $this->assertEquals( $this->editor, $itemUpd->getEditor() );  | 
            ||
| 177 | $this->assertEquals( $itemExp->getTimeCreated(), $itemUpd->getTimeCreated() );  | 
            ||
| 178 | 		$this->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemUpd->getTimeModified() ); | 
            ||
| 179 | |||
| 180 | 		$this->setExpectedException('\\Aimeos\\MShop\\Exception'); | 
            ||
| 181 | $this->object->getItem( $itemSaved->getId() );  | 
            ||
| 182 | }  | 
            ||
| 183 | |||
| 226 |