diff --git a/tests/common/base-test-fixture.h b/tests/common/base-test-fixture.h index 976ee186b9fcb169989da7e8d3fa48886d4012cf..aa9eb838a80c234f674fb04be66c426ffb4bc04d 100644 --- a/tests/common/base-test-fixture.h +++ b/tests/common/base-test-fixture.h @@ -107,3 +107,39 @@ wp_base_test_fixture_teardown (WpBaseTestFixture * self) g_clear_pointer (&self->context, g_main_context_unref); wp_test_server_teardown (&self->server); } + +static G_GNUC_UNUSED void +test_proxy_augment_finish_cb (WpProxy * proxy, GAsyncResult * res, + WpBaseTestFixture * f) +{ + g_autoptr (GError) error = NULL; + gboolean augment_ret = wp_proxy_augment_finish (proxy, res, &error); + g_assert_no_error (error); + g_assert_true (augment_ret); + + g_main_loop_quit (f->loop); +} + +static G_GNUC_UNUSED void +test_si_activate_finish_cb (WpSessionItem * item, GAsyncResult * res, + WpBaseTestFixture * f) +{ + g_autoptr (GError) error = NULL; + gboolean activate_ret = wp_session_item_activate_finish (item, res, &error); + g_assert_no_error (error); + g_assert_true (activate_ret); + + g_main_loop_quit (f->loop); +} + +static G_GNUC_UNUSED void +test_si_export_finish_cb (WpSessionItem * item, GAsyncResult * res, + WpBaseTestFixture * f) +{ + g_autoptr (GError) error = NULL; + gboolean export_ret = wp_session_item_export_finish (item, res, &error); + g_assert_no_error (error); + g_assert_true (export_ret); + + g_main_loop_quit (f->loop); +} diff --git a/tests/modules/si-simple-node-endpoint.c b/tests/modules/si-simple-node-endpoint.c index 10fc737da78bb216deffa6a69dffabe5ce7f7615..25afa9405cc1d02b82640558f433cfd9494ffc77 100644 --- a/tests/modules/si-simple-node-endpoint.c +++ b/tests/modules/si-simple-node-endpoint.c @@ -52,39 +52,6 @@ test_si_simple_node_endpoint_teardown (TestFixture * f, gconstpointer user_data) wp_base_test_fixture_teardown (&f->base); } -static void -on_proxy_augmented (WpProxy * proxy, GAsyncResult * res, TestFixture * f) -{ - g_autoptr (GError) error = NULL; - gboolean augment_ret = wp_proxy_augment_finish (proxy, res, &error); - g_assert_no_error (error); - g_assert_true (augment_ret); - - g_main_loop_quit (f->base.loop); -} - -static void -on_item_activated (WpSessionItem * item, GAsyncResult * res, TestFixture * f) -{ - g_autoptr (GError) error = NULL; - gboolean activate_ret = wp_session_item_activate_finish (item, res, &error); - g_assert_no_error (error); - g_assert_true (activate_ret); - - g_main_loop_quit (f->base.loop); -} - -static void -on_item_exported (WpSessionItem * item, GAsyncResult * res, TestFixture * f) -{ - g_autoptr (GError) error = NULL; - gboolean export_ret = wp_session_item_export_finish (item, res, &error); - g_assert_no_error (error); - g_assert_true (export_ret); - - g_main_loop_quit (f->base.loop); -} - static void test_si_simple_node_endpoint_configure_activate (TestFixture * f, gconstpointer user_data) @@ -111,7 +78,7 @@ test_si_simple_node_endpoint_configure_activate (TestFixture * f, g_assert_nonnull (node); wp_proxy_augment (WP_PROXY (node), WP_PROXY_FEATURES_STANDARD, NULL, - (GAsyncReadyCallback) on_proxy_augmented, f); + (GAsyncReadyCallback) test_proxy_augment_finish_cb, f); g_main_loop_run (f->base.loop); /* configure */ @@ -155,7 +122,8 @@ test_si_simple_node_endpoint_configure_activate (TestFixture * f, /* activate */ - wp_session_item_activate (item, (GAsyncReadyCallback) on_item_activated, f); + wp_session_item_activate (item, + (GAsyncReadyCallback) test_si_activate_finish_cb, f); g_main_loop_run (f->base.loop); g_assert_cmphex (wp_session_item_get_flags (item), ==, @@ -295,7 +263,7 @@ test_si_simple_node_endpoint_export (TestFixture * f, gconstpointer user_data) g_assert_nonnull (node); wp_proxy_augment (WP_PROXY (node), WP_PROXY_FEATURES_STANDARD, NULL, - (GAsyncReadyCallback) on_proxy_augmented, f); + (GAsyncReadyCallback) test_proxy_augment_finish_cb, f); g_main_loop_run (f->base.loop); /* configure */ @@ -317,7 +285,8 @@ test_si_simple_node_endpoint_export (TestFixture * f, gconstpointer user_data) /* activate */ - wp_session_item_activate (item, (GAsyncReadyCallback) on_item_activated, f); + wp_session_item_activate (item, + (GAsyncReadyCallback) test_si_activate_finish_cb, f); g_main_loop_run (f->base.loop); g_assert_cmpint ( @@ -331,13 +300,13 @@ test_si_simple_node_endpoint_export (TestFixture * f, gconstpointer user_data) g_assert_nonnull (session); wp_proxy_augment (WP_PROXY (session), WP_SESSION_FEATURES_STANDARD, NULL, - (GAsyncReadyCallback) on_proxy_augmented, f); + (GAsyncReadyCallback) test_proxy_augment_finish_cb, f); g_main_loop_run (f->base.loop); /* export */ wp_session_item_export (item, session, - (GAsyncReadyCallback) on_item_exported, f); + (GAsyncReadyCallback) test_si_export_finish_cb, f); g_main_loop_run (f->base.loop); g_assert_cmphex (wp_session_item_get_flags (item), ==,