[glib] GApplication: fix bogus testcase



commit ffbf866d3b3dfae234fbdeef8bab683c14d66e30
Author: Ryan Lortie <desrt desrt ca>
Date:   Tue Oct 29 12:02:44 2013 -0700

    GApplication: fix bogus testcase
    
    We had a GApplication testcase that handled both open and commandline.
    This only way that this worked was by implementing the commandline
    handler without actually setting the HANDLES_COMMAND_LINE flag.
    
    This behaviour is now invalid, so just rip out the offending part of the
    test.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=711099

 gio/gapplicationimpl-dbus.c   |    3 ++-
 gio/tests/basic-application.c |   23 -----------------------
 gio/tests/gapplication.c      |    5 -----
 3 files changed, 2 insertions(+), 29 deletions(-)
---
diff --git a/gio/gapplicationimpl-dbus.c b/gio/gapplicationimpl-dbus.c
index 2c77fee..a5b7bdb 100644
--- a/gio/gapplicationimpl-dbus.c
+++ b/gio/gapplicationimpl-dbus.c
@@ -262,7 +262,8 @@ g_application_impl_method_call (GDBusConnection       *connection,
       flags = g_application_get_flags (impl->app);
       if ((flags & G_APPLICATION_HANDLES_COMMAND_LINE) == 0)
         {
-          g_dbus_method_invocation_return_error (invocation, G_DBUS_ERROR, G_DBUS_ERROR_NOT_SUPPORTED, 
"Application does not handle command line arguments");
+          g_dbus_method_invocation_return_error (invocation, G_DBUS_ERROR, G_DBUS_ERROR_NOT_SUPPORTED,
+                                                 "Application does not handle command line arguments");
           return;
         }
 
diff --git a/gio/tests/basic-application.c b/gio/tests/basic-application.c
index 60c979d..a2b797a 100644
--- a/gio/tests/basic-application.c
+++ b/gio/tests/basic-application.c
@@ -31,28 +31,6 @@ open (GApplication  *application,
   g_print ("\n");
 }
 
-static int
-command_line (GApplication            *application,
-              GApplicationCommandLine *cmdline)
-{
-  gchar **argv;
-  gint argc;
-
-  argv = g_application_command_line_get_arguments (cmdline, &argc);
-
-  g_application_command_line_print (cmdline, "%d + %d = %d\n", 40, 2, 42);
-
-  g_assert_cmpint (argc, ==, 3);
-  g_assert_cmpstr (argv[0], ==, "./cmd");
-  g_assert_cmpstr (argv[1], ==, "40 +");
-  g_assert_cmpstr (argv[2], ==, "2");
-  g_assert (argv[3] == NULL);
-  g_print ("cmdline '%s' '%s'\n", argv[1], argv[2]);
-  g_strfreev (argv);
-
-  return 42;
-}
-
 int
 main (int argc, char **argv)
 {
@@ -67,7 +45,6 @@ main (int argc, char **argv)
                            : 0));
   g_signal_connect (app, "activate", G_CALLBACK (activate), NULL);
   g_signal_connect (app, "open", G_CALLBACK (open), NULL);
-  g_signal_connect (app, "command-line", G_CALLBACK (command_line), NULL);
 #ifdef STANDALONE
   g_application_set_inactivity_timeout (app, 10000);
 #else
diff --git a/gio/tests/gapplication.c b/gio/tests/gapplication.c
index c9610c0..b6f911e 100644
--- a/gio/tests/gapplication.c
+++ b/gio/tests/gapplication.c
@@ -107,7 +107,6 @@ basic (void)
   /* spawn the master */
   spawn ("activated\n"
          "open file:///a file:///b\n"
-         "cmdline '40 +' '2'\n"
          "exit status: 0\n",
          "./app", NULL);
 
@@ -115,10 +114,6 @@ basic (void)
   spawn ("exit status: 0\n",
          "./app", "/a", "/b", NULL);
 
-  spawn ("40 + 2 = 42\n"
-         "exit status: 42\n",
-         "./cmd", "40 +", "2", NULL);
-
   g_main_loop_run (main_loop);
 
   g_object_unref (c);


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]