From 34100aa767440a10b63e36ef6b7e2de3c86e681e Mon Sep 17 00:00:00 2001
From: Bernard Hamlin <bernie@silverstripe.com>
Date: Wed, 13 Mar 2019 09:16:31 +1300
Subject: [PATCH] Skip Temp DB creation if state has no database prop

---
 src/TestSessionEnvironment.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/TestSessionEnvironment.php b/src/TestSessionEnvironment.php
index fe42208..a9889e3 100644
--- a/src/TestSessionEnvironment.php
+++ b/src/TestSessionEnvironment.php
@@ -288,6 +288,7 @@ class TestSessionEnvironment
 
         // Database
         if (!$this->isRunningTests()) {
+            $dbCreate = isset($state->createDatabase) ? (bool) $state->createDatabase : false;
             $dbName = (isset($state->database)) ? $state->database : null;
 
             if ($dbName) {
@@ -296,7 +297,7 @@ class TestSessionEnvironment
                 $dbExists = false;
             }
 
-            if (!$dbExists) {
+            if (!$dbExists && $dbCreate) {
                 // Create a new one with a randomized name
                 $tempDB = new TempDatabase();
                 $dbName = $tempDB->build();
@@ -310,7 +311,6 @@ class TestSessionEnvironment
                     throw new InvalidArgumentException("Invalid database name format");
                 }
 
-                $this->oldDatabaseName = $databaseConfig['database'];
                 $databaseConfig['database'] = $dbName; // Instead of calling DB::set_alternative_db_name();
 
                 // Connect to the new database, overwriting the old DB connection (if any)