Reverted and improved TODO:

------
@nupplaphil: There is no easy possibility to add a logger here, that's why there
isn't any yet and instead a placeholder.. This execution point is a critical
state during a testrun, and tbh I'd like to leave here no further logic (yet)
because I spent hours debugging cases, where transactions weren't fully closed
and strange/unpredictable errors occur (sometimes -mainly during debugging other
errors :) ...)
-----

diff --git a/tests/Util/Database/StaticDatabase.php b/tests/Util/Database/StaticDatabase.php
index 22c44affdad99988106d8540ca0d84ff2ebfa144..2c05b921b5da791996a670806ffdd0aea76e899a 100644 (file)
--- a/tests/Util/Database/StaticDatabase.php
+++ b/tests/Util/Database/StaticDatabase.php
@@ -179,8 +179,7 @@ class StaticDatabase extends Database
                        self::$staticConnection = @new ExtendedPDO($connect, $user, $pass);
                        self::$staticConnection->setAttribute(PDO::ATTR_AUTOCOMMIT,0);
                } catch (PDOException $e) {
-                       // Log exception
-                       $this->logger::error('Cannot establish database connection', ['exception' => $e, 'host' => $server, 'dbname' => $db]);
+                       /// @TODO Try to find a way to log this exception as it contains valueable information
                }
        }