MAGETWO-65199: [GitHub][PR] [PSR-2 Compliance] Fix #8612: Hundreds of PHPCS-based static tests violations in mainline #8685

Oleksii Korshenko · Oleksii Korshenko · commit 00279baa578b · 2017-03-14T17:47:47.000-05:00
- fixed code style issue
diff --git a/dev/tests/static/testsuite/Magento/Test/Php/LiveCodeTest.php b/dev/tests/static/testsuite/Magento/Test/Php/LiveCodeTest.php
@@ -198,7 +198,7 @@ private function getFullWhitelist()
         return Files::init()->readLists(__DIR__ . '/_files/whitelist/common.txt');
     }
 
-    public function testNoViolationsDetectedByPhpCodeSniffer()
+    public function testCodeStyle()
     {
         $reportFile = self::$reportDir . '/phpcs_report.txt';
         $codeSniffer = new CodeSniffer('Magento', $reportFile, new Wrapper());
@@ -209,7 +209,7 @@ public function testNoViolationsDetectedByPhpCodeSniffer()
         );
     }
 
-    public function testNoViolationsDetectedByPhpMessDetector()
+    public function testCodeMess()
     {
         $reportFile = self::$reportDir . '/phpmd_report.txt';
         $codeMessDetector = new CodeMessDetector(realpath(__DIR__ . '/_files/phpmd/ruleset.xml'), $reportFile);
@@ -237,7 +237,7 @@ public function testNoViolationsDetectedByPhpMessDetector()
         }
     }
 
-    public function testNoViolationsDetectedByPhpCopyPasteDetector()
+    public function testCopyPaste()
     {
         $reportFile = self::$reportDir . '/phpcpd_report.xml';
         $copyPasteDetector = new CopyPasteDetector($reportFile);

Original file line number	Diff line number	Diff line change
`@@ -198,7 +198,7 @@ private function getFullWhitelist()`
`198`	`198`	`return Files::init()->readLists(__DIR__ . '/_files/whitelist/common.txt');`
`199`	`199`	`}`
`200`	`200`
`201`		`- public function testNoViolationsDetectedByPhpCodeSniffer()`
	`201`	`+ public function testCodeStyle()`
`202`	`202`	`{`
`203`	`203`	`$reportFile = self::$reportDir . '/phpcs_report.txt';`
`204`	`204`	`$codeSniffer = new CodeSniffer('Magento', $reportFile, new Wrapper());`
`@@ -209,7 +209,7 @@ public function testNoViolationsDetectedByPhpCodeSniffer()`
`209`	`209`	`);`
`210`	`210`	`}`
`211`	`211`
`212`		`- public function testNoViolationsDetectedByPhpMessDetector()`
	`212`	`+ public function testCodeMess()`
`213`	`213`	`{`
`214`	`214`	`$reportFile = self::$reportDir . '/phpmd_report.txt';`
`215`	`215`	`$codeMessDetector = new CodeMessDetector(realpath(__DIR__ . '/_files/phpmd/ruleset.xml'), $reportFile);`
`@@ -237,7 +237,7 @@ public function testNoViolationsDetectedByPhpMessDetector()`
`237`	`237`	`}`
`238`	`238`	`}`
`239`	`239`
`240`		`- public function testNoViolationsDetectedByPhpCopyPasteDetector()`
	`240`	`+ public function testCopyPaste()`
`241`	`241`	`{`
`242`	`242`	`$reportFile = self::$reportDir . '/phpcpd_report.xml';`
`243`	`243`	`$copyPasteDetector = new CopyPasteDetector($reportFile);`