Commite1d5f8d

committed

test: cover .get_id() paths

Signed-off-by: Tim Knight <tim.knight1@engineering.digital.dwp.gov.uk>

1 parent8273450 commite1d5f8dCopy full SHA for e1d5f8d

File tree

-6

lines changed

-6

lines changed

Lines changed: 6 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -44,9 +44,9 @@ def get_id(self) -> int:`
`44`	`44`	`"""Returns the id of the resource. This override deals with`
`45`	`45`	the fact that either an `id` or a `target_project_id` attribute
`46`	`46`	`is returned by the server depending on the endpoint called."""`
`47`		`-project_id=cast(int,super().get_id())`
`48`		`-ifproject_idisnotNone:`
`49`		`-returnproject_id`
	`47`	`+target_project_id=cast(int,super().get_id())`
	`48`	`+iftarget_project_idisnotNone:`
	`49`	`+returntarget_project_id`
`50`	`50`	`returncast(int,self.id)`
`51`	`51`
`52`	`52`
`@@ -64,9 +64,9 @@ def get_id(self) -> int:`
`64`	`64`	`"""Returns the id of the resource. This override deals with`
`65`	`65`	the fact that either an `id` or a `target_group_id` attribute
`66`	`66`	`is returned by the server depending on the endpoint called."""`
`67`		`-group_id=cast(int,super().get_id())`
`68`		`-ifgroup_idisnotNone:`
`69`		`-returngroup_id`
	`67`	`+target_group_id=cast(int,super().get_id())`
	`68`	`+iftarget_group_idisnotNone:`
	`69`	`+returntarget_group_id`
`70`	`70`	`returncast(int,self.id)`
`71`	`71`
`72`	`72`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -174,6 +174,7 @@ def test_get_projects_allowlist(job_token_scope, resp_get_allowlist):`
`174`	`174`
`175`	`175`	`allowlist_content=allowlist.list()`
`176`	`176`	`assertisinstance(allowlist_content,list)`
	`177`	`+assertallowlist_content[0].get_id()==4`
`177`	`178`
`178`	`179`
`179`	`180`	`deftest_add_project_to_allowlist(job_token_scope,resp_add_to_allowlist):`
`@@ -190,6 +191,7 @@ def test_get_groups_allowlist(job_token_scope, resp_get_groups_allowlist):`
`190`	`191`
`191`	`192`	`allowlist_content=allowlist.list()`
`192`	`193`	`assertisinstance(allowlist_content,list)`
	`194`	`+assertallowlist_content[0].get_id()==4`
`193`	`195`
`194`	`196`
`195`	`197`	`deftest_add_group_to_allowlist(job_token_scope,resp_add_to_groups_allowlist):`

Comments

(0)