This patch fixes test for Permission plugin - mainly permission-mod
part. Description field that the tests expected and which was
removed in ticket 792 was removed from the tests.

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

>From 40fe8a1cf9151de25c655c4c985905d2895662bf Mon Sep 17 00:00:00 2001
From: Martin Kosek <mko...@redhat.com>
Date: Tue, 1 Feb 2011 12:31:54 +0100
Subject: [PATCH] Permission rename test failing

This patch fixes test for Permission plugin - mainly permission-mod
part. Description field that the tests expected and which was
removed in ticket 792 was removed from the tests.

https://fedorahosted.org/freeipa/ticket/892
---
 tests/test_xmlrpc/test_permission_plugin.py |   14 +++++---------
 1 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/tests/test_xmlrpc/test_permission_plugin.py b/tests/test_xmlrpc/test_permission_plugin.py
index be5d2befbb809229b496d27930202637a012c2db..4ee466b6552e0d3d931a63bb8cdb06fd86361c16 100644
--- a/tests/test_xmlrpc/test_permission_plugin.py
+++ b/tests/test_xmlrpc/test_permission_plugin.py
@@ -332,8 +332,7 @@ class test_permission(Declarative):
                                                                  permission2),
             command=(
                 'permission_mod', [permission1], dict(rename=permission2,
-                                                      description=u"Renamed Desc",
-                                                      permissions=u'read',)
+                                                      permissions=u'all',)
             ),
             expected=errors.DuplicateEntry(),
         ),
@@ -348,10 +347,9 @@ class test_permission(Declarative):
                 result={
                     'dn': permission1_dn,
                     'cn': [permission1],
-                    'description': [u'New desc 1'],
                     'member_privilege': [privilege1],
                     'type': u'user',
-                    'permissions': [u'write'],
+                    'permissions': [u'read'],
                 },
             ),
         ),
@@ -362,8 +360,7 @@ class test_permission(Declarative):
                                                  permission1_renamed),
             command=(
                 'permission_mod', [permission1], dict(rename=permission1_renamed,
-                                                      description=u"Renamed Desc",
-                                                      permissions= u'read',)
+                                                      permissions= u'all',)
             ),
             expected=dict(
                 value=permission1,
@@ -371,10 +368,9 @@ class test_permission(Declarative):
                 result={
                     'dn': permission1_renamed_dn,
                     'cn': [permission1_renamed],
-                    'description': [u'Renamed Desc'],
                     'member_privilege': [privilege1],
                     'type': u'user',
-                    'permissions': [u'read'],
+                    'permissions': [u'all'],
                 },
             ),
         ),
@@ -407,7 +403,7 @@ class test_permission(Declarative):
 
         dict(
             desc='Try to update non-existent %r' % permission1,
-            command=('permission_mod', [permission1], dict(description=u'Foo')),
+            command=('permission_mod', [permission1], dict(rename=u'Foo')),
             expected=errors.NotFound(reason='no such entry'),
         ),
 
-- 
1.7.3.5

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

Reply via email to