Added environment-based Ldap server configuration for tests

Author: csarrazi

Tests/Adapter/ExtLdap/AdapterTest.php

@@ -19,7 +19,7 @@
 /**
  * @requires extension ldap
  */
-class AdapterTest extends \PHPUnit_Framework_TestCase
+class AdapterTest extends LdapTestCase
 {
     public function testLdapEscape()
     {
@@ -33,7 +33,7 @@ public function testLdapEscape()
      */
     public function testLdapQuery()
     {
-        $ldap = new Adapter(array('host' => 'localhost', 'port' => 3389));
+        $ldap = new Adapter($this->getLdapConfig());
 
         $ldap->getConnection()->bind('cn=admin,dc=symfony,dc=com', 'symfony');
         $query = $ldap->createQuery('dc=symfony,dc=com', '(&(objectclass=person)(ou=Maintainers))', array());

Tests/Adapter/ExtLdap/LdapManagerTest.php

@@ -19,14 +19,14 @@
 /**
  * @requires extension ldap
  */
-class LdapManagerTest extends \PHPUnit_Framework_TestCase
+class LdapManagerTest extends LdapTestCase
 {
     /** @var Adapter */
     private $adapter;
 
     protected function setUp()
     {
-        $this->adapter = new Adapter(array('host' => 'localhost', 'port' => 3389));
+        $this->adapter = new Adapter($this->getLdapConfig());
         $this->adapter->getConnection()->bind('cn=admin,dc=symfony,dc=com', 'symfony');
     }
 

Tests/LdapTestCase.php

@@ -0,0 +1,14 @@
+<?php
+
+namespace Symfony\Component\Ldap\Tests;
+
+class LdapTestCase extends \PHPUnit_Framework_TestCase
+{
+    protected function getLdapConfig()
+    {
+        return array(
+            'host' => getenv('LDAP_HOST'),
+            'port' => getenv('LDAP_PORT'),
+        );
+    }
+}

phpunit.xml.dist

@@ -8,6 +8,8 @@
 >
     <php>
         <ini name="error_reporting" value="-1" />
+        <env name="LDAP_HOST" value="127.0.0.1" />
+        <env name="LDAP_PORT" value="3389" />
     </php>
 
     <testsuites>