| Conditions | 1 |
| Paths | 1 |
| Total Lines | 76 |
| Code Lines | 57 |
| Lines | 0 |
| Ratio | 0 % |
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 |
||
| 27 | public function testPermissions() { |
||
| 28 | $environment = $this->getEnvironment(); |
||
| 29 | |||
| 30 | // Check deployer / restorer permissions |
||
| 31 | $viewer = $this->objFromFixture('Member', 'viewer'); |
||
| 32 | $viewerbygroup = $this->objFromFixture('Member', 'viewerbygroup'); |
||
| 33 | $deployer = $this->objFromFixture('Member', 'deployer'); |
||
| 34 | $deployerbygroup = $this->objFromFixture('Member', 'deployerbygroup'); |
||
| 35 | $restorer = $this->objFromFixture('Member', 'restorer'); |
||
| 36 | $restorerbygroup = $this->objFromFixture('Member', 'restorerbygroup'); |
||
| 37 | |||
| 38 | $random = new Member(array('Email' => '[email protected]')); |
||
| 39 | $random->write(); |
||
| 40 | |||
| 41 | $this->assertFalse($environment->canView($random)); |
||
| 42 | $this->assertTrue($environment->canView($viewer)); |
||
|
1 ignored issue
–
show
|
|||
| 43 | $this->assertTrue($environment->canView($viewerbygroup)); |
||
|
1 ignored issue
–
show
|
|||
| 44 | |||
| 45 | $this->assertTrue($environment->canDeploy($deployer)); |
||
|
1 ignored issue
–
show
|
|||
| 46 | $this->assertTrue($environment->canDeploy($deployerbygroup)); |
||
|
1 ignored issue
–
show
|
|||
| 47 | $this->assertFalse($environment->canDeploy($restorer)); |
||
| 48 | $this->assertFalse($environment->canDeploy($restorerbygroup)); |
||
| 49 | |||
| 50 | $this->assertFalse($environment->canRestore($deployer)); |
||
| 51 | $this->assertFalse($environment->canRestore($deployerbygroup)); |
||
| 52 | $this->assertTrue($environment->canRestore($restorer)); |
||
|
1 ignored issue
–
show
|
|||
| 53 | $this->assertTrue($environment->canRestore($restorerbygroup)); |
||
|
1 ignored issue
–
show
|
|||
| 54 | |||
| 55 | // Check backup / uploader permissions |
||
| 56 | $backup = $this->objFromFixture('Member', 'backup'); |
||
| 57 | $backupbygroup = $this->objFromFixture('Member', 'backupbygroup'); |
||
| 58 | $uploader = $this->objFromFixture('Member', 'uploader'); |
||
| 59 | $uploaderbygroup = $this->objFromFixture('Member', 'uploaderbygroup'); |
||
| 60 | |||
| 61 | $this->assertTrue($environment->canBackup($backup)); |
||
|
1 ignored issue
–
show
|
|||
| 62 | $this->assertTrue($environment->canBackup($backupbygroup)); |
||
|
1 ignored issue
–
show
|
|||
| 63 | $this->assertFalse($environment->canBackup($uploader)); |
||
| 64 | $this->assertFalse($environment->canBackup($uploaderbygroup)); |
||
| 65 | |||
| 66 | $this->assertFalse($environment->canUploadArchive($backup)); |
||
| 67 | $this->assertFalse($environment->canUploadArchive($backupbygroup)); |
||
| 68 | $this->assertTrue($environment->canUploadArchive($uploader)); |
||
|
1 ignored issue
–
show
|
|||
| 69 | $this->assertTrue($environment->canUploadArchive($uploaderbygroup)); |
||
|
1 ignored issue
–
show
|
|||
| 70 | |||
| 71 | // Check downloader / deleters permissions |
||
| 72 | $downloader = $this->objFromFixture('Member', 'downloader'); |
||
| 73 | $downloaderbygroup = $this->objFromFixture('Member', 'downloaderbygroup'); |
||
| 74 | $deleter = $this->objFromFixture('Member', 'deleter'); |
||
| 75 | $deleterbygroup = $this->objFromFixture('Member', 'deleterbygroup'); |
||
| 76 | |||
| 77 | $this->assertTrue($environment->canDownloadArchive($downloader)); |
||
|
1 ignored issue
–
show
|
|||
| 78 | $this->assertTrue($environment->canDownloadArchive($downloaderbygroup)); |
||
|
1 ignored issue
–
show
|
|||
| 79 | $this->assertFalse($environment->canDownloadArchive($deleter)); |
||
| 80 | $this->assertFalse($environment->canDownloadArchive($deleterbygroup)); |
||
| 81 | |||
| 82 | $this->assertFalse($environment->canDeleteArchive($downloader)); |
||
| 83 | $this->assertFalse($environment->canDeleteArchive($downloaderbygroup)); |
||
| 84 | $this->assertTrue($environment->canDeleteArchive($deleter)); |
||
|
1 ignored issue
–
show
|
|||
| 85 | $this->assertTrue($environment->canDeleteArchive($deleterbygroup)); |
||
|
1 ignored issue
–
show
|
|||
| 86 | |||
| 87 | // Pipeline permissions |
||
| 88 | $approver = $this->objFromFixture('Member', 'approver'); |
||
| 89 | $approverbygroup = $this->objFromFixture('Member', 'approverbygroup'); |
||
| 90 | $canceller = $this->objFromFixture('Member', 'canceller'); |
||
| 91 | $cancellerbygroup = $this->objFromFixture('Member', 'cancellerbygroup'); |
||
| 92 | |||
| 93 | $this->assertTrue($environment->canApprove($approver)); |
||
|
1 ignored issue
–
show
|
|||
| 94 | $this->assertTrue($environment->canApprove($approverbygroup)); |
||
|
1 ignored issue
–
show
|
|||
| 95 | $this->assertFalse($environment->canApprove($canceller)); |
||
| 96 | $this->assertFalse($environment->canApprove($cancellerbygroup)); |
||
| 97 | |||
| 98 | $this->assertFalse($environment->canAbort($approver)); |
||
| 99 | $this->assertFalse($environment->canAbort($approverbygroup)); |
||
| 100 | $this->assertTrue($environment->canAbort($canceller)); |
||
|
1 ignored issue
–
show
|
|||
| 101 | $this->assertTrue($environment->canAbort($cancellerbygroup)); |
||
|
1 ignored issue
–
show
|
|||
| 102 | } |
||
| 103 | |||
| 168 |
You can fix this by adding a namespace to your class:
When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries.