[SSSD] [PATCH] TESTS: extend PAM responder unit test

Lukas Slebodnik lslebodn at redhat.com
Thu Nov 12 08:42:21 UTC 2015


On (12/11/15 09:35), Lukas Slebodnik wrote:
>On (11/11/15 18:04), Pavel Reichl wrote:
>>
>>
>>On 11/10/2015 05:00 PM, Lukas Slebodnik wrote:
>>[snip]
>>>I tried it and all test passed.
>>>
>>>Please fix small issues and you have my ACK.
>>>
>>>LS
>>
>>Thanks, updated patch set attached.
>
>>From 5f43c3ddef195a7a480755306349555e059778a3 Mon Sep 17 00:00:00 2001
>>From: Pavel Reichl <preichl at redhat.com>
>>Date: Tue, 20 Oct 2015 08:07:02 -0400
>>Subject: [PATCH 1/2] pam-srv-tests: split pam_test_setup() so it can be reused
>>
>>Split pam_test_setup() so domain and pam parameters can be easily set
>>distinctly for each test.
>>
>>Resolves:
>>https://fedorahosted.org/sssd/ticket/2697
>>---
>ACK
>
>
>>From f9d4ecb2ecea78cb974b0b375210c7be19721cfb Mon Sep 17 00:00:00 2001
>>From: Pavel Reichl <preichl at redhat.com>
>>Date: Tue, 20 Oct 2015 09:10:30 -0400
>>Subject: [PATCH 2/2] pam-srv-tests: Add UT for cached 'online' auth.
>>
>>Extend PAM responder unit test to check 'online' cached authentication.
>>
>>Resolves:
>>https://fedorahosted.org/sssd/ticket/2697
>>---
>> src/responder/pam/pamsrv.h      |   5 ++
>> src/responder/pam/pamsrv_cmd.c  |   2 +-
>> src/tests/cmocka/test_pam_srv.c | 188 ++++++++++++++++++++++++++++++++++++++++
>> 3 files changed, 194 insertions(+), 1 deletion(-)
>>
>>diff --git a/src/responder/pam/pamsrv.h b/src/responder/pam/pamsrv.h
>>index 59831f2e73f923053e53cad838fac715330546dd..64a7d85735ac3eb20e4504899916a49a280d4959 100644
>ACK
>
>http://sssd-ci.duckdns.org/logs/job/32/97/summary.html
>

master:
* 4b12be504e20173e0629835818e4db6a9617a9a4
* 6411cd6074688762f793de8f1dddeffcb3a71d02

sssd-1-13:
* f0c4ca6e39f6ab31c146e634a2eb87965aea66d0
* b0bdc28deae3c42526c0a0069d87c49d89d23a9f

LS


More information about the sssd-devel mailing list