Attention is currently required from: dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37504?usp=email )
Change subject: IPAd_Tests: expect the IPAd to request a notifications list on init
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37504?usp=email
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: I2cd7fdfde24404bc8cb1af258923cd86627db68a
Gerrit-Change-Number: 37504
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 16 Jul 2024 09:11:07 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37504?usp=email )
Change subject: IPAd_Tests: expect the IPAd to request a notifications list on init
......................................................................
IPAd_Tests: expect the IPAd to request a notifications list on init
The IPAd now requests a list of pending notifications before it fetches
the a new eIM package.
Related: SYS#6563
Change-Id: I2cd7fdfde24404bc8cb1af258923cd86627db68a
---
M ipad/IPAd_Tests.ttcn
1 file changed, 16 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/04/37504/1
diff --git a/ipad/IPAd_Tests.ttcn b/ipad/IPAd_Tests.ttcn
index d1f9214..04e4b1d 100644
--- a/ipad/IPAd_Tests.ttcn
+++ b/ipad/IPAd_Tests.ttcn
@@ -330,6 +330,9 @@
/* Expect the IPAd to query the eIM configuration data from the eUICC */
f_vpcd_transceive(enc_GetEimConfigurationDataResponse(valueof(ts_getEimConfigurationDataResponse(eim_fqdn))), 'BF5500'O);
+
+ /* Expect the IPAd to query the eUICC for pending notifications, we respond with an empty list */
+ f_vpcd_transceive(enc_RetrieveNotificationsListResponse(valueof(ts_retrieveNotificationsListResponse_empty)), 'BF2B00'O);
}
/* Handle the closing of logical channel 1 */
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37504?usp=email
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: I2cd7fdfde24404bc8cb1af258923cd86627db68a
Gerrit-Change-Number: 37504
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: newchange
dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37503?usp=email )
Change subject: RSPDefinitions_Templates: add variants of ts_retrieveNotificationsListResponse
......................................................................
RSPDefinitions_Templates: add variants of ts_retrieveNotificationsListResponse
There is currently only ts_retrieveNotificationsListResponse, which has a
profileInstallationResult in its notificationList. However, we will need
different variants of this template, so let's give it an appropriate prefix.
Let's also add a first variant that has just an empty notificationList
Change-Id: I1802b63a51bf68080b3ef81a967a964d961a1b96
Related: SYS#6563
---
M ipad/IPAd_Tests.ttcn
M library/euicc/RSPDefinitions_Templates.ttcn
2 files changed, 25 insertions(+), 5 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/03/37503/1
diff --git a/ipad/IPAd_Tests.ttcn b/ipad/IPAd_Tests.ttcn
index 3ea2e4b..d1f9214 100644
--- a/ipad/IPAd_Tests.ttcn
+++ b/ipad/IPAd_Tests.ttcn
@@ -515,7 +515,7 @@
f_vpcd_transceive(enc_EuiccPackageResult(valueof(ts_euiccPackageResult)));
/* Step #9 */
- f_vpcd_transceive(enc_RetrieveNotificationsListResponse(valueof(ts_retrieveNotificationsListResponse)));
+ f_vpcd_transceive(enc_RetrieveNotificationsListResponse(valueof(ts_retrieveNotificationsListResponse_prfleInstRes)));
/* Step #10-14 */
f_esipa_transceive(valueof(ts_provideEimPackageResultResponse_eimAck(eimAcknowledgements := {1,2,3,4})),
@@ -560,7 +560,7 @@
f_vpcd_transceive(enc_EuiccPackageResult(valueof(ts_euiccPackageResult)));
/* Step #9 */
- f_vpcd_transceive(enc_RetrieveNotificationsListResponse(valueof(ts_retrieveNotificationsListResponse)));
+ f_vpcd_transceive(enc_RetrieveNotificationsListResponse(valueof(ts_retrieveNotificationsListResponse_prfleInstRes)));
/* We now ignore the response from the IPAd. The IPAd will interpret this as a disturbed IP connection. */
f_esipa_receive();
@@ -573,7 +573,7 @@
* to continue the procedure normally. */
/* Step #9 */
- f_vpcd_transceive(enc_RetrieveNotificationsListResponse(valueof(ts_retrieveNotificationsListResponse)));
+ f_vpcd_transceive(enc_RetrieveNotificationsListResponse(valueof(ts_retrieveNotificationsListResponse_prfleInstRes)));
/* Step #10-14 */
f_esipa_transceive(valueof(ts_provideEimPackageResultResponse_eimAck(eimAcknowledgements := {1,2,3,4})),
@@ -621,7 +621,7 @@
f_vpcd_transceive(enc_EUICCInfo2(valueof(ts_EUICCInfo2)));
f_vpcd_transceive(enc_GetEimConfigurationDataResponse(valueof(ts_getEimConfigurationDataResponse(eim_fqdn))));
f_vpcd_transceive(enc_GetCertsResponse(valueof(ts_getCertsResponse)));
- f_vpcd_transceive(enc_RetrieveNotificationsListResponse(valueof(ts_retrieveNotificationsListResponse)));
+ f_vpcd_transceive(enc_RetrieveNotificationsListResponse(valueof(ts_retrieveNotificationsListResponse_prfleInstRes)));
/* IPAd will return the data to us */
f_esipa_transceive(valueof(ts_provideEimPackageResultResponse_eimAck(eimAcknowledgements := {1,2,3,4})),
diff --git a/library/euicc/RSPDefinitions_Templates.ttcn b/library/euicc/RSPDefinitions_Templates.ttcn
index ebc2490..2a7df8b 100644
--- a/library/euicc/RSPDefinitions_Templates.ttcn
+++ b/library/euicc/RSPDefinitions_Templates.ttcn
@@ -559,7 +559,7 @@
/* GSMA SGP.22, section 5.7.10 (ES10x) */
template (value) RetrieveNotificationsListResponse
-ts_retrieveNotificationsListResponse := {
+ts_retrieveNotificationsListResponse_prfleInstRes := {
notificationList := {
{
profileInstallationResult := {
@@ -569,6 +569,11 @@
}
}
}
+template (value) RetrieveNotificationsListResponse
+ts_retrieveNotificationsListResponse_empty := {
+ notificationList := {
+ }
+}
/* GSMA SGP.22, section 5.7.11 (ES10x) */
template (present) NotificationSentResponse
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37503?usp=email
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: I1802b63a51bf68080b3ef81a967a964d961a1b96
Gerrit-Change-Number: 37503
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: newchange
Attention is currently required from: dexter, laforge.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/37496?usp=email )
Change subject: pySim.esim.http_json_api: 'header' is not always present in response
......................................................................
Patch Set 1:
(1 comment)
File pySim/esim/http_json_api.py:
https://gerrit.osmocom.org/c/pysim/+/37496/comment/a2fb6d86_ded4985b
PS1, Line 259: return None
this violates the return type hint for this function
I think it should be `Optional[dict]` now
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/37496?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ia3c5703b746c1eba91f85f8545f849a3f2d56e0b
Gerrit-Change-Number: 37496
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 16 Jul 2024 08:16:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment