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(a)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(a)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