laforge has submitted this change. ( 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, 46 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/eim/eIM_Tests.ttcn b/eim/eIM_Tests.ttcn
index 8f48528..a8a0071 100644
--- a/eim/eIM_Tests.ttcn
+++ b/eim/eIM_Tests.ttcn
@@ -793,6 +793,51 @@
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;
+ var template JSON_REST_Results expected_outcome :=
+ { { euiccDataResult := { edrResult := "undefinedError", euiccData := omit } } };
+
+ 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(outcome := expected_outcome) );
+ 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 +852,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: merged
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ied212eb636cf46df5fc9f86ddee15c242c50d45a
Gerrit-Change-Number: 42955
Gerrit-PatchSet: 10
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
laforge has submitted this change. ( 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(-)
Approvals:
jolly: Looks good to me, but someone else must approve
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/eim/eIM_Tests.ttcn b/eim/eIM_Tests.ttcn
index a8a0071..0897865 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: merged
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I045d93b9f40ddfb3dc6dfbd9602e736bbc6b4abc
Gerrit-Change-Number: 42956
Gerrit-PatchSet: 10
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Attention is currently required from: dexter.
laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43012?usp=email )
Change subject: eIM_Tests: add testcase to test stateChangeCause/stateChangeCause
......................................................................
Patch Set 5: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43012?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: I6befc43f2dfd2178e352b9ab0a24a04d6e116fbb
Gerrit-Change-Number: 43012
Gerrit-PatchSet: 5
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 10 Aug 2026 09:41:30 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes