Updating ad_multihost test

* fixing raiseonerr=False to disjoin function * cleaned up code since the line limit has increased * added AD from forest1 to resolv.conf and /etc/hosts * updating test case documentation to clarify the test Signed-off-by: Dan Lavu <[email protected]> Reviewed-by: Jakub Vávra <[email protected]> Reviewed-by: Madhuri Upadhye <[email protected]> (cherry picked from commit cb72984)
SSSD · Oct 18, 2023 · c866b53 · c866b53
1 parent aa3616b
commit c866b53
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/tests/multihost/admultidomain/test_multiforest.py b/src/tests/multihost/admultidomain/test_multiforest.py
@@ -10,7 +10,7 @@
 @pytest.mark.admultiforest
 class TestADMultiForest(object):
 
-    def test_0001_multiforest(multihost, newhostname, adjoin):
+    def test_0001_multiforest(self, multihost, newhostname, adjoin):
         """
         :title: IDM-SSSD-TC: ad_provider: admultiforest : Authentication against two forests
         :id: 900f2467-1aca-430c-bbaa-b22d30a829ad
@@ -102,7 +102,7 @@ def test_0001_multiforest(multihost, newhostname, adjoin):
         id_domain1_user1 = multihost.client[0].run_command(f'id user1@{ad1_domain}', raiseonerr=False)
         id_domain1_user2 = multihost.client[0].run_command(f'id user2@{ad1_domain}', raiseonerr=False)
 
-        multihost.client[0].put_file_contents('/etc/hosts', hosts)
+        multihost.client[0].put_file_contents('/etc/hosts.bak', hosts)
         multihost.client[0].put_file_contents('/etc/krb5.conf', krb5)
         multihost.client[0].run_command(cleanup_krb5)
         client.restore_sssd_conf()