From d3c90b329f2cee1072ba7b68a74640235866192b Mon Sep 17 00:00:00 2001
From: Varun Mylaraiah <mvarun@redhat.com>
Date: Nov 26 2018 09:30:10 +0000
Subject: Added test for ipa-client-install with a non-standard ldap.conf file Ticket: https://pagure.io/freeipa/issue/7418


Signed-off-by: Varun Mylaraiah <mvarun@redhat.com>
Reviewed-By: Mohammad Rizwan Yusuf <myusuf@redhat.com>
Reviewed-By: Florence Blanc-Renaud <flo@redhat.com>

---

diff --git a/ipatests/test_integration/test_installation.py b/ipatests/test_integration/test_installation.py
index b7a9628..1eefbfb 100644
--- a/ipatests/test_integration/test_installation.py
+++ b/ipatests/test_integration/test_installation.py
@@ -176,6 +176,33 @@ class TestInstallWithCA1(InstallTestBase1):
     def test_replica2_ipa_dns_install(self):
         super(TestInstallWithCA1, self).test_replica2_ipa_dns_install()
 
+    def test_install_with_bad_ldap_conf(self):
+        """
+        Test a client install with a non standard ldap.config
+        https://pagure.io/freeipa/issue/7418
+        """
+        ldap_conf = paths.OPENLDAP_LDAP_CONF
+        base_dn = self.master.domain.basedn  # pylint: disable=no-member
+        client = self.replicas[0]
+        tasks.uninstall_master(client)
+        expected_msg1 = "contains deprecated and unsupported " \
+                        "entries: HOST, PORT"
+        file_backup = client.get_file_contents(ldap_conf, encoding='utf-8')
+        constants = "URI ldaps://{}\nBASE {}\nHOST {}\nPORT 636".format(
+            self.master.hostname, base_dn,
+            self.master.hostname)
+        modifications = "{}\n{}".format(file_backup, constants)
+        client.put_file_contents(paths.OPENLDAP_LDAP_CONF, modifications)
+        result = client.run_command(['ipa-client-install', '-U',
+                                     '--domain', client.domain.name,
+                                     '--realm', client.domain.realm,
+                                     '-p', client.config.admin_name,
+                                     '-w', client.config.admin_password,
+                                     '--server', self.master.hostname],
+                                    raiseonerr=False)
+        assert expected_msg1 in result.stderr_text
+        client.put_file_contents(ldap_conf, file_backup)
+
 
 class TestInstallWithCA2(InstallTestBase2):
 
@@ -217,7 +244,6 @@ class TestInstallWithCA_KRA2(InstallTestBase2):
         tasks.install_master(cls.master, setup_dns=False, setup_kra=True)
 
 
-
 class TestInstallWithCA_DNS1(InstallTestBase1):
 
     @classmethod