Update tests on Windows

.github/workflows/ci.yml

@@ -144,22 +144,6 @@ jobs:
       instances: '["stable-3006", "stable-3006-8", "stable-3007", "stable-3007-1", "latest"]'
 
 
-
-  windows-2019:
-    name: Windows 2019
-    if: github.event_name == 'push' || needs.collect-changed-files.outputs.run-tests == 'true'
-    uses: ./.github/workflows/test-windows.yml
-    needs:
-      - lint
-      - generate-actions-workflow
-    with:
-      distro-slug: windows-2019
-      display-name: Windows 2019
-      timeout: 20
-      runs-on: windows-2019
-      instances: '["stable-3006", "stable-3006-8", "stable-3007", "stable-3007-1", "latest"]'
-
-
   windows-2022:
     name: Windows 2022
     if: github.event_name == 'push' || needs.collect-changed-files.outputs.run-tests == 'true'
@@ -480,7 +464,6 @@ jobs:
       - generate-actions-workflow
       - macos-12
       - macos-13
-      - windows-2019
       - windows-2022
       - almalinux-8
       - almalinux-9

.github/workflows/templates/generate.py

@@ -32,7 +32,6 @@
     "ubuntu-2404",
 ]
 WINDOWS = [
-    "windows-2019",
     "windows-2022",
 ]
 
@@ -265,7 +264,6 @@
     "ubuntu-2404": "Ubuntu 24.04",
     "macos-12": "macOS 12",
     "macos-13": "macOS 13",
-    "windows-2019": "Windows 2019",
     "windows-2022": "Windows 2022",
 }
 

.github/workflows/test-windows.yml

@@ -41,7 +41,6 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-
       - name: Set up Python 3.10
         uses: actions/setup-python@v5
         with:

tests/integration/test_installation.py

@@ -1,12 +1,11 @@
 import json
 import logging
+import os
 import platform
 import subprocess
 
 import pytest
 
-## from Demos.win32cred_demo import target
-
 log = logging.getLogger(__name__)
 
 
@@ -26,28 +25,28 @@ def run_salt_call(cmd):
     """
     cmd.append("--out=json")
     result = subprocess.run(cmd, capture_output=True, text=True)
-    if result.stdout:
-        json_data = json.loads(result.stdout)
-        return json_data["local"]
-    return None
+    json_data = json.loads(result.stdout)
+    return json_data["local"]
 
 
-def test_ping():
+def test_ping(path):
     cmd = ["salt-call", "--local", "test.ping"]
     result = run_salt_call(cmd)
     assert result == True
 
 
-def test_target_python_version(target_python_version):
+def test_target_python_version(path, target_python_version):
     cmd = ["salt-call", "--local", "grains.item", "pythonversion", "--timeout=120"]
     result = run_salt_call(cmd)
     # Returns: {'pythonversion': [3, 10, 11, 'final', 0]}
     py_maj_ver = result["pythonversion"][0]
     assert py_maj_ver == target_python_version
 
 
-def test_target_salt_version(target_salt_version):
+def test_target_salt_version(path, target_salt_version):
+    if not target_salt_version:
+        pytest.skip(f"No target version specified")
     cmd = ["salt-call", "--local", "grains.item", "saltversion", "--timeout=120"]
     result = run_salt_call(cmd)
     # Returns: {'saltversion': '3006.9+217.g53cfa53040'}
-    assert result["saltversion"] == target_salt_version
+    assert result["saltversion"] == target_salt_version

tests/requirements.txt

@@ -1 +1,2 @@
 pytest
+requests