Bug 1044482 - self-test fails (FAIL: test_pamcmds.sh)
Summary: self-test fails (FAIL: test_pamcmds.sh)
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: nss-pam-ldapd
Version: 7.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: rc
: ---
Assignee: Nalin Dahyabhai
QA Contact: David Spurek
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-12-18 12:07 UTC by David Spurek
Modified: 2015-03-02 05:28 UTC (History)
4 users (show)

Fixed In Version: nss-pam-ldapd-0.8.13-6.el7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-06-13 10:00:02 UTC
Target Upstream Version:


Attachments (Terms of Use)
A patch that expects error messages as printed by tools in RH based distros (1.26 KB, patch)
2014-01-20 14:40 UTC, Jakub Hrozek
no flags Details | Diff

Description David Spurek 2013-12-18 12:07:37 UTC
Description of problem:
Self test fails, expect script (test_pamcmds.expect) expects different outputs from su and passwd

Founded problems:
1) Most of the proc functions in test_pamcmds.expect expects "password updated successfully" {} output from passwd, but it returns: 'passwd: all authentication tokens updated successfully'

2)proc test_login_unknown expects "Unknown id" {} or "No passwd entry for user" {} but su returns 'su: user foo does not exist'

Version-Release number of selected component (if applicable):
nss-pam-ldapd-0.8.13-4.el7

How reproducible:
always

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Jakub Hrozek 2014-01-20 14:40:52 UTC
Created attachment 852763 [details]
A patch that expects error messages as printed by tools in RH based distros

I've verified that the messages the upstream test expects are the standard on Debian based distributions (and nss-pam-ldapd upstram uses Debian as reference platform).

After a short chat on IRC with Nalin, we decided to simply carry a distribution patch that would expect the messages our tools print.

Comment 2 Jakub Hrozek 2014-01-20 15:04:15 UTC
Here is a test build where I changed the expected error messages:
http://brewweb.devel.redhat.com/brew/taskinfo?taskID=6897544

David, does it work for you? If not, can you set me up a system with the beaker tests so I could take a look? Thanks!

Comment 3 David Spurek 2014-01-21 13:44:44 UTC
Patch looks fine, test_pamcmds.sh subtest now passes.

Comment 7 Ludek Smid 2014-06-13 10:00:02 UTC
This request was resolved in Red Hat Enterprise Linux 7.0.

Contact your manager or support representative in case you have further questions about the request.


Note You need to log in before you can comment on or make changes to this bug.