Merge pull request #1316 from JohnVillalovos/jlvillal/test_wait

nejch · web-flow · commit 5cc60d5a8ac1 · 2021-02-21T21:37:14.000+01:00
test: extend wait timeout for test_delete_user()
diff --git a/tools/functional/api/test_users.py b/tools/functional/api/test_users.py
@@ -56,7 +56,8 @@ def test_delete_user(gl, wait_for_sidekiq):
     )
 
     new_user.delete()
-    wait_for_sidekiq()
+    result = wait_for_sidekiq(timeout=60)
+    assert result == True, "sidekiq process should have terminated but did not"
 
     assert new_user.id not in [user.id for user in gl.users.list()]
 
diff --git a/tools/functional/conftest.py b/tools/functional/conftest.py
@@ -89,9 +89,15 @@ def wait_for_sidekiq(gl):
 
     def _wait(timeout=30, step=0.5):
         for _ in range(timeout):
-            if not gl.sidekiq.process_metrics()["processes"][0]["busy"]:
-                return
             time.sleep(step)
+            busy = False
+            processes = gl.sidekiq.process_metrics()["processes"]
+            for process in processes:
+                if process["busy"]:
+                    busy = True
+            if not busy:
+                return True
+        return False
 
     return _wait
 

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`	`+ assert result == True, "sidekiq process should have terminated but did not"`
`60`	`61`
`61`	`62`	`assert new_user.id not in [user.id for user in gl.users.list()]`
`62`	`63`