Shortened error output of unexpected exit to one line.

Slamdunk · Naktibalda · commit d5333a5e5ea4 · 2019-07-30T20:50:36.000+03:00
* Test early exits

* Exit with error code

* Test exit code
diff --git a/src/Codeception/Subscriber/ErrorHandler.php b/src/Codeception/Subscriber/ErrorHandler.php
@@ -98,8 +98,10 @@ public function shutdownHandler()
         if (!$this->suiteFinished && (
             $error === null || !in_array($error['type'], [E_ERROR, E_COMPILE_ERROR, E_CORE_ERROR])
         )) {
-            throw new \RuntimeException('Command Did Not Finish Properly');
-        } elseif (!is_array($error)) {
+            echo "\n\n\nCOMMAND DID NOT FINISH PROPERLY.\n";
+            exit(255);
+        }
+        if (!is_array($error)) {
             return;
         }
         if (error_reporting() === 0) {
diff --git a/tests/cli/ErrorHandlerCest.php b/tests/cli/ErrorHandlerCest.php
@@ -0,0 +1,14 @@
+<?php
+class ErrorHandlerCest
+{
+    /**
+     * @param CliGuy $I
+     */
+    public function earlyExitWarnsTheUser(\CliGuy $I)
+    {
+        $I->executeFailCommand('run -c tests/data/first_test_exits');
+
+        $I->seeResultCodeIs(255);
+        $I->seeInShellOutput('COMMAND DID NOT FINISH PROPERLY');
+    }
+}
diff --git a/tests/data/first_test_exits/codeception.yml b/tests/data/first_test_exits/codeception.yml
@@ -0,0 +1,11 @@
+actor: Tester
+paths:
+    tests: tests
+    log: tests/_output
+    data: tests/_data
+    support: tests/_support
+    envs: tests/_envs
+settings:
+    bootstrap: _bootstrap.php
+    colors: true
+    memory_limit: 1024M
diff --git a/tests/data/first_test_exits/tests/_bootstrap.php b/tests/data/first_test_exits/tests/_bootstrap.php
@@ -0,0 +1,2 @@
+<?php
+// This is global bootstrap for autoloading
diff --git a/tests/data/first_test_exits/tests/_output/.gitignore b/tests/data/first_test_exits/tests/_output/.gitignore
@@ -0,0 +1,2 @@
+*
+!.gitignore
diff --git a/tests/data/first_test_exits/tests/_support/UnitTester.php b/tests/data/first_test_exits/tests/_support/UnitTester.php
@@ -0,0 +1,26 @@
+<?php
+
+
+/**
+ * Inherited Methods
+ * @method void wantToTest($text)
+ * @method void wantTo($text)
+ * @method void execute($callable)
+ * @method void expectTo($prediction)
+ * @method void expect($prediction)
+ * @method void amGoingTo($argumentation)
+ * @method void am($role)
+ * @method void lookForwardTo($achieveValue)
+ * @method void comment($description)
+ * @method \Codeception\Lib\Friend haveFriend($name, $actorClass = NULL)
+ *
+ * @SuppressWarnings(PHPMD)
+*/
+class UnitTester extends \Codeception\Actor
+{
+    use _generated\UnitTesterActions;
+
+   /**
+    * Define custom actions here
+    */
+}
diff --git a/tests/data/first_test_exits/tests/_support/_generated/.gitignore b/tests/data/first_test_exits/tests/_support/_generated/.gitignore
@@ -0,0 +1,2 @@
+*
+!.gitignore
diff --git a/tests/data/first_test_exits/tests/unit.suite.yml b/tests/data/first_test_exits/tests/unit.suite.yml
@@ -0,0 +1,5 @@
+# Codeception Test Suite Configuration
+#
+# Suite for unit (internal) tests.
+
+class_name: UnitTester
diff --git a/tests/data/first_test_exits/tests/unit/ExitCest.php b/tests/data/first_test_exits/tests/unit/ExitCest.php
@@ -0,0 +1,10 @@
+<?php
+
+
+class ExitCest
+{
+    public function exitNow(UnitTester $I)
+    {
+        exit(0);
+    }
+}
diff --git a/tests/data/first_test_exits/tests/unit/_bootstrap.php b/tests/data/first_test_exits/tests/unit/_bootstrap.php
@@ -0,0 +1,2 @@
+<?php
+// Here you can initialize variables that will be available to your tests

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+<?php`
	`2`	`+// This is global bootstrap for autoloading`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+<?php`
	`2`	`+// Here you can initialize variables that will be available to your tests`