From 767e9c7c8d5eee5efc50fd1a27926d7d46c7f64b Mon Sep 17 00:00:00 2001 From: phvalguima Date: Wed, 17 Jan 2024 08:32:59 +0100 Subject: [PATCH] Review client-integration Break the wait_until in test_scaling_secrets to an idle_period < update-status-interval + time.sleep to ensure we see the result of at least one update-status. --- .../relations/opensearch_provider/test_opensearch_provider.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/integration/relations/opensearch_provider/test_opensearch_provider.py b/tests/integration/relations/opensearch_provider/test_opensearch_provider.py index fae39ea0f..b8a561651 100644 --- a/tests/integration/relations/opensearch_provider/test_opensearch_provider.py +++ b/tests/integration/relations/opensearch_provider/test_opensearch_provider.py @@ -308,8 +308,10 @@ async def _is_number_of_endpoints_valid(client_app: str, rel: str) -> bool: apps_statuses=["active"], units_statuses=["active"], wait_for_exact_units={OPENSEARCH_APP_NAME: len(opensearch_unit_ids)}, - idle_period=70, + idle_period=50, # slightly less than update-status-interval period ) + # Now, we want to sleep until an update-status happens + time.sleep(30) assert await _is_number_of_endpoints_valid( CLIENT_APP_NAME, FIRST_RELATION_NAME ), await rel_endpoints(CLIENT_APP_NAME, FIRST_RELATION_NAME)