test: Fail on exceptions

If we don't output a test plan prove will consider the test failed. The
destructor would run when an uncaught exception is thrown so don't use
a destructor to output the testplan.

Signed-off-by: Florian Pritz <bluewind@xinu.at>

2 files changed, 2 insertions, 1 deletions

diff --git a/application/controllers/tools.php b/application/controllers/tools.php
index e36b09b79..e226b1815 100644
--- a/application/controllers/tools.php
+++ b/application/controllers/tools.php
@@ -82,5 +82,6 @@ class Tools extends MY_Controller {
 				$test->cleanup();
 			}
 		}
+		$test->done_testing();
 	}
 }
diff --git a/application/tests/Test.php b/application/tests/Test.php
index e18aa9374..1f2e4fefa 100644
--- a/application/tests/Test.php
+++ b/application/tests/Test.php
@@ -102,7 +102,7 @@ abstract class Test {
 	{
 	}
 
-	public function __destruct()
+	public function done_testing()
 	{
 		$this->t->done_testing();
 	}