dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42954?usp=email )
Change subject: eIM: echo eimTransactionId from euiccDataReq in provideEimPackageResult
......................................................................
eIM: echo eimTransactionId from euiccDataReq in provideEimPackageResult
The udted eIM now includes an eimTransactionId in its ipaEuiccDataRequest
message, which means that the IPAd (testsuite) has to echo that exact
same eimTransactionId in the IpaEuiccData of the IpaEuiccDataResponse
Related: SYS#8100
Change-Id: I286362e6499eb9440e006f08b7c142c6dfd98843
---
M eim/eIM_Tests.ttcn
1 file changed, 6 insertions(+), 3 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/54/42954/1
diff --git a/eim/eIM_Tests.ttcn b/eim/eIM_Tests.ttcn
index 1e24b9e..8f48528 100644
--- a/eim/eIM_Tests.ttcn
+++ b/eim/eIM_Tests.ttcn
@@ -755,6 +755,8 @@
private function f_TC_proc_euicc_data_req(charstring id) runs on eIM_ConnHdlr {
var charstring resource_id;
var HTTP_Adapter_Params http_adapter_pars;
+ var template EsipaMessageFromEimToIpa esipa_res;
+ var octetstring eimTransactionId;
f_http_register();
@@ -771,11 +773,12 @@
/* Check the eIM for new eIM packages, we expect to get a getEimPackageResponse that contains an
* ipaEuiccDataRequest */
- f_esipa_transceive(valueof(ts_getEimPackageRequest(eID)), tr_getEimPackageResponse_euiccDataReq);
+ esipa_res := f_esipa_transceive(valueof(ts_getEimPackageRequest(eID)), tr_getEimPackageResponse_euiccDataReq);
+ eimTransactionId := valueof(esipa_res.getEimPackageResponse.ipaEuiccDataRequest.eimTransactionId);
/* Provide a plausible ipaEuiccDataResponse to the eIM */
- f_esipa_transceive(valueof(ts_provideEimPackageResult_euiccDataResp),
- tr_provideEimPackageResultResponse_eimAck);
+ f_esipa_transceive(valueof(ts_provideEimPackageResult_euiccDataResp(eID, eimTransactionId)),
+ tr_provideEimPackageResultResponse_empty);
f_rest_lookup_resource(resource_id, "edr", tr_JSON_REST_success);
f_rest_delete_resource(resource_id, "edr");
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42954?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: I286362e6499eb9440e006f08b7c142c6dfd98843
Gerrit-Change-Number: 42954
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42955?usp=email )
Change subject: eIM: add testcase to test an unsuccessful eUICC data request
......................................................................
eIM: add testcase to test an unsuccessful eUICC data request
When we test the ipaEuiccDataRequest/IpaEuiccDataResponse, we currently
only test the success case. However, we should also test the failure
case, bceause an IpaEuiccDataResponseError can also contain an
eimTransactionId, which has to be equal to the eimTransactionId we have
received with the ipaEuiccDataRequest
Related: SYS#8100
Change-Id: Ied212eb636cf46df5fc9f86ddee15c242c50d45a
---
M eim/eIM_Tests.ttcn
1 file changed, 44 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/55/42955/1
diff --git a/eim/eIM_Tests.ttcn b/eim/eIM_Tests.ttcn
index 8f48528..6e7c016 100644
--- a/eim/eIM_Tests.ttcn
+++ b/eim/eIM_Tests.ttcn
@@ -793,6 +793,49 @@
setverdict(pass);
}
+/* A testcase to try out an unsuccessful eUICC data request,
+ * See also: GSMA SGP.32, section 2.11.1.2: IpaEuiccDataRequest */
+private function f_TC_proc_euicc_data_req_err(charstring id) runs on eIM_ConnHdlr {
+ var charstring resource_id;
+ var HTTP_Adapter_Params http_adapter_pars;
+ var template EsipaMessageFromEimToIpa esipa_res;
+ var template octetstring eimTransactionId;
+
+ f_http_register();
+
+ f_init_pipe();
+ http_adapter_pars := {
+ http_host := mp_esipa_ip,
+ http_port := mp_esipa_port,
+ use_ssl := not mp_esipa_disable_ssl
+ };
+ f_http_init(http_adapter_pars);
+
+ /* Create a new download at the eIM */
+ resource_id := f_rest_create_order("edr", ts_JSON_REST_edr_order(eID, '80BF20BF228384A5A688A9BF2B'O));
+
+ /* Check the eIM for new eIM packages, we expect to get a getEimPackageResponse that contains an
+ * ipaEuiccDataRequest */
+ esipa_res := f_esipa_transceive(valueof(ts_getEimPackageRequest(eID)), tr_getEimPackageResponse_euiccDataReq);
+ eimTransactionId := valueof(esipa_res.getEimPackageResponse.ipaEuiccDataRequest.eimTransactionId);
+
+ /* Provide a plausible ipaEuiccDataResponse to the eIM */
+ f_esipa_transceive(valueof(ts_provideEimPackageResult_euiccDataResp_err(eID, eimTransactionId)),
+ tr_provideEimPackageResultResponse_empty);
+
+ f_rest_lookup_resource(resource_id, "edr", tr_JSON_REST_success);
+ f_rest_delete_resource(resource_id, "edr");
+}
+testcase TC_proc_euicc_data_req_err() runs on MTC_CT {
+ var charstring id := testcasename();
+ var eIM_ConnHdlrPars pars := f_init_pars();
+ var eIM_ConnHdlr vc_conn;
+ f_init(id);
+ vc_conn := f_start_handler(refers(f_TC_proc_euicc_data_req_err), pars);
+ vc_conn.done;
+ setverdict(pass);
+}
+
control {
execute ( TC_proc_indirect_prfle_dwnld() );
execute ( TC_proc_euicc_pkg_dwnld_exec_enable_psmo() );
@@ -807,6 +850,7 @@
execute ( TC_proc_euicc_pkg_dwnld_exec_updateEim_eco() );
execute ( TC_proc_euicc_pkg_dwnld_exec_listEim_eco() );
execute ( TC_proc_euicc_data_req() );
+ execute ( TC_proc_euicc_data_req_err() );
}
}
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42955?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: Ied212eb636cf46df5fc9f86ddee15c242c50d45a
Gerrit-Change-Number: 42955
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42956?usp=email )
Change subject: eIM: use eimTransactionId during profile download
......................................................................
eIM: use eimTransactionId during profile download
When the eIM initiates a profile download via the
profileDownloadTriggerRequest, it includes an eimTransactionId to identify
sent by the IPAd (testsuite) in the initiateAuthenticationRequestEsipa
response. The eimTransactionId (even though it is optional) is essential
for the eIM to recognize the response from the eIM correctly.
(SGP.32 V.1.0.1 lacked the eimTransactionId, which meant that the eIM had
to rely on other identifiers such as the TCP connection itself. The problem
is now fixed with SGP.32 V.1.2)
Related: SYS#8100
Change-Id: I045d93b9f40ddfb3dc6dfbd9602e736bbc6b4abc
---
M eim/eIM_Tests.ttcn
1 file changed, 9 insertions(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/56/42956/1
diff --git a/eim/eIM_Tests.ttcn b/eim/eIM_Tests.ttcn
index 6e7c016..8508d39 100644
--- a/eim/eIM_Tests.ttcn
+++ b/eim/eIM_Tests.ttcn
@@ -449,11 +449,12 @@
/* ********************************************* */
/* Common Mutual Authentication Procedure, see also: GSMA SGP.22, section 3.0.1 */
-private function f_proc_cmn_mtl_auth() runs on eIM_ConnHdlr {
+private function f_proc_cmn_mtl_auth(octetstring eimTransactionId) runs on eIM_ConnHdlr {
var charstring smdpAddress := mp_es9p_ip & ":" & int2str(mp_es9p_port);
/* InitiateAuthentication cycle */
- f_esipa_send(valueof(ts_initiateAuthenticationRequestEsipa(smdpAddress := smdpAddress)));
+ f_esipa_send(valueof(ts_initiateAuthenticationRequestEsipa(smdpAddress := smdpAddress,
+ eimTransactionId := eimTransactionId)));
f_es9p_transceive(valueof(ts_initiateAuthenticationResponse), tr_initiateAuthenticationRequest);
f_esipa_receive(tr_initiateAuthenticationResponseEsipa);
@@ -470,6 +471,8 @@
var HTTP_Adapter_Params http_adapter_pars;
var charstring smdpp_add := mp_es9p_ip & ":" & int2str(mp_es9p_port)
var charstring activationCode := "1$" & smdpp_add & "$MyMatchingId"
+ var template EsipaMessageFromEimToIpa esipa_res;
+ var octetstring eimTransactionId;
f_http_register();
@@ -486,10 +489,12 @@
/* Check the eIM for new eIM packages, we expect to get a profileDownloadTriggerRequest that contains the
* activationCode that we have created the download with (see above) */
- f_esipa_transceive(valueof(ts_getEimPackageRequest(eID)), tr_getEimPackageResponse_dnlTrigReq(activationCode));
+ esipa_res := f_esipa_transceive(valueof(ts_getEimPackageRequest(eID)),
+ tr_getEimPackageResponse_dnlTrigReq(activationCode));
+ eimTransactionId := valueof(esipa_res.getEimPackageResponse.profileDownloadTriggerRequest.eimTransactionId);
/* Perform common mutial authentication procedure */
- f_proc_cmn_mtl_auth();
+ f_proc_cmn_mtl_auth(eimTransactionId);
/* Request/download bound profile package */
f_esipa_send(valueof(ts_getBoundProfilePackageRequestEsipa));
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42956?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: I045d93b9f40ddfb3dc6dfbd9602e736bbc6b4abc
Gerrit-Change-Number: 42956
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Attention is currently required from: fixeria, laforge.
daniel has posted comments on this change by daniel. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42934?usp=email )
Change subject: deps: use a fork of titan.ProtocolEmulations.SCCP
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS1:
> I would expect the commit log to contain a mention why that fork is used, i.e. […]
Done
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42934?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: I8c0de4dab088930d5d7a9febcbebcb5e1f9411e8
Gerrit-Change-Number: 42934
Gerrit-PatchSet: 2
Gerrit-Owner: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 08 Jul 2026 13:54:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Attention is currently required from: daniel, fixeria.
Hello Jenkins Builder, fixeria,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42934?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Code-Review+1 by fixeria
Change subject: deps: use a fork of titan.ProtocolEmulations.SCCP
......................................................................
deps: use a fork of titan.ProtocolEmulations.SCCP
The fork adds support for sending UDTS via
ASP_SCCP_N_NOTICE_req.
Change-Id: I8c0de4dab088930d5d7a9febcbebcb5e1f9411e8
---
M deps/Makefile
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/34/42934/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42934?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I8c0de4dab088930d5d7a9febcbebcb5e1f9411e8
Gerrit-Change-Number: 42934
Gerrit-PatchSet: 2
Gerrit-Owner: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
daniel has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42932?usp=email )
Change subject: STP_Tests_TCAP: Test routing of fragments in TC_tcap_loadshare_ipa_tcap_range_success_pc
......................................................................
STP_Tests_TCAP: Test routing of fragments in TC_tcap_loadshare_ipa_tcap_range_success_pc
A TCAP Begin will cause the stp to create a cache entry and route
according to that. Send only a TCAP Continue so we check the tcap range
routing logic.
Change-Id: I1cb47174b8f0e69970e75109b9d15b2012e150c3
---
M stp/STP_Tests_TCAP.ttcn
1 file changed, 22 insertions(+), 2 deletions(-)
Approvals:
laforge: Looks good to me, approved
Jenkins Builder: Verified
fixeria: Looks good to me, but someone else must approve
diff --git a/stp/STP_Tests_TCAP.ttcn b/stp/STP_Tests_TCAP.ttcn
index 5093b78..f8ba240 100644
--- a/stp/STP_Tests_TCAP.ttcn
+++ b/stp/STP_Tests_TCAP.ttcn
@@ -324,6 +324,24 @@
f_asp_tx_tcap_exp_any(tcap_msg, t_asp_idx);
}
+private function f_asp_tx_tcap_m3ua_ipa_continue(integer o_asp_idx,
+ integer t_asp_idx,
+ OCT4 o_tid,
+ OCT4 t_tid)
+runs on TCAP_CT
+{
+ var template (value) TCMessage tcap_msg;
+
+ /* TCAP Continue: O -> T */
+ tcap_msg := ts_TCAP_Continue(o_tid, t_tid);
+ f_asp_tx_tcap_exp(tcap_msg, o_asp_idx, t_asp_idx);
+
+ /* TCAP Continue: O <- T */
+ tcap_msg := ts_TCAP_Continue(t_tid, o_tid);
+ f_asp_tx_tcap_exp_any(tcap_msg, t_asp_idx);
+
+}
+
private function f_asp_tx_tcap_m3ua_ipa_transaction_alt(integer o_asp_idx,
OCT4 o_tid,
integer t_asp_idx1, OCT4 t_tid1,
@@ -632,6 +650,8 @@
*
* node a: 0..99
* node b: 60..80/pc (own)
+ * Don't test a complete tcap transaction because that will create a cache entry.
+ * Testing with a continue will definitely use the tcap range.
*/
testcase TC_tcap_loadshare_ipa_tcap_range_success_pc() runs on TCAP_CT {
var Misc_Helpers.ro_charstring tcap_asps := { "asp-m3ua-loadshare-0-0",
@@ -642,8 +662,8 @@
f_init_tcap(tcap_configs);
f_tcap_loadshare_ipa_tcap_range_success_pc(tcap_configs);
- f_asp_tx_tcap_m3ua_ipa_transaction(0, 2, int2oct(23, 4), int2oct(98, 4));
- f_asp_tx_tcap_m3ua_ipa_transaction(0, 3, int2oct(60, 4), int2oct(79, 4));
+ f_asp_tx_tcap_m3ua_ipa_continue(0, 2, int2oct(98, 4), int2oct(23, 4));
+ f_asp_tx_tcap_m3ua_ipa_continue(0, 3, int2oct(79, 4), int2oct(60, 4));
/* TODO: check traffic routing with the correct PC, but different SSN, otid := 60 */
setverdict(pass);
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42932?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I1cb47174b8f0e69970e75109b9d15b2012e150c3
Gerrit-Change-Number: 42932
Gerrit-PatchSet: 1
Gerrit-Owner: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>