neels has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/22264 )

Change subject: fixate test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty
......................................................................

fixate test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty

The test shows a cascade of failures. When we fix the first failure, it
will change the sequence of events that follow after that. But it will
still be interesting to evaluate all the situations currently shown.

Hence fixate each stage's initial situation, by duplicating the
expect-ts-use with an identical set-ts-use. Then, when each individual
scenario gets fixed, subsequent scenarios still remain unchanged.

Change-Id: Ifeaec39ecb64b476ff1438cf987ba0403489c43b
---
M tests/handover/test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty
1 file changed, 24 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved
  pespin: Looks good to me, but someone else must approve



diff --git a/tests/handover/test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty 
b/tests/handover/test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty
index 1273d75..bc2c884 100644
--- a/tests/handover/test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty
+++ b/tests/handover/test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty
@@ -25,6 +25,12 @@

 expect-ts-use trx 0 0 states * - - - pdch pdch pdch *
 expect-ts-use trx 1 0 states * TCH/F TCH/F TCH/F TCH/HH TCH/F pdch *
+
+
+###
+
+set-ts-use trx 0 0 states * - - - pdch pdch pdch *
+set-ts-use trx 1 0 states * TCH/F TCH/F TCH/F TCH/HH TCH/F pdch *
 meas-rep lchan * * * * rxlev 40 rxqual 0 ta 0 neighbors 30

 congestion-check
@@ -34,12 +40,24 @@
 expect-ts-use trx 0 0 states * - - - pdch pdch pdch *
 expect-ts-use trx 1 0 states * TCH/F TCH/F TCH/F TCH/H- TCH/F TCH/F *

+
+###
+
+set-ts-use trx 0 0 states * - - - pdch pdch pdch *
+set-ts-use trx 1 0 states * TCH/F TCH/F TCH/F TCH/H- TCH/F TCH/F *
+
 congestion-check
 # more FAIL: TCH/H moves to worse bts 0 due to congestion
 expect-ho from lchan 1 0 4 0 to lchan 0 0 4 0
 expect-ts-use trx 0 0 states * - - - TCH/H- pdch pdch *
 expect-ts-use trx 1 0 states * TCH/F TCH/F TCH/F pdch TCH/F TCH/F *

+
+###
+
+set-ts-use trx 0 0 states * - - - TCH/H- pdch pdch *
+set-ts-use trx 1 0 states * TCH/F TCH/F TCH/F pdch TCH/F TCH/F *
+
 congestion-check
 expect-no-chan

@@ -53,6 +71,12 @@
 expect-ts-use trx 0 0 states * - - - pdch pdch pdch *
 expect-ts-use trx 1 0 states * TCH/F TCH/F TCH/F TCH/F TCH/F TCH/F *

+
+###
+
+set-ts-use trx 0 0 states * - - - pdch pdch pdch *
+set-ts-use trx 1 0 states * TCH/F TCH/F TCH/F TCH/F TCH/F TCH/F *
+
 congestion-check
 expect-no-chan
 # Stable situation now only because TCH/F has min-free-slots set to 0

--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/22264
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ifeaec39ecb64b476ff1438cf987ba0403489c43b
Gerrit-Change-Number: 22264
Gerrit-PatchSet: 2
Gerrit-Owner: neels <nhofm...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: neels <nhofm...@sysmocom.de>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-MessageType: merged

Reply via email to