Skip to content

Commit e44798e

Browse files
Merge pull request #627 from element-hq/gaelg/fix-flaking-tests
Fix flaking tests
2 parents 66bc76e + 2d119d7 commit e44798e

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

newsfragments/627.internal.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Fix flaking tests when checking upgrades.

tests/integration/test_networking.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ async def test_services_have_matching_labels(
4040
label_selectors = {label: value for label, value in service.spec.selector.items()}
4141

4242
async for pod in kube_client.list(Pod, namespace=generated_data.ess_namespace, labels=label_selectors):
43-
if pod.status and pod.status.phase not in ("Terminating", "Succeeded"):
43+
if pod.status and pod.status.phase in ("Terminating", "Succeeded"):
4444
continue # Skip terminating pods
4545
assert service.metadata, f"Encountered a service without metadata : {service}"
4646
assert pod.metadata, f"Encountered a pod without metadata : {pod}"

0 commit comments

Comments
 (0)