From 2c9168cf34d370b4ea2b21177402d01a7cfffea8 Mon Sep 17 00:00:00 2001 From: Holger Hans Peter Freyther Date: Thu, 10 Oct 2013 20:21:33 +0200 Subject: vty: Make vty_event dispatch signals and use it in the testcase The testcase didn't work on Ubuntu 12.04 because vty_create will directly call vty_event (e.g. not through the plt). This means that the approach to override vty_event in the testcase failed. Use the signal interface of libosmocore and make the testcase use it. The signals can be generally useful as well. --- tests/vty/vty_test.c | 21 +++++++++++++++++---- 1 file changed, 17 insertions(+), 4 deletions(-) (limited to 'tests/vty') diff --git a/tests/vty/vty_test.c b/tests/vty/vty_test.c index 5f93b3db..2a7347f8 100644 --- a/tests/vty/vty_test.c +++ b/tests/vty/vty_test.c @@ -28,6 +28,7 @@ #include #include #include +#include #include #include #include @@ -67,12 +68,22 @@ static int do_vty_command(struct vty *vty, const char *cmd) return ret; } -/* Override the implementation from telnet_interface.c */ -void vty_event(enum event event, int sock, struct vty *vty) +/* Handle the events from telnet_interface.c */ +static int vty_event_cb(unsigned int subsys, unsigned int signal, + void *handler_data, void *_signal_data) { - last_vty_connection_event = event; + struct vty_signal_data *signal_data; - fprintf(stderr, "Got VTY event: %d\n", event); + if (subsys != SS_L_VTY) + return 0; + if (signal != S_VTY_EVENT) + return 0; + + signal_data = _signal_data; + last_vty_connection_event = signal_data->event; + + fprintf(stderr, "Got VTY event: %d\n", signal_data->event); + return 0; } static void test_node_tree_structure(void) @@ -169,6 +180,8 @@ static void test_node_tree_structure(void) int main(int argc, char **argv) { + osmo_signal_register_handler(SS_L_VTY, vty_event_cb, NULL); + test_cmd_string_from_valstr(); test_node_tree_structure(); printf("All tests passed\n"); -- cgit v1.2.3