c8cd81
From b6520bef2ef05dd87636d8b57e3247d451af81d8 Mon Sep 17 00:00:00 2001
c8cd81
From: Florence Blanc-Renaud <flo@redhat.com>
c8cd81
Date: Tue, 14 Dec 2021 16:33:29 +0100
c8cd81
Subject: [PATCH] ipatests: Fix expected object classes
c8cd81
c8cd81
Because the sidgen plugin is a postop plugin, it is not
c8cd81
always triggered before the result of an ADD is returned
c8cd81
and the objectclasses of the user may / may not contain
c8cd81
ipantuserattrs.
c8cd81
Fix the expected object classes.
c8cd81
c8cd81
Related: https://pagure.io/freeipa/issue/9062
c8cd81
Signed-off-by: Florence Blanc-Renaud <flo@redhat.com>
c8cd81
Reviewed-By: Rob Crittenden <rcritten@redhat.com>
c8cd81
Reviewed-By: Alexander Bokovoy <abokovoy@redhat.com>
c8cd81
Reviewed-By: Stanislav Levin <slev@altlinux.org>
c8cd81
---
c8cd81
 ipatests/test_xmlrpc/test_user_plugin.py | 20 ++++++++++++++------
c8cd81
 1 file changed, 14 insertions(+), 6 deletions(-)
c8cd81
c8cd81
diff --git a/ipatests/test_xmlrpc/test_user_plugin.py b/ipatests/test_xmlrpc/test_user_plugin.py
c8cd81
index 74d78b0c6d75590640da1357da1f9f4570307878..c156a8793c5765df96029753f4a278e961d4e895 100644
c8cd81
--- a/ipatests/test_xmlrpc/test_user_plugin.py
c8cd81
+++ b/ipatests/test_xmlrpc/test_user_plugin.py
c8cd81
@@ -125,7 +125,9 @@ def user_npg(request, group):
c8cd81
     del tracker.attrs['mepmanagedentry']
c8cd81
     tracker.attrs.update(
c8cd81
         description=[], memberof_group=[group.cn],
c8cd81
-        objectclass=objectclasses.user_base + [u'ipantuserattrs'],
c8cd81
+        objectclass=fuzzy_set_optional_oc(
c8cd81
+            objectclasses.user_base, 'ipantuserattrs'
c8cd81
+        ),
c8cd81
     )
c8cd81
     return tracker.make_fixture(request)
c8cd81
 
c8cd81
@@ -139,7 +141,9 @@ def user_npg2(request, group):
c8cd81
     del tracker.attrs['mepmanagedentry']
c8cd81
     tracker.attrs.update(
c8cd81
         gidnumber=[u'1000'], description=[], memberof_group=[group.cn],
c8cd81
-        objectclass=objectclasses.user_base + [u'ipantuserattrs'],
c8cd81
+        objectclass=fuzzy_set_optional_oc(
c8cd81
+            objectclasses.user_base, 'ipantuserattrs'
c8cd81
+        ),
c8cd81
     )
c8cd81
     return tracker.make_fixture(request)
c8cd81
 
c8cd81
@@ -151,8 +155,9 @@ def user_radius(request, xmlrpc_setup):
c8cd81
                           sn=u'radiususer1',
c8cd81
                           ipatokenradiususername=u'radiususer')
c8cd81
     tracker.track_create()
c8cd81
-    tracker.attrs.update(
c8cd81
-        objectclass=objectclasses.user + [u'ipatokenradiusproxyuser']
c8cd81
+    tracker.attrs.update(objectclass=fuzzy_set_optional_oc(
c8cd81
+        objectclasses.user + [u'ipatokenradiusproxyuser'],
c8cd81
+        'ipantuserattrs'),
c8cd81
     )
c8cd81
     return tracker.make_fixture(request)
c8cd81
 
c8cd81
@@ -647,7 +652,8 @@ class TestCreate(XMLRPC_test):
c8cd81
         testuser.attrs.update(gidnumber=[u'1000'])
c8cd81
         testuser.attrs.update(
c8cd81
             description=[],
c8cd81
-            objectclass=objectclasses.user_base + [u'ipantuserattrs']
c8cd81
+            objectclass=fuzzy_set_optional_oc(
c8cd81
+                objectclasses.user_base, 'ipantuserattrs'),
c8cd81
         )
c8cd81
         command = testuser.make_create_command()
c8cd81
         result = command()
c8cd81
@@ -865,7 +871,9 @@ class TestUserWithUPGDisabled(XMLRPC_test):
c8cd81
         testuser.attrs.update(gidnumber=[u'1000'])
c8cd81
         testuser.attrs.update(
c8cd81
             description=[],
c8cd81
-            objectclass=objectclasses.user_base + [u'ipantuserattrs'],
c8cd81
+            objectclass=fuzzy_set_optional_oc(
c8cd81
+                objectclasses.user_base, 'ipantuserattrs'
c8cd81
+            ),
c8cd81
         )
c8cd81
         command = testuser.make_create_command()
c8cd81
         result = command()
c8cd81
-- 
c8cd81
2.37.2
c8cd81