Skip to content

Commit 29cfacf

Browse files
authored
Try to fix the flaky automation integration test (#5656)
Signed-off-by: Ravi Nadahar <nadahar@rediffmail.com>
1 parent d4d7455 commit 29cfacf

1 file changed

Lines changed: 8 additions & 0 deletions

File tree

itests/org.openhab.core.automation.integration.tests/src/main/java/org/openhab/core/automation/integration/test/AutomationIntegrationTest.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,8 @@
1414

1515
import static org.hamcrest.CoreMatchers.*;
1616
import static org.hamcrest.MatcherAssert.assertThat;
17+
import static org.hamcrest.Matchers.greaterThanOrEqualTo;
18+
import static org.hamcrest.Matchers.hasSize;
1719
import static org.junit.jupiter.api.Assertions.assertTrue;
1820
import static org.mockito.Mockito.mock;
1921
import static org.mockito.Mockito.when;
@@ -162,9 +164,15 @@ public void removeProviderChangeListener(ProviderChangeListener<Item> listener)
162164
}
163165
};
164166

167+
int expectedItemCount = itemProvider.getAll().size();
165168
registerService(itemProvider);
166169
registerVolatileStorageService();
167170

171+
waitForAssert(() -> {
172+
ItemRegistry registry = getService(ItemRegistry.class);
173+
assertThat(registry.getAll(), hasSize(greaterThanOrEqualTo(expectedItemCount)));
174+
}, 5000, 100);
175+
168176
StorageService storageService = getService(StorageService.class);
169177
ruleRegistry = getService(RuleRegistry.class);
170178
ruleEngine = getService(RuleManager.class);

0 commit comments

Comments
 (0)