Bug 1044482

Summary: self-test fails (FAIL: test_pamcmds.sh)
Product: Red Hat Enterprise Linux 7 Reporter: David Spurek <dspurek>
Component: nss-pam-ldapdAssignee: Nalin Dahyabhai <nalin>
Status: CLOSED CURRENTRELEASE QA Contact: David Spurek <dspurek>
Severity: high Docs Contact:
Priority: high    
Version: 7.0CC: dspurek, ebenes, jhrozek, mkosek
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: nss-pam-ldapd-0.8.13-6.el7 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-06-13 10:00:02 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
A patch that expects error messages as printed by tools in RH based distros none

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.