laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/38271?usp=email ) Change subject: pySim-shell_test: disable test_list_and_rm_notif ...................................................................... pySim-shell_test: disable test_list_and_rm_notif The testcase euicc.test_list_and_rm_notif fails due to a problem with the eUICC. The eUICC reports the following error when a delete notification attempt is made: "delete_notification_status": "undefinedError" Let's temporarily disable this testcase until the problem is resolved. Change-Id: I7d0b6a998499d84f0eb4e24592ad43210ac54806 --- M tests/pySim-shell_test/euicc/test.py 1 file changed, 28 insertions(+), 28 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/tests/pySim-shell_test/euicc/test.py b/tests/pySim-shell_test/euicc/test.py index b634f22..f7226a3 100644 --- a/tests/pySim-shell_test/euicc/test.py +++ b/tests/pySim-shell_test/euicc/test.py @@ -62,34 +62,34 @@ self.runPySimShell(cardname, "test_set_nickname.script") self.assertEqualFiles("set_nickname.tmp") - def test_list_and_rm_notif(self): - cardname = 'sysmoEUICC1-C2T' - - # Generate two (additional) notification - self.runPySimShell(cardname, "test_gen_notif.script") - - # List notifications into a file - self.runPySimShell(cardname, "test_list_notif.script") - - # Parse notifications file (JSON) - notifications_tmp = open("notifications.tmp") - notifications = json.load(notifications_tmp) - notifications_tmp.close() - - # Delete notifications one by one, we expect to see at least one notification - notification_metadata_list = notifications['notification_metadata_list'] - removed = 0 - for nm in notification_metadata_list: - seq_number = nm['notification_metadata']['seq_number'] - print("removing notification with seq_number %s:" % seq_number) - self.equipTemplate("test_rm_notif.script", SEQ_NUMBER = seq_number) - self.runPySimShell(cardname, "test_rm_notif.script") - removed = removed + 1 - self.assertTrue(removed >= 2, "we expected to remove at least two notifications, but we have removed none!") - - # List notifications again, require none to be present - self.runPySimShell(cardname, "test_list_notif.script") - self.assertEqualFiles("notifications.tmp") +# def test_list_and_rm_notif(self): +# cardname = 'sysmoEUICC1-C2T' +# +# # Generate two (additional) notification +# self.runPySimShell(cardname, "test_gen_notif.script") +# +# # List notifications into a file +# self.runPySimShell(cardname, "test_list_notif.script") +# +# # Parse notifications file (JSON) +# notifications_tmp = open("notifications.tmp") +# notifications = json.load(notifications_tmp) +# notifications_tmp.close() +# +# # Delete notifications one by one, we expect to see at least one notification +# notification_metadata_list = notifications['notification_metadata_list'] +# removed = 0 +# for nm in notification_metadata_list: +# seq_number = nm['notification_metadata']['seq_number'] +# print("removing notification with seq_number %s:" % seq_number) +# self.equipTemplate("test_rm_notif.script", SEQ_NUMBER = seq_number) +# self.runPySimShell(cardname, "test_rm_notif.script") +# removed = removed + 1 +# self.assertTrue(removed >= 2, "we expected to remove at least two notifications, but we have removed none!") +# +# # List notifications again, require none to be present +# self.runPySimShell(cardname, "test_list_notif.script") +# self.assertEqualFiles("notifications.tmp") if __name__ == "__main__": unittest.main() -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38271?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I7d0b6a998499d84f0eb4e24592ad43210ac54806 Gerrit-Change-Number: 38271 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>