On 06/07/2012 09:48 AM, Martin Kosek wrote:
On Wed, 2012-06-06 at 12:50 +0300, Alexander Bokovoy wrote:
On Mon, 04 Jun 2012, Martin Kosek wrote:
2) Unit tests need to be updated, currently there is about a dozen test
case errors, e.g. extra ipakrbprincipalalias attribute in services or
new ipakrbprincipal objectclass for hosts
I did run unit tests. All passed except two which are not related to
trusts code and were failing before as well. All other issues found
during previous rounds of the review are fixed and squashed in patches
in my fedorapeople's tree.

======================================================================
FAIL: test_automember[39]: host_add: Create u'web5.ipa.local'
----------------------------------------------------------------------
Traceback (most recent call last):
    File "/usr/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
      self.test(*self.arg)
    File 
"/root/rpmbuild/BUILD/freeipa-2.99.0GIT0b74e24/tests/test_xmlrpc/xmlrpc_test.py", 
line 249, in<lambda>
      func = lambda: self.check(nice, **test)
    File 
"/root/rpmbuild/BUILD/freeipa-2.99.0GIT0b74e24/tests/test_xmlrpc/xmlrpc_test.py",
 line 264, in check
      self.check_output(nice, cmd, args, options, expected, extra_check)
    File 
"/root/rpmbuild/BUILD/freeipa-2.99.0GIT0b74e24/tests/test_xmlrpc/xmlrpc_test.py",
 line 290, in check_output
      assert_deepequal(expected, got, nice)
    File "/root/rpmbuild/BUILD/freeipa-2.99.0GIT0b74e24/tests/util.py", line 
331, in assert_deepequal
      assert_deepequal(e_sub, g_sub, doc, stack + (key,))
    File "/root/rpmbuild/BUILD/freeipa-2.99.0GIT0b74e24/tests/util.py", line 
325, in assert_deepequal
      doc, sorted(missing), sorted(extra), expected, got, stack
AssertionError: assert_deepequal: dict keys mismatch.
    test_automember[39]: host_add: Create u'web5.ipa.local'
    missing keys = []
    extra keys = ['memberof_hostgroup', 'memberofindirect_netgroup']
    expected = {'dn': 
u'fqdn=web5.ipa.local,cn=computers,cn=accounts,dc=ipa,dc=local',
'has_keytab': False, 'description': [u'Test host 3'], 'objectclass':
[u'ipasshhost', u'ipaSshGroupOfPubKeys', u'ieee802device', u'ipaobject',
u'nshost', u'ipahost', u'pkiuser', u'ipaservice', u'krbprincipalaux',
u'krbprincipal', u'top'], 'l': [u'Undisclosed location 1'], 'fqdn':
[u'web5.ipa.local'], 'has_password': False, 'ipauniqueid':
[Fuzzy('^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$',
<type 'unicode'>, None)], 'krbprincipalname':
[u'host/web5.ipa.local@IPA.LOCAL'], 'managedby_host':
[u'web5.ipa.local']}
    got = {'dn': 
u'fqdn=web5.ipa.local,cn=computers,cn=accounts,dc=ipa,dc=local',
'has_keytab': False, 'description': (u'Test host 3',), 'objectclass':
(u'ipaobject', u'nshost', u'ipahost', u'pkiuser', u'ipaservice',
u'krbprincipalaux', u'krbprincipal', u'ieee802device', u'ipasshhost',
u'top', u'ipaSshGroupOfPubKeys'), 'l': (u'Undisclosed location 1',),
'fqdn': (u'web5.ipa.local',), 'memberof_hostgroup': (u'hostgroup2',),
'has_password': False, 'ipauniqueid':
(u'd08fbcb4-afb9-11e1-b06b-5254007ad848',), 'krbprincipalname':
(u'host/web5.ipa.local@IPA.LOCAL',), 'managedby_host':
(u'web5.ipa.local',), 'memberofindirect_netgroup': (u'hostgroup2',)}
    path = ('result',)

======================================================================
FAIL: test_group[21]: group_find: Search for all groups
----------------------------------------------------------------------
Traceback (most recent call last):
    File "/usr/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
      self.test(*self.arg)
    File 
"/root/rpmbuild/BUILD/freeipa-2.99.0GIT0b74e24/tests/test_xmlrpc/xmlrpc_test.py", 
line 249, in<lambda>
      func = lambda: self.check(nice, **test)
    File 
"/root/rpmbuild/BUILD/freeipa-2.99.0GIT0b74e24/tests/test_xmlrpc/xmlrpc_test.py",
 line 264, in check
      self.check_output(nice, cmd, args, options, expected, extra_check)
    File 
"/root/rpmbuild/BUILD/freeipa-2.99.0GIT0b74e24/tests/test_xmlrpc/xmlrpc_test.py",
 line 290, in check_output
      assert_deepequal(expected, got, nice)
    File "/root/rpmbuild/BUILD/freeipa-2.99.0GIT0b74e24/tests/util.py", line 
331, in assert_deepequal
      assert_deepequal(e_sub, g_sub, doc, stack + (key,))
    File "/root/rpmbuild/BUILD/freeipa-2.99.0GIT0b74e24/tests/util.py", line 
339, in assert_deepequal
      VALUE % (doc, expected, got, stack)
AssertionError: assert_deepequal: expected != got.
    test_group[21]: group_find: Search for all groups
    expected = 5
    got = 6
    path = ('count',)

----------------------------------------------------------------------
Ran 1407 tests in 358.194s

FAILED (errors=2, failures=2)
======================================================================
FAILED under '/usr/bin/python2.7'

** FAIL **


It may have been an issue on my side. I will open a ticket if I hit a
unit test error again.

I did a next round of review for your patches, I did not find any
show-stopper why not to push your patches. Lets get them grilled also by
other team members :-)I just logged one issue I found with
ipa-adtrust-install:
https://fedorahosted.org/freeipa/ticket/2815

ACK. Pushed all 13 patches to master.

Martin


I tried building, and found that samba4-devel needs to be in BuildRequires. Without it, make-lint complains. Attaching the one-liner fix.

--
PetrĀ³
From 96ff40d7ce6b8c334074155d61b26516149b9770 Mon Sep 17 00:00:00 2001
From: Petr Viktorin <pvikt...@redhat.com>
Date: Thu, 7 Jun 2012 05:24:35 -0400
Subject: [PATCH] Add samba4-python to BuildRequires

---
 freeipa.spec.in |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/freeipa.spec.in b/freeipa.spec.in
index e15c79d5d7919418a86fdcdacc0e54c464df59c7..987ba9082f2f817400f388227f01adcb6801f2f0 100644
--- a/freeipa.spec.in
+++ b/freeipa.spec.in
@@ -31,7 +31,8 @@ BuildRequires:  policycoreutils >= %{POLICYCOREUTILSVER}
 %if 0%{?fedora} >= 16
 BuildRequires:  systemd-units
 %endif
-BuildRequires: samba4-devel
+BuildRequires:  samba4-devel
+BuildRequires:  samba4-python
 %endif
 BuildRequires:  nspr-devel
 BuildRequires:  nss-devel
-- 
1.7.10.2

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to