Attention is currently required from: laforge.
dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/33963 )
Change subject: tests: add test script for pySim-trace ......................................................................
Patch Set 4:
(1 comment)
Patchset:
PS4:
this is good for a very basic test, but *not raising an exception* is of course a rather vague "pass […]
I think matching the number of APDUs is a good idea. (Unfortunately I already spotted discrepancies between the number of packets shown in Wireshark and the number of APDUs we currently get - but the end looks good.)
Comparing the output might be prone to produce false alarms from time to time. Fortunately the test does not need any special environment, so it should be easy to manage. I would say we should try it.