Commit5cc60d5

authored

Merge pull request#1316 from JohnVillalovos/jlvillal/test_wait

test: extend wait timeout for test_delete_user()

2 parents2b29776 +19fde8e commit5cc60d5Copy full SHA for 5cc60d5

File tree

+10

-3

lines changed

+10

-3

lines changed

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,8 @@ def test_delete_user(gl, wait_for_sidekiq):`
`56`	`56`	`)`
`57`	`57`
`58`	`58`	`new_user.delete()`
`59`		`-wait_for_sidekiq()`
	`59`	`+result=wait_for_sidekiq(timeout=60)`
	`60`	`+assertresult==True,"sidekiq process should have terminated but did not"`
`60`	`61`
`61`	`62`	`assertnew_user.idnotin [user.idforuseringl.users.list()]`
`62`	`63`

Lines changed: 8 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -89,9 +89,15 @@ def wait_for_sidekiq(gl):`
`89`	`89`
`90`	`90`	`def_wait(timeout=30,step=0.5):`
`91`	`91`	`for_inrange(timeout):`
`92`		`-ifnotgl.sidekiq.process_metrics()["processes"][0]["busy"]:`
`93`		`-return`
`94`	`92`	`time.sleep(step)`
	`93`	`+busy=False`
	`94`	`+processes=gl.sidekiq.process_metrics()["processes"]`
	`95`	`+forprocessinprocesses:`
	`96`	`+ifprocess["busy"]:`
	`97`	`+busy=True`
	`98`	`+ifnotbusy:`
	`99`	`+returnTrue`
	`100`	`+returnFalse`
`95`	`101`
`96`	`102`	`return_wait`
`97`	`103`

Comments

(0)