NotificationsYou must be signed in to change notification settings
Fork675
Star2.4k

Commitd236267

authored

Merge pull request#1375 from JohnVillalovos/jlvillal/update_uses_post

chore: remove usage of getattr()

2 parentsd1697d4 +2afd18a commitd236267Copy full SHA for d236267

File tree

2 files changed

+27

-2

lines changed

gitlab
- mixins.py
- tests/objects
  - test_project_merge_request_approvals.py

2 files changed

+27

-2

lines changed

`‎gitlab/mixins.py‎`

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -331,6 +331,7 @@ class UpdateMixin(_RestManagerBase):`
`331`	`331`	`_parent=Optional[base.RESTObject]`
`332`	`332`	`_parent_attrs=Dict[str,Any]`
`333`	`333`	`_path:Optional[str]`
	`334`	`+_update_uses_post:bool=False`
`334`	`335`	`gitlab:gitlab.Gitlab`
`335`	`336`
`336`	`337`	`def_check_missing_update_attrs(self,data:Dict[str,Any])->None:`
`@@ -357,7 +358,7 @@ def _get_update_method(`
`357`	`358`	`Returns:`
`358`	`359`	`object: http_put (default) or http_post`
`359`	`360`	`"""`
`360`		`-ifgetattr(self,"_update_uses_post",False):`
	`361`	`+ifself._update_uses_post:`
`361`	`362`	`http_method=self.gitlab.http_post`
`362`	`363`	`else:`
`363`	`364`	`http_method=self.gitlab.http_put`

`‎gitlab/tests/objects/test_project_merge_request_approvals.py‎`

Lines changed: 25 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -2,9 +2,12 @@`
`2`	`2`	`Gitlab API: https://docs.gitlab.com/ee/api/merge_request_approvals.html`
`3`	`3`	`"""`
`4`	`4`
	`5`	`+importcopy`
	`6`	`+`
`5`	`7`	`importpytest`
`6`	`8`	`importresponses`
`7`		`-importcopy`
	`9`	`+`
	`10`	`+importgitlab`
`8`	`11`
`9`	`12`
`10`	`13`	`approval_rule_id=1`
`@@ -230,6 +233,17 @@ def resp_snippet():`
`230`	`233`	`yieldrsps`
`231`	`234`
`232`	`235`
	`236`	`+deftest_project_approval_manager_update_uses_post(project,resp_snippet):`
	`237`	`+"""Ensure the`
	`238`	`+ gitlab.v4.objects.merge_request_approvals.ProjectApprovalManager object has`
	`239`	`+ _update_uses_post set to True"""`
	`240`	`+approvals=project.approvals`
	`241`	`+assertisinstance(`
	`242`	`+approvals,gitlab.v4.objects.merge_request_approvals.ProjectApprovalManager`
	`243`	`+ )`
	`244`	`+assertapprovals._update_uses_post==True`
	`245`	`+`
	`246`	`+`
`233`	`247`	`deftest_list_merge_request_approval_rules(project,resp_snippet):`
`234`	`248`	`approval_rules=project.mergerequests.get(1).approval_rules.list()`
`235`	`249`	`assertlen(approval_rules)==1`
`@@ -239,6 +253,11 @@ def test_list_merge_request_approval_rules(project, resp_snippet):`
`239`	`253`
`240`	`254`	`deftest_update_merge_request_approvals_set_approvers(project,resp_snippet):`
`241`	`255`	`approvals=project.mergerequests.get(1).approvals`
	`256`	`+assertisinstance(`
	`257`	`+approvals,`
	`258`	`+gitlab.v4.objects.merge_request_approvals.ProjectMergeRequestApprovalManager,`
	`259`	`+ )`
	`260`	`+assertapprovals._update_uses_post==True`
`242`	`261`	`response=approvals.set_approvers(`
`243`	`262`	`updated_approval_rule_approvals_required,`
`244`	`263`	`approver_ids=updated_approval_rule_user_ids,`
`@@ -254,6 +273,11 @@ def test_update_merge_request_approvals_set_approvers(project, resp_snippet):`
`254`	`273`
`255`	`274`	`deftest_create_merge_request_approvals_set_approvers(project,resp_snippet):`
`256`	`275`	`approvals=project.mergerequests.get(1).approvals`
	`276`	`+assertisinstance(`
	`277`	`+approvals,`
	`278`	`+gitlab.v4.objects.merge_request_approvals.ProjectMergeRequestApprovalManager,`
	`279`	`+ )`
	`280`	`+assertapprovals._update_uses_post==True`
`257`	`281`	`response=approvals.set_approvers(`
`258`	`282`	`new_approval_rule_approvals_required,`
`259`	`283`	`approver_ids=new_approval_rule_user_ids,`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitd236267

File tree

2 files changed

2 files changed

`‎gitlab/mixins.py‎`

`‎gitlab/tests/objects/test_project_merge_request_approvals.py‎`

0 commit comments