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 634383 - t/GtkStatusIcon.t fails tests 31 and 33 on Windows
t/GtkStatusIcon.t fails tests 31 and 33 on Windows
Status: RESOLVED FIXED
Product: gnome-perl
Classification: Bindings
Component: Gtk2
unspecified
Other Windows
: Normal normal
: ---
Assigned To: gtk2-perl-bugs
gtk2-perl-bugs
Depends on:
Blocks:
 
 
Reported: 2010-11-09 04:26 UTC by sisyphus1
Modified: 2011-08-23 18:42 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix a test failure in t/GtkStatusIcon.t (1.17 KB, patch)
2011-08-16 19:34 UTC, Torsten Schoenfeld
committed Details | Review

Description sisyphus1 2010-11-09 04:26:46 UTC
Hi,

This is in relation to version 1.222 of Gtk2.

Here's the relevant section of the output from running the test script:

############################
ok 30
not ok 31
#   Failed test at t/GtkStatusIcon.t line 144.
#          got: ''
#     expected: '1'
ok 32
not ok 33
#   Failed test at t/GtkStatusIcon.t line 150.
#          got: 'TEST1'
#     expected: '<b>TEST1</b>'
ok 34
############################

And the relevant section of t/GtkStatusIcon.t:

############################
143:   $icon->set_has_tooltip(TRUE);
144:   is ($icon->get_has_tooltip(), TRUE);
145:
146:   $icon->set_has_tooltip(FALSE);
147:   is ($icon->get_has_tooltip(), FALSE);
148:
############################

The same tests fail (in exactly the same way) for perls 5.8.0, 5.10.0 and 5.12.0 (both 32-bit and 64-bit builds of 5.12.0).

Cheers,
Rob
Comment 1 Torsten Schoenfeld 2011-08-16 19:34:05 UTC
Created attachment 193989 [details] [review]
Fix a test failure in t/GtkStatusIcon.t
Comment 2 Torsten Schoenfeld 2011-08-16 19:34:32 UTC
Sorry for the late response.  Does the above patch fix the problem?
Comment 3 sisyphus1 2011-08-18 07:28:42 UTC
Yes, the patch looks good.
Thanks, Torsten.
Comment 4 Torsten Schoenfeld 2011-08-23 18:42:14 UTC
Attachment 193989 [details] pushed as e6fedbf - Fix a test failure in t/GtkStatusIcon.t

Thanks for the report.