Make hide() work same as toggle()

This also fixes unreliable tests.

Author: hluk

src/scriptable/scriptableproxy.cpp

@@ -1073,7 +1073,7 @@ void ScriptableProxy::exit()
 void ScriptableProxy::close()
 {
     INVOKE2(close, ());
-    m_wnd->close();
+    m_wnd->hideWindow();
 }
 
 bool ScriptableProxy::focusPrevious()

src/tests/tests.cpp

@@ -430,6 +430,8 @@ class TestInterfaceImpl final : public TestInterface {
             settings.beginGroup("Options");
             settings.setValue( Config::clipboard_tab::name(), clipboardTabName );
             settings.setValue( Config::close_on_unfocus::name(), false );
+            // Hide the main window even if there is no tray or minimize support.
+            settings.setValue( Config::hide_main_window::name(), true );
             // Exercise limiting rows in Process Manager dialog when testing.
             settings.setValue( Config::max_process_manager_rows::name(), 4 );
             settings.endGroup();

src/tests/tests.h

@@ -48,8 +48,7 @@ private slots:
 
     void commandVisible();
     void commandToggle();
-    void commandHide();
-    void commandShow();
+    void commandShowHide();
     void commandShowAt();
     void commandFocused();
 

src/tests/tests_scripts.cpp

@@ -137,14 +137,7 @@ void Tests::commandToggle()
     WAIT_ON_OUTPUT("visible", "true\n");
 }
 
-void Tests::commandHide()
-{
-    RUN("visible", "true\n");
-    RUN("hide", "");
-    WAIT_ON_OUTPUT("visible", "false\n");
-}
-
-void Tests::commandShow()
+void Tests::commandShowHide()
 {
     RUN("visible", "true\n");
     RUN("hide", "");