Merge pull request #971 from jooola/ci_vars_type

max-wittig · web-flow · commit 938fc0ae1eff · 2019-12-16T10:36:47.000+01:00
feat: add more options for project/group ci variables manipulation
diff --git a/gitlab/v4/objects.py b/gitlab/v4/objects.py
@@ -1184,8 +1184,8 @@ class GroupVariableManager(CRUDMixin, RESTManager):
     _path = "/groups/%(group_id)s/variables"
     _obj_cls = GroupVariable
     _from_parent_attrs = {"group_id": "id"}
-    _create_attrs = (("key", "value"), ("protected",))
-    _update_attrs = (("key", "value"), ("protected",))
+    _create_attrs = (("key", "value"), ("protected", "variable_type"))
+    _update_attrs = (("key", "value"), ("protected", "variable_type"))
 
 
 class Group(SaveMixin, ObjectDeleteMixin, RESTObject):
@@ -3716,8 +3716,8 @@ class ProjectVariableManager(CRUDMixin, RESTManager):
     _path = "/projects/%(project_id)s/variables"
     _obj_cls = ProjectVariable
     _from_parent_attrs = {"project_id": "id"}
-    _create_attrs = (("key", "value"), tuple())
-    _update_attrs = (("key", "value"), tuple())
+    _create_attrs = (("key", "value"), ("protected", "variable_type"))
+    _update_attrs = (("key", "value"), ("protected", "variable_type"))
 
 
 class ProjectService(SaveMixin, ObjectDeleteMixin, RESTObject):
