Attention is currently required from: osmith.
pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39765?usp=email )
Change subject: stp: STP_Tests_IPA.TC_tmt_loadshare_sls: Fix test expectancies
......................................................................
Patch Set 1:
(1 comment)
File stp/STP_Tests_IPA.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39765/comment/afb5093d_b6e9… :
PS1, Line 313: if (not ((num_rx_1 == iter_per_asp and num_rx_2 == 0) or (num_rx_2 == iter_per_asp and num_rx_1 == 0))) {
> Could be rewritten as the following, not sure if more readable: […]
I initially did that but tbh it didn't look proper in my mind so I ended up writing the way it is now.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39765?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I68856e11a92b707eb11ab4813390e6faf3c54b69
Gerrit-Change-Number: 39765
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 11 Mar 2025 14:55:37 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: osmith <osmith(a)sysmocom.de>
Attention is currently required from: pespin.
osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39769?usp=email )
Change subject: SCCP_Templates: Expect either proto class0 or class1 upon rx SCCP
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39769?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ia1e05b24f1b56404d951eece0e9314d784d375db
Gerrit-Change-Number: 39769
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 11 Mar 2025 13:42:17 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: pespin.
osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39765?usp=email )
Change subject: stp: STP_Tests_IPA.TC_tmt_loadshare_sls: Fix test expectancies
......................................................................
Patch Set 1: Code-Review+1
(2 comments)
File stp/STP_Tests_IPA.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39765/comment/e08b0c9e_6d98… :
PS1, Line 309: where
were
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39765/comment/6fbce1b6_bbca… :
PS1, Line 313: if (not ((num_rx_1 == iter_per_asp and num_rx_2 == 0) or (num_rx_2 == iter_per_asp and num_rx_1 == 0))) {
Could be rewritten as the following, not sure if more readable:
```suggestion
if (num_rx1 != iter_per_asp or num_rx2 != 0) and (num_rx2 != iter_per_asp or num_rx1 != 0) {
```
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39765?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I68856e11a92b707eb11ab4813390e6faf3c54b69
Gerrit-Change-Number: 39765
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 11 Mar 2025 13:40:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39768?usp=email )
Change subject: sccp: Fix test rx SLS expectancies
......................................................................
sccp: Fix test rx SLS expectancies
The SLS is the same for all messages in conn being sent in one direction, but
doesn't need to be the same value on both directions.
Since the SLS value on the other direction is not selected by the test
itself, we cannot expect a given specifi value.
Update the test expectancies.
This started to fail since recently libosmo-sigtran started properly
setting SLS values, eg libosmo-sigtran.git
7781eb275da41a9b6b1ea5d8b0e802e87a8e9d53 and
0061e8d0bcba3b0ed5ea255588619627d0975380.
Change-Id: Ic288b58629361de9cfadb568a27b51b7e02c9e99
---
M sccp/SCCP_Tests_RAW.ttcn
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/68/39768/1
diff --git a/sccp/SCCP_Tests_RAW.ttcn b/sccp/SCCP_Tests_RAW.ttcn
index f7c5357..f0d716a 100644
--- a/sccp/SCCP_Tests_RAW.ttcn
+++ b/sccp/SCCP_Tests_RAW.ttcn
@@ -131,7 +131,7 @@
sio := g_param.sio,
opc := g_param.dpc,
dpc := g_param.opc,
- sls := g_param.sls,
+ sls := ?,
data := sccp
};
return exp;
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39768?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ic288b58629361de9cfadb568a27b51b7e02c9e99
Gerrit-Change-Number: 39768
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>