Minor code improvements (#2949)

* Unit test for widgets.HtmlWindow

* Improve unit tests on MacOS

* Unit test for widgets.HtmlWindow

* Improve unit tests on MacOS

* Attempt to cover branch in test_htmlwindow. WIP

* Improve two 100% tests

* Debug test_htmlwnd

* Increase low coverage

* Add fs_watcher test

* Omit non tested files from coverage

* Add DEBUG to Grid. Improve kwedito tests

* Minor fixes in utests

* Improve Grid tests WIP debug failed tests

* Improve RF version detection

* Exclude files from coverage. Grid tests fail in shell not IDE

* Fixes missing context menu in Grid Editor (#2948)

* Minor code improvements

---------

Author: HelioGuilherme66

.coveragerc

@@ -15,6 +15,9 @@ omit =
     ./src/robotide/lib/robot/*
     # ./src/robotide/preferences/configobj/*
     */.venv/*
+    ./src/robotide/__main__.py
+    ./src/robotide/postinstall/__main__.py
+    ./src/robotide/ui/preview.py
 
 [report]
 skip_empty = True

README.adoc

@@ -41,7 +41,7 @@ Likewise, the current version of wxPython, is 4.2.3, but RIDE is known to work w
 
 `pip install -U robotframework-ride`
 
-(3.8 <= python <= 3.13) Install current development version (**2.2dev24**) with:
+(3.8 <= python <= 3.13) Install current development version (**2.2dev25**) with:
 
 `pip install -U https://github.com/robotframework/RIDE/archive/develop.zip`
 

src/robotide/application/CHANGELOG.html

@@ -38,6 +38,8 @@
 </li><li class="listitem">
 Improved colorization for multiple Gherkin words, for example in the French language.
 </li></ul></div></div><div class="section"><div class="titlepage"><div><div><h3 class="title"><a id="_fixed"></a>2.3. Fixed</h3></div></div></div><div class="itemizedlist"><ul class="itemizedlist" style="list-style-type: disc; "><li class="listitem">
+Fixed white blocks on Tree due to failed animation when test execution is too rapid, causing crash on Windows.
+</li><li class="listitem">
 Fixed not set text color on row labels in Grid Editor. Now the General ``secondary foreground`` is applied.
 </li><li class="listitem">
 Fixed multiple scroll bars in Grid Editor when editing Test Cases or Keywords. This caused bad navigation on cells.

src/robotide/application/application.py

@@ -15,6 +15,8 @@
 
 import builtins
 import os
+import subprocess
+
 import wx
 
 from contextlib import contextmanager
@@ -342,14 +344,23 @@ def _find_robot_installation():
         output = run_python_command(
             ['import robot; print(robot.__file__ + \", \" + robot.__version__)'])
         robot_found = b"ModuleNotFoundError" not in output and output
+        system_encoding = get_system_encoding()
+        if not robot_found:
+            rf_info = subprocess.run(['robot', '--version'], capture_output=True)
+            print(f"DEBUG: application-py RIDE _find_robot_installation command VERSION={rf_info.stdout}")
+            if b"Robot Framework" in rf_info.stdout:
+                rf_version = rf_info.stdout.replace(b"Robot Framework", b"").strip(b" ").split(b" ")[0]
+                publish.RideLogMessage(_("Found Robot Framework version %s from %s.") % (
+                    str(rf_version, system_encoding), "PATH")).publish()
+                return rf_version
         if robot_found:
-            system_encoding = get_system_encoding()
             rf_file, rf_version = output.strip().split(b", ")
             publish.RideLogMessage(_("Found Robot Framework version %s from %s.") % (
                 str(rf_version, system_encoding), str(os.path.dirname(rf_file), system_encoding))).publish()
             return rf_version
         else:
             publish.RideLogMessage(publish.get_html_message('no_robot'), notify_user=True).publish()
+            return None
 
     def _get_latest_path(self):
         recent = self._get_recentfiles_plugin()

src/robotide/application/editorprovider.py

@@ -50,7 +50,7 @@ def add(self, editor, default=True):
             self._editors.insert(0, editor)
 
     def set_default(self, editor):
-        if self._editors.index(editor) != -1:
+        if self._editors.index(editor) != len(self._editors) - 1:
             self._editors.remove(editor)
             self._editors.append(editor)
 

src/robotide/application/releasenotes.py

@@ -237,7 +237,7 @@ def set_content(self, html_win, content):
 <pre class="literal-block">python -m robotide.postinstall -install</pre>
 <p>or</p>
 <pre class="literal-block">ride_postinstall.py -install</pre>
-<p>RIDE {VERSION} was released on 03/May/2025.</p>
+<p>RIDE {VERSION} was released on 15/May/2025.</p>
 <!-- <br/>
 <h3>May The Fourth Be With You!</h3>
 <h3>Celebrate the bank holiday, 10th June, Day of Portugal, Portuguese Communities and Camões!!</h3>

src/robotide/editor/editors.py

@@ -60,7 +60,7 @@ class EditorPanel(wx.lib.scrolledpanel.ScrolledPanel):
         = show_content_assist = lambda self: None
 
     def __init__(self, plugin, parent, controller, tree):
-        wx.lib.scrolledpanel.ScrolledPanel.__init__(self, parent)
+        wx.lib.scrolledpanel.ScrolledPanel.__init__(self, parent, style=wx.HSCROLL|wx.VSCROLL|wx.ALWAYS_SHOW_SB)
         from ..preferences import RideSettings
         _settings = RideSettings()
         self.general_settings = _settings['General']
@@ -312,6 +312,7 @@ class _FileEditor(_RobotTableEditor):
 
     def __init__(self, *args):
         _RobotTableEditor.__init__(self, *args)
+        # DEBUG
         self.SetupScrolling()
         self.plugin.subscribe(
             self._update_source_and_name, RideFileNameChanged)

src/robotide/editor/gridbase.py

@@ -33,7 +33,7 @@ class GridEditor(grid.Grid):
     _popup_items_nt = []
 
     def __init__(self, parent, num_rows, num_cols, popup_creator=None):
-        grid.Grid.__init__(self, parent)
+        grid.Grid.__init__(self, parent.frame if isinstance(parent, wx.App) else parent)
         try:
             self.settings = parent.plugin.global_settings['Grid']
             self.general_settings = parent.plugin.global_settings['General']

src/robotide/namespace/cache.py

@@ -184,12 +184,14 @@ def _build_default_kws(self):
         from robotide.context import APP
         try:
             robot_version = APP.robot_version
-        except AttributeError:
+        except (AttributeError, TypeError):
+            robot_version = b'7.0.1'
+        if not robot_version:
             robot_version = b'7.0.1'
         try:
             rbt_version = int(str(robot_version, encoding='UTF-8').replace('.', ''))
             rbt_version = rbt_version if rbt_version > 99 else rbt_version * 10
-        except ValueError:
+        except (ValueError, TypeError):
             rbt_version = 701
         var_note = "*NOTE:* This marker exists since version 7.0" + (f" and is an error to use because your "
                                                                      f"version of Robot Framework is"