minor #23934 [Console] Require PHP 7 for ApplicationTest (siwinski)

Robin Chalas · Robin Chalas · commit 7ce0665c3b93 · 2017-08-24T00:35:30.000+02:00
This PR was merged into the 3.3 branch. Discussion ---------- [Console] Require PHP 7 for ApplicationTest | Q | A | ------------- | --- | Branch? | 3.3  | Bug fix? | no | New feature? | no  | BC breaks? | no | Deprecations? | no  | Tests pass? | yes | Fixed tickets |  | License | MIT | Doc PR |  Since [ApplicationTest](https://github.com/symfony/symfony/blob/3.3/src/Symfony/Component/Console/Tests/ApplicationTest.php) uses the `Error` class, this component should require-dev `symfony/polyfill-php70` Commits ------- 94d55ca [Console] Require PHP 7 for ApplicationTest
diff --git a/src/Symfony/Component/Console/Tests/ApplicationTest.php b/src/Symfony/Component/Console/Tests/ApplicationTest.php
@@ -1040,6 +1040,9 @@ public function testRunDispatchesAllEventsWithExceptionInListener()
         $this->assertContains('before.error.after.', $tester->getDisplay());
     }
 
+    /**
+     * @requires PHP 7
+     */
     public function testRunWithError()
     {
         $application = new Application();
@@ -1160,6 +1163,7 @@ public function testErrorIsRethrownIfNotHandledByConsoleErrorEvent()
     }
 
     /**
+     * @requires PHP 7
      * @expectedException        \LogicException
      * @expectedExceptionMessage error
      */
@@ -1181,6 +1185,9 @@ public function testRunWithErrorAndDispatcher()
         $this->assertContains('before.dym.error.after.', $tester->getDisplay(), 'The PHP Error did not dispached events');
     }
 
+    /**
+     * @requires PHP 7
+     */
     public function testRunDispatchesAllEventsWithError()
     {
         $application = new Application();
@@ -1198,6 +1205,9 @@ public function testRunDispatchesAllEventsWithError()
         $this->assertContains('before.dym.error.after.', $tester->getDisplay(), 'The PHP Error did not dispached events');
     }
 
+    /**
+     * @requires PHP 7
+     */
     public function testRunWithErrorFailingStatusCode()
     {
         $application = new Application();

Original file line number	Diff line number	Diff line change
`@@ -1040,6 +1040,9 @@ public function testRunDispatchesAllEventsWithExceptionInListener()`
`1040`	`1040`	`$this->assertContains('before.error.after.', $tester->getDisplay());`
`1041`	`1041`	`}`
`1042`	`1042`
	`1043`	`+ /**`
	`1044`	`+ * @requires PHP 7`
	`1045`	`+ */`
`1043`	`1046`	`public function testRunWithError()`
`1044`	`1047`	`{`
`1045`	`1048`	`$application = new Application();`
`@@ -1160,6 +1163,7 @@ public function testErrorIsRethrownIfNotHandledByConsoleErrorEvent()`
`1160`	`1163`	`}`
`1161`	`1164`
`1162`	`1165`	`/**`
	`1166`	`+ * @requires PHP 7`
`1163`	`1167`	`* @expectedException \LogicException`
`1164`	`1168`	`* @expectedExceptionMessage error`
`1165`	`1169`	`*/`
`@@ -1181,6 +1185,9 @@ public function testRunWithErrorAndDispatcher()`
`1181`	`1185`	`$this->assertContains('before.dym.error.after.', $tester->getDisplay(), 'The PHP Error did not dispached events');`
`1182`	`1186`	`}`
`1183`	`1187`
	`1188`	`+ /**`
	`1189`	`+ * @requires PHP 7`
	`1190`	`+ */`
`1184`	`1191`	`public function testRunDispatchesAllEventsWithError()`
`1185`	`1192`	`{`
`1186`	`1193`	`$application = new Application();`
`@@ -1198,6 +1205,9 @@ public function testRunDispatchesAllEventsWithError()`
`1198`	`1205`	`$this->assertContains('before.dym.error.after.', $tester->getDisplay(), 'The PHP Error did not dispached events');`
`1199`	`1206`	`}`
`1200`	`1207`
	`1208`	`+ /**`
	`1209`	`+ * @requires PHP 7`
	`1210`	`+ */`
`1201`	`1211`	`public function testRunWithErrorFailingStatusCode()`
`1202`	`1212`	`{`
`1203`	`1213`	`$application = new Application();`