laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
RemsimServer Tests: give the remsim server some time to update the slotmap

The check of the slotmap can race if it is requests straight away,
because the slotmap might be still in the slotmap list as long the
bankd hasn't responded yet.

Change-Id: I6784e05380d844aa4bb9cde2bbe648ff37bee91b
---
M remsim/RemsimServer_Tests.ttcn
1 file changed, 9 insertions(+), 0 deletions(-)

diff --git a/remsim/RemsimServer_Tests.ttcn b/remsim/RemsimServer_Tests.ttcn
index 5850d29..de2c478 100644
--- a/remsim/RemsimServer_Tests.ttcn
+++ b/remsim/RemsimServer_Tests.ttcn
@@ -512,6 +512,9 @@
/* delete the slotmap via REST */
res := f_rsres_delete_slotmap(sm.bank);

+ /* an immediate slotmaps check doesn't work */
+ f_sleep(1.0);
+
/* verify the slotmap is gone */
f_ensure_slotmaps({});
}
@@ -549,6 +552,9 @@
/* delete the slotmap via REST */
res := f_rsres_delete_slotmap(sm.bank);

+ /* an immediate slotmaps check doesn't work */
+ f_sleep(1.0);
+
/* verify the slotmap is gone from REST interface immediately */
f_ensure_slotmaps({});

@@ -600,6 +606,9 @@
/* delete the slotmap via REST */
res := f_rsres_delete_slotmap(sm.bank);

+ /* an immediate slotmaps check doesn't work */
+ f_sleep(1.0);
+
/* verify the slotmap is gone from REST interface immediately */
f_ensure_slotmaps({});


To view, visit change 42742. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I6784e05380d844aa4bb9cde2bbe648ff37bee91b
Gerrit-Change-Number: 42742
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis@fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>