# HG changeset patch
# User Mads Kiilerich <mads@kiilerich.com>
# Date 1670690586 -3600
# Node ID 5f8f52a2f4b40477b7b3be7a11844825114bfe72
# Parent  f2dc57c123cf448cca9dc6cfb9cc1054ff491035
tests: introduce test_api_update_repo_group covering update_repo_group api

This shows that some of the API is broken. Failing test cases are disabled and
will be enabled when the problems are fixed.

diff -r f2dc57c123cf -r 5f8f52a2f4b4 kallithea/tests/api/api_base.py
--- a/kallithea/tests/api/api_base.py	Mon Dec 12 00:25:22 2022 +0100
+++ b/kallithea/tests/api/api_base.py	Sat Dec 10 17:43:06 2022 +0100
@@ -1850,6 +1850,47 @@
         )
         self._compare_error(id_, expected, given=response.body)
 
+    @base.parametrize('changing_attr,updates', [
+        #('owner', {'owner': base.TEST_USER_REGULAR_LOGIN}),  # currently broken
+        ('description', {'description': 'new description'}),
+        ('group_name', {'group_name': 'new_repo_name'}),
+        #('parent', {'parent': 'test_group_for_update'}),  # currently broken
+    ])
+    def test_api_update_repo_group(self, changing_attr, updates):
+        group_name = 'lololo'
+        repo_group = fixture.create_repo_group(group_name)
+
+        new_group_name = group_name
+        if changing_attr == 'group_name':
+            assert repo_group.parent_group_id is None  # lazy assumption for this test
+            new_group_name = updates['group_name']
+        if changing_attr == 'parent':
+            new_group_name = '/'.join([updates['parent'], group_name.rsplit('/', 1)[-1]])
+
+        expected = {
+            'msg': 'updated repository group ID:%s %s' % (repo_group.group_id, new_group_name),
+            'repo_group': repo_group.get_api_data()
+        }
+        expected['repo_group'].update(updates)
+        if 'description' in updates:
+            expected['repo_group']['group_description'] = expected['repo_group'].pop('description')
+
+        if changing_attr == 'parent':
+            new_parent = fixture.create_repo_group(updates['parent'])
+            expected['repo_group']['parent_group'] = expected['repo_group'].pop('parent')
+            expected['repo_group']['group_name'] = new_group_name
+
+        id_, params = _build_data(self.apikey, 'update_repo_group',
+                                  repogroupid=group_name, **updates)
+        response = api_call(self, params)
+
+        try:
+            self._compare_ok(id_, expected, given=response.body)
+        finally:
+            if changing_attr == 'parent':
+                fixture.destroy_repo_group(new_parent.group_id)
+            fixture.destroy_repo_group(new_group_name)
+
     @base.parametrize('name,perm,apply_to_children', [
         ('none', 'group.none', 'none'),
         ('read', 'group.read', 'none'),