Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.
Common duplication problems, and corresponding solutions are:
| 1 | <?php |
||
| 5 | class JsonFormExtensionTest extends WebTestCase |
||
| 6 | { |
||
| 7 | private $client; |
||
| 8 | |||
| 9 | public function setUp() |
||
| 10 | { |
||
| 11 | $this->client = $this->createClient(['test_case' => 'JsonFormExtension']); |
||
| 12 | } |
||
| 13 | |||
| 14 | public function testJsonRequest() |
||
| 15 | { |
||
| 16 | $json = '{ "name": "test1" }'; |
||
| 17 | $this->client->request( |
||
| 18 | 'POST', |
||
| 19 | '/json-form-extension/json', |
||
| 20 | [], |
||
| 21 | [], |
||
| 22 | ['CONTENT_TYPE' => 'application/json'], |
||
| 23 | $json |
||
| 24 | ); |
||
| 25 | |||
| 26 | $expectedJson = json_encode([ |
||
| 27 | 'Data' => ['name' => 'test1', 'lastname' => null], |
||
| 28 | 'NormData' => ['name' => 'test1', 'lastname' => null], |
||
| 29 | 'ViewData' => ['name' => 'test1', 'lastname' => null], |
||
| 30 | ], 15); |
||
| 31 | $this->assertEquals( |
||
| 32 | $expectedJson, |
||
| 33 | $this->client->getResponse()->getContent() |
||
| 34 | ); |
||
| 35 | } |
||
| 36 | |||
| 37 | public function testJsonRequestInvalidJsonError() |
||
| 38 | { |
||
| 39 | $json = '{ "name" "test1" }'; |
||
| 40 | $this->client->request( |
||
| 41 | 'POST', |
||
| 42 | '/json-form-extension/json', |
||
| 43 | [], |
||
| 44 | [], |
||
| 45 | ['CONTENT_TYPE' => 'application/json'], |
||
| 46 | $json |
||
| 47 | ); |
||
| 48 | |||
| 49 | $expectedJsonRegExp = json_encode([ |
||
| 50 | 'Class' => 'InvalidArgumentException', |
||
| 51 | 'Message' => "Invalid submitted json data, error (.*) : (.*)", |
||
| 52 | ], 15); |
||
| 53 | |||
| 54 | $this->assertRegExp( |
||
| 55 | $expectedJsonRegExp, |
||
| 56 | $this->client->getResponse()->getContent() |
||
| 57 | ); |
||
| 58 | } |
||
| 59 | |||
| 60 | public function testJsonRequestNotAStringError() |
||
| 61 | { |
||
| 62 | $json = ['test' => 'test']; |
||
| 63 | $this->client->request( |
||
| 64 | 'POST', |
||
| 65 | '/json-form-extension/json', |
||
| 66 | [], |
||
| 67 | [], |
||
| 68 | ['CONTENT_TYPE' => 'application/json'], |
||
| 69 | $json |
||
| 70 | ); |
||
| 71 | |||
| 72 | $expectedJson = json_encode([ |
||
| 73 | 'Class' => 'InvalidArgumentException', |
||
| 74 | 'Message' => 'Invalid argument: the submitted variable must be a string when you enable the json_format option.', |
||
| 75 | ], 15); |
||
| 76 | $this->assertEquals( |
||
| 77 | $expectedJson, |
||
| 78 | $this->client->getResponse()->getContent() |
||
| 79 | ); |
||
| 80 | } |
||
| 81 | |||
| 82 | public function testPostRequest() |
||
| 83 | { |
||
| 84 | $this->client->request( |
||
| 85 | 'POST', |
||
| 86 | '/json-form-extension/post', |
||
| 87 | ['form' => ['name' => 'test1']] |
||
| 88 | ); |
||
| 89 | $expectedJson = json_encode([ |
||
| 90 | 'Data' => ['name' => 'test1', 'lastname' => null], |
||
| 91 | 'NormData' => ['name' => 'test1', 'lastname' => null], |
||
| 92 | 'ViewData' => ['name' => 'test1', 'lastname' => null], |
||
| 93 | ], 15); |
||
| 94 | $this->assertEquals( |
||
| 95 | $expectedJson, |
||
| 96 | $this->client->getResponse()->getContent() |
||
| 97 | ); |
||
| 98 | } |
||
| 99 | } |
||
| 100 |