diff options
| author | Simo Sorce <simo@redhat.com> | 2015-02-24 17:34:09 -0500 |
|---|---|---|
| committer | Patrick Uiterwijk <puiterwijk@redhat.com> | 2015-02-24 23:57:47 +0100 |
| commit | 1d2fda874a7285e88b9aac870e7d9bc998fe7464 (patch) | |
| tree | 9805dd3eafcdcbea1dbfc88b49d69ffeebf5953b | |
| parent | b861b85d778bb2ba67f715568a26b08814c6fc89 (diff) | |
| download | ipsilon-1d2fda874a7285e88b9aac870e7d9bc998fe7464.tar.gz ipsilon-1d2fda874a7285e88b9aac870e7d9bc998fe7464.tar.xz ipsilon-1d2fda874a7285e88b9aac870e7d9bc998fe7464.zip | |
Avoid attrs test flakines, stop using info_nss
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>
Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
| -rwxr-xr-x | tests/attrs.py | 6 |
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) |