Replace os.getenv('USER') with getpass.getuser() for better cross-platform compatibility

Use getpass.getuser() instead of os.getenv('USER') in SSH tests to ensure
compatibility with GitHub Actions and other CI environments where USER
environment variable may not be set.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: yannrichet

tests/conftest.py

@@ -7,6 +7,7 @@
 import pytest
 import socket
 import subprocess
+import getpass
 from pathlib import Path
 
 
@@ -66,7 +67,7 @@ def is_ssh_server_available():
                 "-o", "StrictHostKeyChecking=no",
                 "-o", "UserKnownHostsFile=/dev/null",
                 "-o", "LogLevel=ERROR",
-                f"{os.getenv('USER')}@localhost",
+                f"{getpass.getuser()}@localhost",
                 "exit 0"
             ],
             capture_output=True,

tests/test_ssh_many_cases.py

@@ -14,6 +14,7 @@
 import time
 from pathlib import Path
 import pytest
+import getpass
 from conftest import SSH_AVAILABLE
 
 # Check if paramiko is available
@@ -128,7 +129,7 @@ def test_ssh_many_cases_localhost():
                 "-o", "UserKnownHostsFile=/dev/null",
                 "-o", "ConnectTimeout=5",
                 "-o", "LogLevel=ERROR",
-                f"{os.getenv('USER')}@localhost",
+                f"{getpass.getuser()}@localhost",
                 "echo 'SSH OK'"
             ],
             capture_output=True,
@@ -193,7 +194,7 @@ def test_ssh_many_cases_localhost():
         from fz import fzr
 
         # Build SSH calculator URL
-        ssh_calculator = f"ssh://{os.getenv('USER')}@localhost/bash {calc_script.resolve()}"
+        ssh_calculator = f"ssh://{getpass.getuser()}@localhost/bash {calc_script.resolve()}"
 
         print(f"   Calculator: {ssh_calculator}")
 
@@ -455,7 +456,7 @@ def test_ssh_many_cases_many_localhost():
                 "-o", "UserKnownHostsFile=/dev/null",
                 "-o", "ConnectTimeout=5",
                 "-o", "LogLevel=ERROR",
-                f"{os.getenv('USER')}@localhost",
+                f"{getpass.getuser()}@localhost",
                 "echo 'SSH OK'"
             ],
             capture_output=True,
@@ -520,7 +521,7 @@ def test_ssh_many_cases_many_localhost():
         from fz import fzr
 
         # Build SSH calculator URL
-        ssh_calculator = [f"ssh://{os.getenv('USER')}@localhost/bash {calc_script.resolve()}"]*3
+        ssh_calculator = [f"ssh://{getpass.getuser()}@localhost/bash {calc_script.resolve()}"]*3
 
         print(f"   Calculators: {ssh_calculator}")
 

tests/test_ssh_perfectgaz.py

@@ -21,6 +21,7 @@
 import time
 from pathlib import Path
 import pytest
+import getpass
 from fz import fzr
 from conftest import SSH_AVAILABLE
 
@@ -151,7 +152,7 @@ def test_perfectgaz_via_ssh_localhost():
                 "-o", "UserKnownHostsFile=/dev/null",
                 "-o", "ConnectTimeout=5",
                 "-o", "LogLevel=ERROR",
-                f"{os.getenv('USER')}@localhost",
+                f"{getpass.getuser()}@localhost",
                 "echo 'SSH OK'"
             ],
             capture_output=True,
@@ -212,7 +213,7 @@ def test_perfectgaz_via_ssh_localhost():
         print("\n6. Running fzr with SSH calculator...")
 
         # Build SSH calculator URL
-        ssh_calculator = f"ssh://{os.getenv('USER')}@localhost/bash " + str(calc_script.resolve()) #./PerfectGazPressure.sh"
+        ssh_calculator = f"ssh://{getpass.getuser()}@localhost/bash " + str(calc_script.resolve()) #./PerfectGazPressure.sh"
 
         print(f"Calculator: {ssh_calculator}")