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
=== modified file 'midori/midori-window.vala'
--- midori/midori-window.vala 2015-08-10 23:55:02 +0000
+++ midori/midori-window.vala 2015-08-16 00:19:06 +0000
@@ -66,7 +66,7 @@
66 public Window () {66 public Window () {
67 }67 }
6868
69 Gtk.ToolItem? get_tool_item (string name) {69 public Gtk.ToolItem? get_tool_item (string name) {
70 /* Name is the empty string if actions has ,, or trailing , */70 /* Name is the empty string if actions has ,, or trailing , */
71 if (name == "")71 if (name == "")
72 return null;72 return null;
7373
=== modified file 'tests/browser.c'
--- tests/browser.c 2014-07-03 08:03:18 +0000
+++ tests/browser.c 2015-08-16 00:19:06 +0000
@@ -159,6 +159,43 @@
159 g_object_unref (settings);159 g_object_unref (settings);
160}160}
161161
162static void
163browser_appmenu_visibility (void)
164{
165 MidoriApp* app = midori_app_new (NULL);
166 MidoriBrowser* browser = midori_app_create_browser (app);
167 GtkToolItem* appmenu = midori_window_get_tool_item (MIDORI_WINDOW (browser), "CompactMenu");
168 gboolean menubar_visible;
169 gboolean appmenu_visible;
170
171 midori_test_log_set_fatal_handler_for_icons ();
172
173 g_object_get (appmenu, "visible", &appmenu_visible, NULL);
174 g_object_get (browser, "show-menubar", &menubar_visible, NULL);
175 g_assert (menubar_visible == !appmenu_visible);
176
177 g_object_set (browser, "show-menubar", !menubar_visible, NULL);
178
179 g_object_get (appmenu, "visible", &appmenu_visible, NULL);
180 g_object_get (browser, "show-menubar", &menubar_visible, NULL);
181 g_assert (menubar_visible == !appmenu_visible);
182
183 g_object_set (browser, "show-menubar", TRUE, NULL);
184
185 g_object_get (appmenu, "visible", &appmenu_visible, NULL);
186 g_object_get (browser, "show-menubar", &menubar_visible, NULL);
187 g_assert (menubar_visible && !appmenu_visible);
188
189 g_object_set (browser, "show-menubar", FALSE, NULL);
190
191 g_object_get (appmenu, "visible", &appmenu_visible, NULL);
192 g_object_get (browser, "show-menubar", &menubar_visible, NULL);
193 g_assert (!menubar_visible && appmenu_visible);
194
195 gtk_widget_destroy (GTK_WIDGET (browser));
196 g_object_unref (app);
197}
198
162int199int
163main (int argc,200main (int argc,
164 char** argv)201 char** argv)
@@ -176,6 +213,7 @@
176 g_test_add_func ("/browser/tooltips", browser_tooltips);213 g_test_add_func ("/browser/tooltips", browser_tooltips);
177 g_test_add_func ("/browser/site_data", browser_site_data);214 g_test_add_func ("/browser/site_data", browser_site_data);
178 g_test_add_func ("/browser/block_uris", browser_block_uris);215 g_test_add_func ("/browser/block_uris", browser_block_uris);
216 g_test_add_func ("/browser/appmenu_visibility", browser_appmenu_visibility);
179217
180 return g_test_run ();218 return g_test_run ();
181}219}

Subscribers

People subscribed via source and target branches

to all changes: