[#11094] fix(client-python): address review feedback for Group auth

- Remove test_builder_no_audit_raises to match relaxed GroupDTO builder
- Add unit test for GroupAddRequest
- Add serviceAdmins config in group integration test

Author: sunyuhan1998

clients/client-python/tests/integration/test_group_management.py

@@ -38,12 +38,12 @@ class TestGroupManagement(IntegrationTestEnv):
     def setUpClass(cls):
         cls._get_gravitino_home()
         conf_path = os.path.join(cls.gravitino_home, "conf", "gravitino.conf")
-        cls._reset_conf(
-            {"gravitino.authorization.enable": "true"}, conf_path
-        )
-        cls._append_conf(
-            {"gravitino.authorization.enable": "true"}, conf_path
-        )
+        auth_confs = {
+            "gravitino.authorization.enable": "true",
+            "gravitino.authorization.serviceAdmins": "anonymous",
+        }
+        cls._reset_conf(auth_confs, conf_path)
+        cls._append_conf(auth_confs, conf_path)
         if (
             os.environ.get("START_EXTERNAL_GRAVITINO") is not None
             and os.environ.get("START_EXTERNAL_GRAVITINO").lower() == "true"
@@ -59,7 +59,11 @@ def setUpClass(cls):
     def tearDownClass(cls):
         conf_path = os.path.join(cls.gravitino_home, "conf", "gravitino.conf")
         cls._reset_conf(
-            {"gravitino.authorization.enable": "false"}, conf_path
+            {
+                "gravitino.authorization.enable": "false",
+                "gravitino.authorization.serviceAdmins": "",
+            },
+            conf_path,
         )
         if (
             os.environ.get("START_EXTERNAL_GRAVITINO") is not None

clients/client-python/tests/unittests/dto/requests/test_group_add_request.py

@@ -0,0 +1,38 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+from __future__ import annotations
+
+import json as _json
+import unittest
+
+from gravitino.dto.requests.group_add_request import GroupAddRequest
+
+
+class TestGroupAddRequest(unittest.TestCase):
+    def test_group_add_request_serde(self) -> None:
+        group_add_request = GroupAddRequest(_name="group1")
+        ser_json = _json.dumps(group_add_request.to_dict())
+        deser_dict = _json.loads(ser_json)
+
+        self.assertEqual("group1", deser_dict["name"])
+
+    def test_group_add_request_validate(self) -> None:
+        group_add_request = GroupAddRequest(_name="group1")
+        group_add_request.validate()
+
+        with self.assertRaises(ValueError):
+            GroupAddRequest(_name="").validate()

clients/client-python/tests/unittests/dto/test_group_dto.py

@@ -74,10 +74,6 @@ def test_builder_empty_name_raises(self):
                 AuditDTO(_creator="test", _create_time="2024-01-01T00:00:00Z")
             ).build()
 
-    def test_builder_no_audit_raises(self):
-        with self.assertRaises(ValueError):
-            GroupDTO.builder().with_name("test_group").build()
-
     def test_equality_and_hash(self):
         audit = AuditDTO(_creator="test", _create_time="2024-01-01T00:00:00Z")