Merge lp:~gue5t/midori/appmenu-visibility-test into lp:midori

Proposed by gue5t gue5t
Status: Merged
Approved by: Paweł Forysiuk
Approved revision: 7022
Merged at revision: 7022
Proposed branch: lp:~gue5t/midori/appmenu-visibility-test
Merge into: lp:midori
Diff against target: 68 lines (+39/-1)
2 files modified
midori/midori-window.vala (+1/-1)
tests/browser.c (+38/-0)
To merge this branch: bzr merge lp:~gue5t/midori/appmenu-visibility-test
Reviewer Review Type Date Requested Status
Paweł Forysiuk Approve
Review via email: mp+268169@code.launchpad.net

Commit message

Add unit test to check appmenu/menubar visibility

Description of the change

Just add a test for the recent bugfixes

To post a comment you must log in.
Revision history for this message
Paweł Forysiuk (tuxator) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'midori/midori-window.vala'
2--- midori/midori-window.vala 2015-08-10 23:55:02 +0000
3+++ midori/midori-window.vala 2015-08-16 00:19:06 +0000
4@@ -66,7 +66,7 @@
5 public Window () {
6 }
7
8- Gtk.ToolItem? get_tool_item (string name) {
9+ public Gtk.ToolItem? get_tool_item (string name) {
10 /* Name is the empty string if actions has ,, or trailing , */
11 if (name == "")
12 return null;
13
14=== modified file 'tests/browser.c'
15--- tests/browser.c 2014-07-03 08:03:18 +0000
16+++ tests/browser.c 2015-08-16 00:19:06 +0000
17@@ -159,6 +159,43 @@
18 g_object_unref (settings);
19 }
20
21+static void
22+browser_appmenu_visibility (void)
23+{
24+ MidoriApp* app = midori_app_new (NULL);
25+ MidoriBrowser* browser = midori_app_create_browser (app);
26+ GtkToolItem* appmenu = midori_window_get_tool_item (MIDORI_WINDOW (browser), "CompactMenu");
27+ gboolean menubar_visible;
28+ gboolean appmenu_visible;
29+
30+ midori_test_log_set_fatal_handler_for_icons ();
31+
32+ g_object_get (appmenu, "visible", &appmenu_visible, NULL);
33+ g_object_get (browser, "show-menubar", &menubar_visible, NULL);
34+ g_assert (menubar_visible == !appmenu_visible);
35+
36+ g_object_set (browser, "show-menubar", !menubar_visible, NULL);
37+
38+ g_object_get (appmenu, "visible", &appmenu_visible, NULL);
39+ g_object_get (browser, "show-menubar", &menubar_visible, NULL);
40+ g_assert (menubar_visible == !appmenu_visible);
41+
42+ g_object_set (browser, "show-menubar", TRUE, NULL);
43+
44+ g_object_get (appmenu, "visible", &appmenu_visible, NULL);
45+ g_object_get (browser, "show-menubar", &menubar_visible, NULL);
46+ g_assert (menubar_visible && !appmenu_visible);
47+
48+ g_object_set (browser, "show-menubar", FALSE, NULL);
49+
50+ g_object_get (appmenu, "visible", &appmenu_visible, NULL);
51+ g_object_get (browser, "show-menubar", &menubar_visible, NULL);
52+ g_assert (!menubar_visible && appmenu_visible);
53+
54+ gtk_widget_destroy (GTK_WIDGET (browser));
55+ g_object_unref (app);
56+}
57+
58 int
59 main (int argc,
60 char** argv)
61@@ -176,6 +213,7 @@
62 g_test_add_func ("/browser/tooltips", browser_tooltips);
63 g_test_add_func ("/browser/site_data", browser_site_data);
64 g_test_add_func ("/browser/block_uris", browser_block_uris);
65+ g_test_add_func ("/browser/appmenu_visibility", browser_appmenu_visibility);
66
67 return g_test_run ();
68 }

Subscribers

People subscribed via source and target branches

to all changes: