Skip tests that rely on DB if DB is not MySQL

silverstripe · Aug 21, 2018 · 1f6fcd9 · 1f6fcd9
1 parent df45d32
commit 1f6fcd9
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 0 deletions.
diff --git a/tests/DatabaseStoreTest.php b/tests/DatabaseStoreTest.php
@@ -2,11 +2,22 @@
 
 namespace SilverStripe\HybridSessions\Tests;
 
+use SilverStripe\ORM\DB;
+use SilverStripe\ORM\Connect\MySQLDatabase;
 use SilverStripe\Core\Injector\Injector;
 use SilverStripe\HybridSessions\Store\DatabaseStore;
 
 class DatabaseStoreTest extends AbstractTest
 {
+    protected function setUp()
+    {
+        parent::setUp();
+
+        if (!DB::get_conn() instanceof MySQLDatabase) {
+            $this->markTestSkipped('Only MySQL databases are supported');
+        }
+    }
+
     protected function getStore()
     {
         $store = Injector::inst()->get(DatabaseStore::class);

diff --git a/tests/HybridSessionTest.php b/tests/HybridSessionTest.php
@@ -2,13 +2,30 @@
 
 namespace SilverStripe\HybridSessions\Tests;
 
+use SilverStripe\ORM\DB;
+use SilverStripe\ORM\Connect\MySQLDatabase;
 use SilverStripe\Core\Injector\Injector;
+use SilverStripe\Core\Config\Config;
 use SilverStripe\Core\TempFolder;
 use SilverStripe\HybridSessions\HybridSession;
 
 class HybridSessionTest extends AbstractTest
 {
 
+    protected function setUp()
+    {
+        parent::setUp();
+
+        if (!DB::get_conn() instanceof MySQLDatabase) {
+            // we can't always use the DB driver, so remove it if we aren't running a MySQL DB
+            Config::modify()->set(HybridSession::class, 'dependencies', [
+                'handlers' => [
+                    '%$\SilverStripe\HybridSessions\Store\CookieStore',
+                ],
+            ]);
+        }
+    }
+
     /**
      * @return HybridSessionStore_Cookie
      */