disable some tests with mmfiles to avoid hangs

jsteemann · jsteemann · commit 5f6964e3b02b · 2019-08-08T10:32:11.000+02:00
diff --git a/tests/CollectionBasicTest.php b/tests/CollectionBasicTest.php
@@ -39,8 +39,6 @@ public function setUp()
         $this->collectionHandler->create('ArangoDB_PHP_TestSuite_IndexTestCollection' . '_' . static::$testsTimestamp);
 
         $adminHandler = new AdminHandler($this->connection);
-        $version      = preg_replace('/-[a-z0-9]+$/', '', $adminHandler->getServerVersion());
-
         $this->isMMFilesEngine         = ($adminHandler->getEngine()["name"] == "mmfiles"); 
     }
 
diff --git a/tests/StreamingTransactionTest.php b/tests/StreamingTransactionTest.php
@@ -299,6 +299,9 @@ public function testGetCollection()
     
     public function testInsert()
     {
+        if ($this->isMMFilesEngine) {
+            $this->markTestSkipped("test is only meaningful with the rocksdb engine");
+        }
         $trx = new StreamingTransaction($this->connection, [
             TransactionBase::ENTRY_COLLECTIONS => [
                 TransactionBase::ENTRY_WRITE => [ $this->collection1->getName() ]
@@ -342,6 +345,9 @@ public function testInsert()
     
     public function testRemove()
     {
+        if ($this->isMMFilesEngine) {
+            $this->markTestSkipped("test is only meaningful with the rocksdb engine");
+        }
         // insert a document before the transaction
         $documentHandler = new DocumentHandler($this->connection);
         $result = $documentHandler->save($this->collection1->getName(), [ '_key' => 'test', 'value' => 'test' ]);
@@ -400,6 +406,9 @@ public function testRemove()
     
     public function testUpdate()
     {
+        if ($this->isMMFilesEngine) {
+            $this->markTestSkipped("test is only meaningful with the rocksdb engine");
+        }
         // insert a document before the transaction
         $documentHandler = new DocumentHandler($this->connection);
         $result = $documentHandler->save($this->collection1->getName(), [ '_key' => 'test', 'value' => 'test' ]);
@@ -450,6 +459,9 @@ public function testUpdate()
     
     public function testReplace()
     {
+        if ($this->isMMFilesEngine) {
+            $this->markTestSkipped("test is only meaningful with the rocksdb engine");
+        }
         // insert a document before the transaction
         $documentHandler = new DocumentHandler($this->connection);
         $result = $documentHandler->save($this->collection1->getName(), [ '_key' => 'test', 'value' => 'test' ]);
@@ -504,6 +516,9 @@ public function testReplace()
     
     public function testTruncate()
     {
+        if ($this->isMMFilesEngine) {
+            $this->markTestSkipped("test is only meaningful with the rocksdb engine");
+        }
         $stmt = new Statement($this->connection, [
             'query' => 'FOR i IN 1..10 INSERT { _key: CONCAT("test", i), value: i } INTO @@collection',
             'bindVars' => [ '@collection' => $this->collection1->getName() ]
@@ -554,6 +569,9 @@ public function testTruncate()
     
     public function testQuery()
     {
+        if ($this->isMMFilesEngine) {
+            $this->markTestSkipped("test is only meaningful with the rocksdb engine");
+        }
         $trx = new StreamingTransaction($this->connection, [
             TransactionBase::ENTRY_COLLECTIONS => [
                 TransactionBase::ENTRY_WRITE => [ $this->collection1->getName() ]

Original file line number	Diff line number	Diff line change
`@@ -39,8 +39,6 @@ public function setUp()`
`39`	`39`	`$this->collectionHandler->create('ArangoDB_PHP_TestSuite_IndexTestCollection' . '_' . static::$testsTimestamp);`
`40`	`40`
`41`	`41`	`$adminHandler = new AdminHandler($this->connection);`
`42`		`- $version = preg_replace('/-[a-z0-9]+$/', '', $adminHandler->getServerVersion());`
`43`		`-`
`44`	`42`	`$this->isMMFilesEngine = ($adminHandler->getEngine()["name"] == "mmfiles");`
`45`	`43`	`}`
`46`	`44`