From: Gunnar Wrobel <p@rdus.de>
Date: Fri, 30 Oct 2009 23:19:10 +0000 (+0100)
Subject: Skip only those tests that really rely on Net_LDAP2
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=74d419a6028855c86c1595f1f653b0850e03c0b8;p=horde.git

Skip only those tests that really rely on Net_LDAP2
---

diff --git a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/ConfigurationTest.php b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/ConfigurationTest.php
index 8e7a6016e..03650533c 100644
--- a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/ConfigurationTest.php
+++ b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/ConfigurationTest.php
@@ -35,6 +35,7 @@ extends Horde_Kolab_Server_LdapTestCase
 {
     public function testMethodGetserverHasResultLoggedServerIfALoggerWasProvidedInTheConfiguration()
     {
+        $this->skipIfNoLdap();
         $factory = new Horde_Kolab_Server_Factory_Configuration(
             array('logger' => 'set', 'basedn' => '')
         );
@@ -43,6 +44,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetserverHasResultMappedServerIfAMappedWasProvidedInTheConfiguration()
     {
+        $this->skipIfNoLdap();
         $factory = new Horde_Kolab_Server_Factory_Configuration(
             array('map' => array(), 'basedn' => '')
         );
@@ -51,6 +53,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetserverHasResultCleanerServerIfACleanedWasProvidedInTheConfiguration()
     {
+        $this->skipIfNoLdap();
         $factory = new Horde_Kolab_Server_Factory_Configuration(
             array('cleanup' => true, 'basedn' => '')
         );
@@ -77,6 +80,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetserverHasResultServer()
     {
+        $this->skipIfNoLdap();
         $factory = new Horde_Kolab_Server_Factory_Configuration(
             array('basedn' => '')
         );
@@ -99,6 +103,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetconnectionHasResultServerconnection()
     {
+        $this->skipIfNoLdap();
         $factory = new Horde_Kolab_Server_Factory_Configuration(
             array('basedn' => '')
         );
@@ -110,6 +115,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetcompositeHasResultServercomposite()
     {
+        $this->skipIfNoLdap();
         $factory = new Horde_Kolab_Server_Factory_Configuration(
             array('basedn' => '')
         );
diff --git a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/Conn/ConfigurationTest.php b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/Conn/ConfigurationTest.php
index e081dbcfe..746c67381 100644
--- a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/Conn/ConfigurationTest.php
+++ b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/Conn/ConfigurationTest.php
@@ -71,6 +71,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetconnectionHasResultLdapConnectionIfConfiguredThatWay()
     {
+        $this->skipIfNoLdap();
         $factory = new Horde_Kolab_Server_Factory_Conn_Configuration(
             array('basedn' => 'a')
         );
diff --git a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/Conn/LdapTest.php b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/Conn/LdapTest.php
index b00c13717..880b81626 100644
--- a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/Conn/LdapTest.php
+++ b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/Conn/LdapTest.php
@@ -103,6 +103,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetconnectionHasResultConnectionSimpleldap()
     {
+        $this->skipIfNoLdap();
         $factory = new Horde_Kolab_Server_Factory_Conn_Ldap();
         $factory->setConfiguration(array('basedn' => 'test'));
         $this->assertType(
@@ -113,6 +114,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetconnectionHasResultConnectionSplittedldapIfTheHostMasterIsSet()
     {
+        $this->skipIfNoLdap();
         $factory = new Horde_Kolab_Server_Factory_Conn_Ldap();
         $factory->setConfiguration(array('basedn' => 'test', 'host_master' => 'dummy'));
         $this->assertType(
diff --git a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/ConstructorTest.php b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/ConstructorTest.php
index ed0ca15d5..6b719e9e6 100644
--- a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/ConstructorTest.php
+++ b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/ConstructorTest.php
@@ -53,6 +53,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetserverReturnsServer()
     {
+        $this->skipIfNoLdap();
         $this->factory->expects($this->once())
             ->method('getConnection')
             ->will(
@@ -99,6 +100,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetcompositeReturnsComposite()
     {
+        $this->skipIfNoLdap();
         $this->factory->expects($this->once())
             ->method('getConnection')
             ->will(
diff --git a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/InjectorTest.php b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/InjectorTest.php
index 3e522348a..85787a0e6 100644
--- a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/InjectorTest.php
+++ b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/InjectorTest.php
@@ -65,6 +65,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetserverHasResultServerldapstandard()
     {
+        $this->skipIfNoLdap();
         $factory = $this->_getFactory(array('basedn' => 'test'));
         $this->assertType(
             'Horde_Kolab_Server_Ldap_Standard',
@@ -88,6 +89,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetserverHasResultServerldapFilteredIfAFilterWasSet()
     {
+        $this->skipIfNoLdap();
         $factory = $this->_getFactory(array('filter' => 'test', 'basedn' => 'test'));
         $this->assertType(
             'Horde_Kolab_Server_Ldap_Filtered',
@@ -148,6 +150,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetcompositeHasResultComposite()
     {
+        $this->skipIfNoLdap();
         $factory = $this->_getFactory(array('basedn' => 'test'));
         $this->assertType(
             'Horde_Kolab_Server_Composite',
diff --git a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/KolabTest.php b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/KolabTest.php
index 64e4c29ad..e0bb3eb94 100644
--- a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/KolabTest.php
+++ b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Factory/KolabTest.php
@@ -35,7 +35,6 @@ extends Horde_Kolab_Server_LdapTestCase
 {
     public function setUp()
     {
-        parent::setUp();
         $this->conn_factory = $this->getMock('Horde_Kolab_Server_Factory_Conn');
         $this->connection = $this->getMock('Horde_Kolab_Server_Connection');
     }
@@ -126,6 +125,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetserverHasResultServerldapstandard()
     {
+        $this->skipIfNoLdap();
         $this->conn_factory->expects($this->once())
             ->method('getConnection')
             ->will($this->returnValue($this->connection));
@@ -140,6 +140,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetserverHasResultServerldapfilteredIfTheFilterOptionIsSet()
     {
+        $this->skipIfNoLdap();
         $this->conn_factory->expects($this->once())
             ->method('getConnection')
             ->will($this->returnValue($this->connection));
@@ -167,6 +168,7 @@ extends Horde_Kolab_Server_LdapTestCase
 
     public function testMethodGetcompositeHasResultComposite()
     {
+        $this->skipIfNoLdap();
         $this->conn_factory->expects($this->once())
             ->method('getConnection')
             ->will($this->returnValue($this->connection));
diff --git a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Ldap/FilteredTest.php b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Ldap/FilteredTest.php
index e23aabd0b..e91dc2a75 100644
--- a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Ldap/FilteredTest.php
+++ b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Ldap/FilteredTest.php
@@ -34,7 +34,7 @@ class Horde_Kolab_Server_Class_Server_Ldap_FilteredTest extends Horde_Kolab_Serv
 {
     public function setUp()
     {
-        parent::setUp();
+        $this->skipIfNoLdap();
 
         $this->ldap_read  = $this->getMock('Net_LDAP2');
         $this->ldap_write = $this->getMock('Net_LDAP2');
diff --git a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Ldap/StandardTest.php b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Ldap/StandardTest.php
index be5bf7cfc..3a704b72a 100644
--- a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Ldap/StandardTest.php
+++ b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Ldap/StandardTest.php
@@ -34,7 +34,7 @@ class Horde_Kolab_Server_Class_Server_Ldap_StandardTest extends Horde_Kolab_Serv
 {
     public function setUp()
     {
-        parent::setUp();
+        $this->skipIfNoLdap();
 
         $this->ldap_read  = $this->getMock('Net_LDAP2');
         $this->ldap_write = $this->getMock('Net_LDAP2');
diff --git a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/LdapTest.php b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/LdapTest.php
index 1bdfb16c3..6316fe055 100644
--- a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/LdapTest.php
+++ b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/LdapTest.php
@@ -34,7 +34,7 @@ class Horde_Kolab_Server_Class_Server_LdapTest extends Horde_Kolab_Server_LdapTe
 {
     public function setUp()
     {
-        parent::setUp();
+        $this->skipIfNoLdap();
 
         $this->ldap_read  = $this->getMock('Net_LDAP2');
         $this->ldap_write = $this->getMock('Net_LDAP2');
diff --git a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Structure/LdapTest.php b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Structure/LdapTest.php
index 96725b5d6..3db79724a 100644
--- a/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Structure/LdapTest.php
+++ b/framework/Kolab_Server/test/Horde/Kolab/Server/Class/Server/Structure/LdapTest.php
@@ -14,7 +14,7 @@
 /**
  * Require our basic test case definition
  */
-require_once dirname(__FILE__) . '/../../../Autoload.php';
+require_once dirname(__FILE__) . '/../../../LdapTestCase.php';
 
 /**
  * Test the LDAP backend.
@@ -30,7 +30,7 @@ require_once dirname(__FILE__) . '/../../../Autoload.php';
  * @license  http://www.fsf.org/copyleft/lgpl.html LGPL
  * @link     http://pear.horde.org/index.php?package=Kolab_Server
  */
-class Horde_Kolab_Server_Class_Server_Structure_LdapTest extends PHPUnit_Framework_TestCase
+class Horde_Kolab_Server_Class_Server_Structure_LdapTest extends Horde_Kolab_Server_LdapTestCase
 {
     public function setUp()
     {
@@ -46,6 +46,7 @@ class Horde_Kolab_Server_Class_Server_Structure_LdapTest extends PHPUnit_Framewo
 
     public function testMethodFindHasResultServerResultTheSearchResult()
     {
+        $this->skipIfNoLdap();
         $result = $this->getMock('Horde_Kolab_Server_Result');
         $this->composite->server->expects($this->exactly(1))
             ->method('find')
@@ -60,6 +61,7 @@ class Horde_Kolab_Server_Class_Server_Structure_LdapTest extends PHPUnit_Framewo
 
     public function testMethodFindBelowHasResultServerResultTheSearchResult()
     {
+        $this->skipIfNoLdap();
         $result = $this->getMock('Horde_Kolab_Server_Result');
         $this->composite->server->expects($this->exactly(1))
             ->method('findBelow')
diff --git a/framework/Kolab_Server/test/Horde/Kolab/Server/Connection/MockTest.php b/framework/Kolab_Server/test/Horde/Kolab/Server/Connection/MockTest.php
index 64cd092ed..23a4206ca 100644
--- a/framework/Kolab_Server/test/Horde/Kolab/Server/Connection/MockTest.php
+++ b/framework/Kolab_Server/test/Horde/Kolab/Server/Connection/MockTest.php
@@ -33,7 +33,6 @@ require_once dirname(__FILE__) . '/../LdapTestCase.php';
 class Horde_Kolab_Server_Connection_MockTest
 extends Horde_Kolab_Server_LdapTestCase
 {
-
     /**
      * Test parsing of LDAP filters.
      *
@@ -41,6 +40,8 @@ extends Horde_Kolab_Server_LdapTestCase
      */
     public function testFilterParse()
     {
+        $this->skipIfNoLdap();
+
         $conn = new Horde_Kolab_Server_Connection_Mock(array());
 
         $a = $conn->parse('(a=b)');
@@ -90,6 +91,8 @@ extends Horde_Kolab_Server_LdapTestCase
      */
     public function testSearch()
     {
+        $this->skipIfNoLdap();
+
         $conn = new Horde_Kolab_Server_Connection_Mock(
             array(
                 'data' =>
diff --git a/framework/Kolab_Server/test/Horde/Kolab/Server/LdapTestCase.php b/framework/Kolab_Server/test/Horde/Kolab/Server/LdapTestCase.php
index fb83bc07a..9cd106a2f 100644
--- a/framework/Kolab_Server/test/Horde/Kolab/Server/LdapTestCase.php
+++ b/framework/Kolab_Server/test/Horde/Kolab/Server/LdapTestCase.php
@@ -32,7 +32,7 @@ require_once dirname(__FILE__) . '/TestCase.php';
  */
 class Horde_Kolab_Server_LdapTestCase extends Horde_Kolab_Server_TestCase
 {
-    public function setUp()
+    public function skipIfNoLdap()
     {
         if (!extension_loaded('ldap') && !@dl('ldap.' . PHP_SHLIB_SUFFIX)) {
             $this->markTestSuiteSkipped('Ldap extension is missing!');
diff --git a/framework/Kolab_Server/test/Horde/Kolab/Server/Query/LdapTest.php b/framework/Kolab_Server/test/Horde/Kolab/Server/Query/LdapTest.php
index b6e941b6b..b8733af49 100644
--- a/framework/Kolab_Server/test/Horde/Kolab/Server/Query/LdapTest.php
+++ b/framework/Kolab_Server/test/Horde/Kolab/Server/Query/LdapTest.php
@@ -34,7 +34,7 @@ class Horde_Kolab_Server_Query_LdapTest extends Horde_Kolab_Server_LdapTestCase
 {
     public function setUp()
     {
-        parent::setUp();
+        $this->skipIfNoLdap();
         $this->structure = $this->getMock('Horde_Kolab_Server_Structure');
     }