Address code review feedback: improve test assertions and Windows compatibility

Co-authored-by: willtheorangeguy <[email protected]>

Author: Copilot

.github/workflows/tests.yml

@@ -36,8 +36,19 @@ jobs:
       run: |
         python -m pip install --upgrade pip
         pip install pytest pytest-cov
+      shell: bash
+
+    - name: Install requirements if present
+      run: |
         if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
       shell: bash
+      if: runner.os != 'Windows'
+
+    - name: Install requirements if present (Windows)
+      run: |
+        if (Test-Path requirements.txt) { pip install -r requirements.txt }
+      shell: pwsh
+      if: runner.os == 'Windows'
 
     - name: Run tests (Linux)
       if: runner.os == 'Linux'

tests/test_main.py

@@ -7,7 +7,9 @@
 the Free Software Foundation, version 3 of the License.
 """
 
-# pylint: disable=import-error, invalid-name, unused-argument, wrong-import-position, import-outside-toplevel
+# pylint: disable=import-error, invalid-name, wrong-import-position, import-outside-toplevel, unused-argument
+# unused-argument is disabled because @patch decorators inject mocked objects as parameters
+# even when not all mocks are used in every test
 
 import unittest
 from unittest.mock import Mock, patch, mock_open
@@ -157,7 +159,8 @@ def test_programver_creates_window(
         mock_tk.assert_called_once()
         # Verify window title was set
         mock_window.title.assert_called_once()
-        assert "ProgramVer" in str(mock_window.title.call_args)
+        title_text = mock_window.title.call_args[0][0]
+        assert "ProgramVer" in title_text
 
     @patch("main.Tk")
     @patch("main.PhotoImage")
@@ -195,9 +198,8 @@ def test_programver_creates_labels(
 
         ProgramVer()
 
-        # Verify Label was called multiple times
+        # Verify Label was called multiple times to create all labels
         assert mock_label.call_count >= 5
-        # Verify labels were created with window
 
     @patch("main.Tk")
     @patch("main.PhotoImage")