From 8aa780bf80fa4ffde8a5df9d8b87a2e919193f1f Mon Sep 17 00:00:00 2001 From: Neels Hofmeyr Date: Sun, 2 Dec 2018 18:52:49 +0100 Subject: [PATCH] add 'show subscriber' command, alias for 'subscriber ... show' When I wrote the osmo-hlr subscriber command, I failed to heed the common 'show foo' scheme and instead created a 'subscriber [...] show' command. Relieve that weirdness by creating an alias that has 'show' at the start. Arrange string macros so that the 'show subscriber' cmd doesn't end in a space (the SUBSCR macro ends in a space ' ' to implicitly include the space to commands like 'create', 'show', 'update'). Add the new command to test_nodes.vty and test_subscriber.vty. Change-Id: I01ce9b0868302d40ed05c6a588316a194d6071e4 --- src/hlr_vty_subscr.c | 9 +++++++-- tests/test_nodes.vty | 1 + tests/test_subscriber.vty | 8 ++++++++ 3 files changed, 16 insertions(+), 2 deletions(-) diff --git a/src/hlr_vty_subscr.c b/src/hlr_vty_subscr.c index 6ce96fe2..ddbaf261 100644 --- a/src/hlr_vty_subscr.c +++ b/src/hlr_vty_subscr.c @@ -130,14 +130,14 @@ static int get_subscr_by_argv(struct vty *vty, const char *type, const char *id, #define SUBSCR_CMD "subscriber " #define SUBSCR_CMD_HELP "Subscriber management commands\n" -#define SUBSCR_ID "(imsi|msisdn|id) IDENT " +#define SUBSCR_ID "(imsi|msisdn|id) IDENT" #define SUBSCR_ID_HELP \ "Identify subscriber by IMSI\n" \ "Identify subscriber by MSISDN (phone number)\n" \ "Identify subscriber by database ID\n" \ "IMSI/MSISDN/ID of the subscriber\n" -#define SUBSCR SUBSCR_CMD SUBSCR_ID +#define SUBSCR SUBSCR_CMD SUBSCR_ID " " #define SUBSCR_HELP SUBSCR_CMD_HELP SUBSCR_ID_HELP #define SUBSCR_UPDATE SUBSCR "update " @@ -159,6 +159,10 @@ DEFUN(subscriber_show, return CMD_SUCCESS; } +ALIAS(subscriber_show, show_subscriber_cmd, + "show " SUBSCR_CMD SUBSCR_ID, + SHOW_STR SUBSCR_CMD_HELP SUBSCR_ID_HELP); + DEFUN(subscriber_create, subscriber_create_cmd, SUBSCR_CMD "imsi IDENT create", @@ -478,6 +482,7 @@ DEFUN(subscriber_aud3g, void hlr_vty_subscriber_init(void) { install_element_ve(&subscriber_show_cmd); + install_element_ve(&show_subscriber_cmd); install_element(ENABLE_NODE, &subscriber_create_cmd); install_element(ENABLE_NODE, &subscriber_delete_cmd); install_element(ENABLE_NODE, &subscriber_msisdn_cmd); diff --git a/tests/test_nodes.vty b/tests/test_nodes.vty index a9d4ac40..e1f57a96 100644 --- a/tests/test_nodes.vty +++ b/tests/test_nodes.vty @@ -18,6 +18,7 @@ OsmoHLR> list show talloc-context (application|all) (full|brief|DEPTH) filter REGEXP show gsup-connections subscriber (imsi|msisdn|id) IDENT show + show subscriber (imsi|msisdn|id) IDENT OsmoHLR> enable OsmoHLR# list diff --git a/tests/test_subscriber.vty b/tests/test_subscriber.vty index 2da455fe..c22f2df6 100644 --- a/tests/test_subscriber.vty +++ b/tests/test_subscriber.vty @@ -3,6 +3,7 @@ OsmoHLR> enable OsmoHLR# list ... subscriber (imsi|msisdn|id) IDENT show + show subscriber (imsi|msisdn|id) IDENT subscriber imsi IDENT create subscriber (imsi|msisdn|id) IDENT delete subscriber (imsi|msisdn|id) IDENT update msisdn MSISDN @@ -33,6 +34,13 @@ OsmoHLR# subscriber id 1 show OsmoHLR# subscriber msisdn 12345 show % No subscriber for msisdn = '12345' +OsmoHLR# show subscriber imsi 123456789023000 +% No subscriber for imsi = '123456789023000' +OsmoHLR# show subscriber id 1 +% No subscriber for id = '1' +OsmoHLR# show subscriber msisdn 12345 +% No subscriber for msisdn = '12345' + OsmoHLR# subscriber imsi 1234567890230001 create % Not a valid IMSI: 1234567890230001 OsmoHLR# subscriber imsi 12345678902300x create