Hi List,

Please find the following patch with review comments.

Thanks,
Abhijeet Kasurde

On 09/03/2015 06:10 PM, Tomas Babej wrote:

On 09/03/2015 08:16 AM, Abhijeet Kasurde wrote:
Ping

On 08/27/2015 10:37 AM, Abhijeet Kasurde wrote:
Hi All,

This patch fixes bug - https://fedorahosted.org/freeipa/ticket/5264

Thanks,
Abhijeet Kasurde
ACK, the patch needs a minor rebase on master due to python3 refactoring.

From 41536f492fb7302d4c2e68b195cdffd94fc97cc7 Mon Sep 17 00:00:00 2001
From: Abhijeet Kasurde <akasu...@redhat.com>
Date: Thu, 10 Sep 2015 11:46:01 +0530
Subject: [PATCH] Updated number of legacy permission in ipatests

Since IPA 4.2 has an additional permission
"Request Certificate ignoring CA ACLs", the number of legacy
permission in testcase is updated from 8 to 9.

https://fedorahosted.org/freeipa/ticket/5264

Signed off-by: Abhijeet Kasurde <akasu...@redhat.com>
---
 ipatests/test_xmlrpc/test_permission_plugin.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ipatests/test_xmlrpc/test_permission_plugin.py b/ipatests/test_xmlrpc/test_permission_plugin.py
index e791b598149fb9b5291687b101428107415dfc85..88401190a01d8d8420f135ccd8e204172d2ecca2 100644
--- a/ipatests/test_xmlrpc/test_permission_plugin.py
+++ b/ipatests/test_xmlrpc/test_permission_plugin.py
@@ -2849,7 +2849,7 @@ def check_legacy_results(results):
     legacy_permissions = [p for p in results
                           if not p.get('ipapermissiontype')]
     print(legacy_permissions)
-    assert len(legacy_permissions) == 8, len(legacy_permissions)
+    assert len(legacy_permissions) == 9, len(legacy_permissions)
     return True
 
 class test_permission_legacy(Declarative):
-- 
2.4.3

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to