[SSSD] [PATCH] intg_tests: Add regression test for 2163

Jakub Hrozek jhrozek at redhat.com
Wed Jul 22 09:43:54 UTC 2015


On Mon, Jul 20, 2015 at 03:52:52PM +0200, Lukas Slebodnik wrote:
> On (20/07/15 15:46), Lukas Slebodnik wrote:
> >ehlo,
> >
> >attached is a regression test for old ticket 2163.
> >Currently it fails due to fix for ticket 2717
> >
> And now with attached patch.
> 
> BTW I think that we should split fixtures
> and statiting sssd and loading data to LDAP should be two separate fixtures.
> 
> LS

> From cd765024b9662eef537e1a84f1da23ce29173dcd Mon Sep 17 00:00:00 2001
> From: Lukas Slebodnik <lslebodn at redhat.com>
> Date: Mon, 20 Jul 2015 14:09:52 +0200
> Subject: [PATCH] intg_tests: Add regression test for 2163
> 
> ---
>  src/tests/intg/ldap_test.py | 40 ++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 40 insertions(+)
> 
> diff --git a/src/tests/intg/ldap_test.py b/src/tests/intg/ldap_test.py
> index afc77d7023ea31757def84f21e5d15997ba72013..914e0d0cc8680d73f0caed233965dfc537daf896 100644
> --- a/src/tests/intg/ldap_test.py
> +++ b/src/tests/intg/ldap_test.py
> @@ -141,6 +141,38 @@ def sanity_rfc2307(request, ldap_conn):
>      create_sssd_fixture(request)
>      return None
>  
> +
> + at pytest.fixture
> +def simple_rfc2307(request, ldap_conn):
> +    ent_list = ldap_ent.List(LDAP_BASE_DN)
> +    ent_list.add_user('usr\\\\001', 181818, 181818)
> +    ent_list.add_group("group1", 181818)
> +
> +    create_ldap_fixture(request, ldap_conn, ent_list)
> +
> +    conf = unindent("""\
> +        [sssd]
> +        config_file_version = 2
> +        domains             = LDAP
> +        services            = nss, pam
> +
> +        [nss]
> +
> +        [pam]
> +
> +        [domain/LDAP]
> +        ldap_auth_disable_tls_never_use_in_production = true
> +        ldap_schema         = rfc2307
> +        id_provider         = ldap
> +        auth_provider       = ldap
> +        ldap_uri            = {ldap_conn.ds_inst.ldap_url}
> +        ldap_search_base    = {ldap_conn.ds_inst.base_dn}
> +    """).format(**locals())
> +    create_conf_fixture(request, conf)
> +    create_sssd_fixture(request)
> +    return None
> +
> +
>  @pytest.fixture
>  def sanity_rfc2307_bis(request, ldap_conn):
>      ent_list = ldap_ent.List(LDAP_BASE_DN)
> @@ -200,6 +232,14 @@ def sanity_rfc2307_bis(request, ldap_conn):
>      create_sssd_fixture(request)
>      return None
>  
> +
> +def test_refression_ticket2163(ldap_conn, simple_rfc2307):
       ~~~~~~~~~~
       typo: Should read regression.

Otherwise LGTM.

> +    ent.assert_passwd_by_name(
> +        'usr\\001',
> +        dict(name='usr\\001', passwd='*', uid=181818, gid=181818,
> +             gecos='181818', shell='/bin/bash'))
> +
> +
>  def test_sanity_rfc2307(ldap_conn, sanity_rfc2307):
>      passwd_pattern = ent.contains_only(
>          dict(name='user1', passwd='*', uid=1001, gid=2001, gecos='1001', dir='/home/user1', shell='/bin/bash'),
> -- 
> 2.4.3
> 

> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel




More information about the sssd-devel mailing list