Attention is currently required from: fixeria, dexter.
8 comments:
File pySim/ts_31_102.py:
Patch Set #4, Line 294: _encode_decode_testvector = ['{"prot_scheme_id_list": [{"priority": 0, "identifier": 2, "key_index": 1}, {"priority": 1, "identifier": 1, "key_index": 2}, {"priority": 2, "identifier": 0, "key_index": 0}], "hnet_pubkey_list": [{"hnet_pubkey_identifier": 27, "hnet_pubkey": "0272da71976234ce833a6907425867b82e074d44ef907dfb4b3e21c1c2256ebcd1"}, {"hnet_pubkey_identifier": 30, "hnet_pubkey": "5a8d38864820197c3394b92613b20b91633cbd897119273bf8e4a6f4eec0a650"}]}']
> I agree that this line is not easy to read but you can copy+paste it into pySim-shell and it will […]
I went for python notation and not json/string, after all.
File pySim/ts_31_103.py:
Patch Set #4, Line 121: '{"addr": "hello,world", "addr_type": "00"}']
mixing tabs and spaces, let's avoid this
Done
File pySim/ts_51_011.py:
Patch Set #4, Line 380: '{"msisdn": [1, 3, "123456"]}']
mixing tabs and spaces, let's avoid this
Done
Patch Set #4, Line 471: '{"imsi": "123456789012345"}']
same here and below
Done
File tests/test_files.py:
Patch Set #4, Line 57: for testvec_json in ef.__class__._encode_decode_testvector:
The problem without subTest is that it simply stops a the first assert and doesn't continue to check […]
Done
Patch Set #4, Line 71: testcase.assertEqual(testvec_json, decoded_json)
In my opinion matching the strings is the best way possible. […]
Done
Patch Set #4, Line 94: for obj in gc.get_objects():
we already have utils.py:all_subclasses() for getting that recursive list.
Done
Patch Set #4, Line 96: if not file_in_list(test_candidates, obj.name):
Ack
Done
To view, visit change 24012. To unsubscribe, or for help writing mail filters, visit settings.