tests: fix SoupRequester deprecation warnings
authorDan Winship <danw@gnome.org>
Mon, 10 Dec 2012 16:30:21 +0000 (17:30 +0100)
committerDan Winship <danw@gnome.org>
Mon, 10 Dec 2012 16:30:21 +0000 (17:30 +0100)
Fix multipart-test and resource-test to use the new SoupSession APIs
rather than SoupRequester. Fix requester-test (which intentionally
tests that SoupRequester still works too) to not show deprecation
warnings.

tests/multipart-test.c
tests/requester-test.c
tests/resource-test.c

index a299684..8bcc4cd 100644 (file)
@@ -20,7 +20,7 @@ typedef enum {
 } MultipartMode;
 
 char *buffer;
-SoupRequester *requester;
+SoupSession *session;
 char *base_uri_string;
 SoupURI *base_uri;
 SoupMultipartInputStream *multipart;
@@ -494,7 +494,7 @@ static void
 test_multipart (int headers_expected, int sniffed_expected, MultipartMode multipart_mode)
 {
        GError* error = NULL;
-       SoupRequest* request = soup_requester_request (requester, base_uri_string, &error);
+       SoupRequest* request = soup_session_request (session, base_uri_string, &error);
 
        SoupMessage *msg = soup_request_http_get_message (SOUP_REQUEST_HTTP (request));
        GMainLoop *loop = g_main_loop_new (NULL, TRUE);
@@ -571,7 +571,6 @@ int
 main (int argc, char **argv)
 {
        SoupServer *server;
-       SoupSession *session;
 
        test_init (argc, argv, NULL);
 
@@ -591,11 +590,8 @@ main (int argc, char **argv)
                                         "max-conns", 20,
                                         "max-conns-per-host", 20,
                                         NULL);
-       soup_session_add_feature_by_type (session, SOUP_TYPE_REQUESTER);
        soup_session_add_feature_by_type (session, SOUP_TYPE_CONTENT_SNIFFER);
 
-       requester = SOUP_REQUESTER (soup_session_get_feature (session, SOUP_TYPE_REQUESTER));
-
        test_multipart (1, 1, NO_MULTIPART);
        test_multipart (1, 1, SYNC_MULTIPART);
        test_multipart (1, 1, ASYNC_MULTIPART);
index 4a27678..0a61815 100644 (file)
@@ -3,6 +3,9 @@
  * Copyright (C) 2011 Red Hat, Inc.
  */
 
+/* Kill SoupRequester-related deprecation warnings */
+#define SOUP_VERSION_MIN_REQUIRED SOUP_VERSION_2_40
+
 #include "test-utils.h"
 
 SoupServer *server;
index 449820a..21e2f98 100644 (file)
@@ -185,8 +185,8 @@ do_sync_request (SoupRequest *request)
 }
 
 static void
-do_request_file_test (SoupRequester *requester,
-                     gboolean       async)
+do_request_file_test (SoupSession *session,
+                     gboolean     async)
 {
        SoupRequest *request;
        GFile *index;
@@ -200,7 +200,7 @@ do_request_file_test (SoupRequester *requester,
        uri = soup_uri_new (uri_string);
        g_free (uri_string);
 
-       request = soup_requester_request_uri (requester, uri, NULL);
+       request = soup_session_request_uri (session, uri, NULL);
        if (async)
                do_async_request (request);
        else
@@ -211,8 +211,8 @@ do_request_file_test (SoupRequester *requester,
 }
 
 static void
-do_request_data_test (SoupRequester *requester,
-                     gboolean       async)
+do_request_data_test (SoupSession *session,
+                     gboolean     async)
 {
        SoupRequest *request;
        gchar *base64;
@@ -226,7 +226,7 @@ do_request_data_test (SoupRequester *requester,
        uri = soup_uri_new (uri_string);
        g_free (uri_string);
 
-       request = soup_requester_request_uri (requester, uri, NULL);
+       request = soup_session_request_uri (session, uri, NULL);
        if (async)
                do_async_request (request);
        else
@@ -237,14 +237,14 @@ do_request_data_test (SoupRequester *requester,
 }
 
 static void
-do_request_gresource_test (SoupRequester *requester,
-                          gboolean       async)
+do_request_gresource_test (SoupSession *session,
+                          gboolean     async)
 {
        SoupRequest *request;
        SoupURI *uri;
 
        uri = soup_uri_new ("resource:///org/gnome/libsoup/tests/index.txt");
-       request = soup_requester_request_uri (requester, uri, NULL);
+       request = soup_session_request_uri (session, uri, NULL);
        if (async)
                do_async_request (request);
        else
@@ -258,7 +258,6 @@ int
 main (int argc, char **argv)
 {
        SoupSession *session;
-       SoupRequester *requester;
 
        test_init (argc, argv, NULL);
 
@@ -267,13 +266,10 @@ main (int argc, char **argv)
 
        /* Sync tests */
        session = soup_test_session_new (SOUP_TYPE_SESSION_SYNC, NULL);
-       requester = soup_requester_new ();
-       soup_session_add_feature (session, SOUP_SESSION_FEATURE (requester));
-       g_object_unref (requester);
 
-       do_request_file_test (requester, FALSE);
-       do_request_data_test (requester, FALSE);
-       do_request_gresource_test (requester, FALSE);
+       do_request_file_test (session, FALSE);
+       do_request_data_test (session, FALSE);
+       do_request_gresource_test (session, FALSE);
 
        soup_test_session_abort_unref (session);
 
@@ -281,13 +277,10 @@ main (int argc, char **argv)
        session = soup_test_session_new (SOUP_TYPE_SESSION_ASYNC,
                                         SOUP_SESSION_USE_THREAD_CONTEXT, TRUE,
                                         NULL);
-       requester = soup_requester_new ();
-       soup_session_add_feature (session, SOUP_SESSION_FEATURE (requester));
-       g_object_unref (requester);
 
-       do_request_file_test (requester, TRUE);
-       do_request_data_test (requester, TRUE);
-       do_request_gresource_test (requester, TRUE);
+       do_request_file_test (session, TRUE);
+       do_request_data_test (session, TRUE);
+       do_request_gresource_test (session, TRUE);
 
        soup_test_session_abort_unref (session);