After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 711099 - gapplication test failure
gapplication test failure
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: gapplication
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2013-10-29 19:47 UTC by Christian Persch
Modified: 2013-11-07 18:51 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
GApplication: fix bogus testcase (3.31 KB, patch)
2013-10-29 20:42 UTC, Allison Karlitskaya (desrt) 		committed Details | Review

Description Christian Persch 2013-10-29 19:47:49 UTC 
This is 100% reproducible here (glib git master on otherwise F19 system).

$ ./gapplication
/gapplication/no-dbus: OK
/gapplication/basic: GDBus.Error:org.freedesktop.DBus.Error.NotSupported: Application does not handle command line arguments

(/home/chpe/source/git/glib/gio/tests/.libs/lt-gapplication:20222): GLib-GIO-ERROR **: 
Expected
-----
40 + 2 = 42
exit status: 42
-----
Got (full)
-----
exit status: 1
-----

cleaning up pid 20238


Adding some debug output to basic-application.c reveals that for the 3rd spawn() in gapplicaton.c:basic() test, the output received from the child in child_quit() is the output of the 2nd spawn() invocation in basic().
Comment 1 Allison Karlitskaya (desrt) 2013-10-29 20:42:34 UTC 
Created attachment 258487 [details] [review]
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.
Comment 2 Christian Persch 2013-10-29 20:46:22 UTC 
The patch fixes the test failure, thanks :-)
Comment 3 Allison Karlitskaya (desrt) 2013-11-07 18:51:52 UTC 
Attachment 258487 [details] pushed as ffbf866 - GApplication: fix bogus testcase