summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSimo Sorce <simo@redhat.com>2015-02-24 17:34:09 -0500
committerSimo Sorce <simo@redhat.com>2015-02-24 17:48:44 -0500
commit755862e93a82355a6d2df35ac47faf10992ad466 (patch)
tree9805dd3eafcdcbea1dbfc88b49d69ffeebf5953b
parentb861b85d778bb2ba67f715568a26b08814c6fc89 (diff)
downloadipsilon-attrs-test.zip
ipsilon-attrs-test.tar.gz
ipsilon-attrs-test.tar.xz
Avoid attrs test flakines, stop using info_nssattrs-test
authtest already sets the fullname attribute, just use that one instead of relying on nss which, on test systems may have a completely empty gecos field, which makes the test fail. Signed-off-by: Simo Sorce <simo@redhat.com>
-rwxr-xr-xtests/attrs.py6
1 files changed, 2 insertions, 4 deletions
diff --git a/tests/attrs.py b/tests/attrs.py
index f29cbb4..4a3c8d8 100755
--- a/tests/attrs.py
+++ b/tests/attrs.py
@@ -41,7 +41,6 @@ idp_a = {'hostname': '${ADDRESS}:${PORT}',
'instance': '${NAME}',
'secure': 'no',
'testauth': 'yes',
- 'info_nss': 'yes',
'pam': 'no',
'krb': 'no',
'ipa': 'no',
@@ -123,8 +122,7 @@ if __name__ == '__main__':
idpname = 'idp1'
spname = 'sp1'
- userpwd = pwd.getpwuid(os.getuid())
- user = userpwd[0]
+ user = pwd.getpwuid(os.getuid())[0]
sess = HttpSessions()
sess.add_server(idpname, 'http://127.0.0.10:45080', user, 'ipsilon')
@@ -150,7 +148,7 @@ if __name__ == '__main__':
try:
page = sess.fetch_page(idpname,
'http://127.0.0.11:45081/sp/index.shtml')
- page.expected_value('text()', str(userpwd[4]))
+ page.expected_value('text()', 'Test User %s' % user)
except ValueError, e:
print >> sys.stderr, " ERROR: %s" % repr(e)
sys.exit(1)