fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/31979 )


Change subject: BTS_Tests: keep pchan configuration unchanged in TC_pcu_*
......................................................................

BTS_Tests: keep pchan configuration unchanged in TC_pcu_*

Change [1] fixes support of dynamic timeslots in osmo-bts-virtual,
so we don't need to change the pchan configuration anymore.

Change-Id: I98397ac0fc17ece8f0e41b1ef1c158b47c9de026
Related: [1] osmo-bts.git I5db5b7dd6a8e84cf9a0d84f04a650c2ed8a4e368
---
M bts/BTS_Tests.ttcn
1 file changed, 20 insertions(+), 17 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks 
refs/changes/79/31979/1

diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn
index 70a2137..c36e27d 100644
--- a/bts/BTS_Tests.ttcn
+++ b/bts/BTS_Tests.ttcn
@@ -5509,7 +5509,7 @@
 testcase TC_pcu_data_req_wrong_bts() runs on test_CT {
        var octetstring data := '0000'O & f_rnd_octstring(21);

-       f_virtphy_common();
+       f_TC_pcu_common();

        f_TC_pcu_act_req(0, 0, 7, true);
        f_TC_pcu_l1ctl_est_pdch(7);
@@ -5522,7 +5522,7 @@
 testcase TC_pcu_data_req_wrong_trx() runs on test_CT {
        var octetstring data := '0000'O & f_rnd_octstring(21);

-       f_virtphy_common();
+       f_TC_pcu_common();

        f_TC_pcu_act_req(0, 0, 7, true);
        f_TC_pcu_l1ctl_est_pdch(7);
@@ -5535,7 +5535,7 @@
 testcase TC_pcu_data_req_wrong_ts() runs on test_CT {
        var octetstring data := '0000'O & f_rnd_octstring(21);

-       f_virtphy_common();
+       f_TC_pcu_common();

        f_TC_pcu_act_req(0, 0, 7, true);
        f_TC_pcu_l1ctl_est_pdch(7);
@@ -5548,7 +5548,7 @@
 testcase TC_pcu_data_req_ts_inactive() runs on test_CT {
        var octetstring data := '0000'O & f_rnd_octstring(21);

-       f_virtphy_common();
+       f_TC_pcu_common();

        /* intentionally not calling f_TC_pcu_act_req() */
        f_TC_pcu_l1ctl_est_pdch(7);
@@ -5593,18 +5593,8 @@
        }
 }

-private function f_disable_dynamic_ts() runs on test_CT
-{
-       f_init_vty_bsc();
-       /* I'm not quite sure why we need this with osmo-bts-virtual.  Somehow 
it deosn't seem to
-        * support dynamic timeslots?  But it uses the same scheduler as 
osmo-bts-trx ?!? */
-       f_vty_config2(BSCVTY, {"network", "bts 0", "trx 0", "timeslot 3"}, 
"phys_chan_config TCH/F");
-       f_vty_config2(BSCVTY, {"network", "bts 0", "trx 0", "timeslot 4"}, 
"phys_chan_config TCH/F");
+private function f_TC_pcu_common() runs on test_CT {
        f_init_with_pcuif();
-}
-
-private function f_virtphy_common() runs on test_CT {
-       f_disable_dynamic_ts();
        f_init_l1ctl();
        f_l1_tune(L1CTL);
 }
@@ -5612,7 +5602,7 @@
 testcase TC_pcu_data_req_pdtch() runs on test_CT {
        var octetstring data := '0000'O & f_rnd_octstring(21);

-       f_virtphy_common();
+       f_TC_pcu_common();

        f_TC_pcu_act_req(0, 0, 7, true);
        f_TC_pcu_l1ctl_est_pdch(7);
@@ -5624,7 +5614,7 @@
 testcase TC_pcu_data_req_ptcch() runs on test_CT {
        var octetstring data := '0000'O & f_rnd_octstring(21);

-       f_virtphy_common();
+       f_TC_pcu_common();

        f_TC_pcu_act_req(0, 0, 7, true);
        f_TC_pcu_l1ctl_est_pdch(7);

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

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I98397ac0fc17ece8f0e41b1ef1c158b47c9de026
Gerrit-Change-Number: 31979
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanits...@sysmocom.de>
Gerrit-MessageType: newchange

Reply via email to