summaryrefslogtreecommitdiffstats
path: root/ipatests
diff options
context:
space:
mode:
authorTomas Babej <tbabej@redhat.com>2014-02-27 15:40:54 +0100
committerMartin Kosek <mkosek@redhat.com>2014-03-24 09:51:52 +0100
commit8c8cc8b6146f286023b2398ca99feae784b0993d (patch)
treeadc448de5a9fe1b3960d50f69f76aec57874d8d6 /ipatests
parent20c716ec9a69a4a24ff4138471a0ce457cabf99c (diff)
downloadfreeipa-8c8cc8b6146f286023b2398ca99feae784b0993d.tar.gz
freeipa-8c8cc8b6146f286023b2398ca99feae784b0993d.tar.xz
freeipa-8c8cc8b6146f286023b2398ca99feae784b0993d.zip
ipatests: test_trust: Change expected home directories for posix users
Information from the AD about the home directories is not leveraged at all, but is generated from the username and domain. Fix the assumptions in the tests. Also changes 'Subdomain Test User' to 'Subdomaintest User' to be more consistent. https://fedorahosted.org/freeipa/ticket/4184 Reviewed-By: Jakub Hrozek <jhrozek@redhat.com> Reviewed-By: Alexander Bokovoy <abokovoy@redhat.com>
Diffstat (limited to 'ipatests')
-rw-r--r--ipatests/test_integration/test_legacy_clients.py4
-rw-r--r--ipatests/test_integration/test_trust.py5
2 files changed, 5 insertions, 4 deletions
diff --git a/ipatests/test_integration/test_legacy_clients.py b/ipatests/test_integration/test_legacy_clients.py
index d70dd1010..072a2939d 100644
--- a/ipatests/test_integration/test_legacy_clients.py
+++ b/ipatests/test_integration/test_legacy_clients.py
@@ -248,7 +248,7 @@ class BaseTestLegacyClient(object):
result = self.legacy_client.run_command(['getent', 'passwd', testuser])
testuser_regex = "subdomaintestuser@%s:\*:%s:%s:"\
- "Subdomain Test User:%s:"\
+ "Subdomaintest User:%s:"\
"/bin/sh"\
% (re.escape(self.ad_subdomain),
self.subdomain_testuser_uid_regex,
@@ -405,7 +405,7 @@ class BaseTestLegacyClientPosix(BaseTestLegacyClient,
testuser_gid_regex = '10047'
subdomain_testuser_uid_regex = '10142'
subdomain_testuser_gid_regex = '10147'
- homedir_template = "/home/{username}"
+ homedir_template = "/home/{domain}/{username}"
posix_trust = True
def test_remove_trust_with_posix_attributes(self):
diff --git a/ipatests/test_integration/test_trust.py b/ipatests/test_integration/test_trust.py
index ec545baba..4afc13440 100644
--- a/ipatests/test_integration/test_trust.py
+++ b/ipatests/test_integration/test_trust.py
@@ -131,8 +131,9 @@ class TestPosixADTrust(ADTrustBase):
result = self.master.run_command(['getent', 'passwd', testuser])
testuser_stdout = "testuser@%s:*:10042:10047:"\
- "Test User:/home/testuser:/bin/sh"\
- % self.ad.domain.name
+ "Test User:/home/%s/testuser:/bin/sh"\
+ % (self.ad.domain.name,
+ self.ad.domain.name)
assert testuser_stdout in result.stdout_text