aboutsummaryrefslogtreecommitdiffstats
path: root/tests/msc_vlr/msc_vlr_tests.h
diff options
context:
space:
mode:
authorNeels Hofmeyr <neels@hofmeyr.de>2018-03-02 00:40:58 +0100
committerNeels Hofmeyr <neels@hofmeyr.de>2018-03-02 03:22:16 +0100
commitdfdc61de68f9b2942994b4cfe6af4ebb52ffc96e (patch)
tree308f4d9ed78f0e6b3d4161a920e6c99297084ab4 /tests/msc_vlr/msc_vlr_tests.h
parentc5e0ace751c72a08d464f50a1ade33ba7af04cc1 (diff)
msc_vlr_tests: revert IMSI parameter and test nr output
Three recently merged commits take the msc_vlr_tests in a wrong direction. The IMSI is usually encoded in the hex streams. The rationale behind hex streams is that it is a) easily copied from a wireshark trace and b) exactly the bytes as sent by an actual phone. It is hard to parameterize the IMSI because we would have to employ our encoding functions, which I intentionally want to keep out of the loop here. The test number should not appear in the normal test output, so that adding a test or changing their order does not affect expected output for following tests. The nr is simply for manual invocation, only seen when invoked with -v. Revert - "VLR tests: always print test parameters" b0a4314911140b1599cccfc8171fcdab4cd9bfab. - "Expand VLR tests" d5feadeee8dd24f991df2892d6bcf0be8b0cf707. - "Move IMSI into test parameters" 093300d141c300651954473d73138b72de04d931. Change-Id: Ie1b49237746751021da88f6f07bbb9f780d077c9
Diffstat (limited to 'tests/msc_vlr/msc_vlr_tests.h')
-rw-r--r--tests/msc_vlr/msc_vlr_tests.h6
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/msc_vlr/msc_vlr_tests.h b/tests/msc_vlr/msc_vlr_tests.h
index d00eb9ffa..03aa3aeef 100644
--- a/tests/msc_vlr/msc_vlr_tests.h
+++ b/tests/msc_vlr/msc_vlr_tests.h
@@ -46,8 +46,8 @@ extern bool _log_lines;
#define btw(fmt, args...) _log("- " fmt, ## args )
#define log(fmt, args...) _log(" " fmt, ## args )
-#define comment_start(nr, imsi) fprintf(stderr, "===== [%u] %s(%s)...\n", nr, __func__, imsi);
-#define comment_end(nr, imsi) fprintf(stderr, "===== [%u] %s(%s): SUCCESS\n\n", nr, __func__, imsi);
+#define comment_start() fprintf(stderr, "===== %s\n", __func__);
+#define comment_end() fprintf(stderr, "===== %s: SUCCESS\n\n", __func__);
extern struct gsm_subscriber_connection *g_conn;
extern struct gsm_network *net;
@@ -122,7 +122,7 @@ struct msc_vlr_test_cmdline_opts {
int run_test_nr;
};
-typedef void (* msc_vlr_test_func_t )(uint8_t, const char *);
+typedef void (* msc_vlr_test_func_t )(void);
extern msc_vlr_test_func_t msc_vlr_tests[];
struct msgb *msgb_from_hex(const char *label, uint16_t size, const char *hex);