chore: make tests running on windows.

- update decode of `executor`.
- skip some test case that not support on windows.
- adjust test case compatible with windows.

pigit/ext/executor.py

@@ -31,6 +31,20 @@
 SILENT: Final = 1 << 5  # silent mode. output will be discarded.
 
 
+def _detect_encoding(data:ByteString)->str:
+    encodings = ['utf-8', 'gbk','latin-1','iso-8859-1']
+
+    for encoding in encodings:
+        try:
+            #  TODO: It may be possible to decode, but the result is not correct.
+            data.decode(encoding)
+            return encoding
+        except UnicodeDecodeError:
+            continue
+
+    return  ''
+
+
 @dataclasses.dataclass
 class ExecState:
     """State ctx of ~Executor."""
@@ -115,7 +129,11 @@ def _try_decode(self, content: ExecResType, state: "ExecState") -> ExecResType:
             The decoded output if decoding is enabled, otherwise the original output.
         """
         if state.decoding and content is not None and not isinstance(content, str):
-            return content.decode()
+            try:
+                return content.decode()
+            except UnicodeDecodeError:
+                # Default encoding may not is 'utf-8' on windows.
+                return content.decode(_detect_encoding(content))
         else:
             return content
 

requirements.txt

@@ -4,6 +4,7 @@ pylint
 pytest
 pytest-mock
 pytest-cov
+pytest-asyncio
 pyinstrument
 requests
 setuptools

tests/conftest.py

@@ -8,4 +8,15 @@
 
 TEST_PATH = os.path.dirname(__file__)
 
+# Not support.
+if sys.platform == "win32":
+    collect_ignore_glob = ["test_tui_input.py", "test_tui_eventloop.py"]
+
+PYTHON_VERSION = sys.version_info[:3]
+if PYTHON_VERSION < (3, 8, 5):
+    raise Exception(
+        "The current version of pigit does not support less than (Python3.8)."
+    )
+
+
 # /opt/homebrew/opt/python@3.8/bin/python3 -m pytest

tests/test_ext_executor.py

@@ -1,3 +1,4 @@
+import sys
 import time
 import textwrap
 import pytest
@@ -13,6 +14,8 @@
     Executor,
 )
 
+win_skip_mark = pytest.mark.skipif(sys.platform == "win32", reason="windows skip.")
+
 
 class TestExecutor:
     @classmethod
@@ -58,6 +61,7 @@ def test_exec(self, cmd, flags, kws, expected):
             assert result == expected
             # mock_popen.assert_called_once_with(args=cmd, **kws)
 
+    @win_skip_mark
     def test_exec_with_more(self):
         print()
 
@@ -111,13 +115,17 @@ def test_exec_parallel(self):
             if __name__ == '__main__':
                 import time
 
-                print({0})
+                print({0}, end='')
                 time.sleep(int({0}))
-                print({0})
+                print({0}, end='')
             """
         )
 
-        cmds = [["python3", "-c", code.format(i)] for i in range(3, 0, -1)]
+        cmd = 'python3'
+        if self.executor.exec('python -V', flags=REPLY)[0] == 0 :
+            cmd ='python'
+
+        cmds = [[cmd, "-c", code.format(i)] for i in range(3, 0, -1)]
         # pprint(cmds)
 
         start_t = time.time()
@@ -129,4 +137,4 @@ def test_exec_parallel(self):
         assert end_t - start_t < 4
 
         results2 = self.executor.exec_parallel(*cmds, flags=REPLY | DECODE)
-        assert results2 == [(0, "", "{0}\n{0}\n".format(i)) for i in range(3, 0, -1)]
+        assert results2 == [(0, "", "{0}{0}".format(i)) for i in range(3, 0, -1)]

tests/test_ext_lcstat.py

@@ -1,16 +1,20 @@
 from typing import Optional
 import os
 import time
+from pathlib import Path
+
 import pytest
 
+from pigit.const import USER_HOME
 from pigit.ext.lcstat import Counter
 
 
 @pytest.mark.parametrize(
     "path",
     [
-        # os.getcwd(),
-        os.environ["HOME"] + "/.config",
+        os.getcwd(),
+        Path(USER_HOME, ".config")
+        # USER_HOME + "/.config",
         # *os.environ.values(),
     ],
 )

tests/test_git.py

@@ -1,4 +1,5 @@
 import os
+import stat
 import shutil
 import pytest
 from unittest.mock import patch
@@ -18,7 +19,15 @@ def create_repo(test_repo: str):
 
     # re-create test repo
     if os.path.isdir(test_repo):
-        shutil.rmtree(test_repo)
+        shutil.rmtree(
+            test_repo,
+            onerror=lambda func, path, _: (
+                # Fix `PermissionError` on windows.
+                # Will deprecated `onerror` on Py312.
+                os.chmod(path, stat.S_IWRITE),
+                func(path),
+            ),
+        )
     os.makedirs(test_repo, exist_ok=True)
     print(executor.exec("git init", flags=WAITING, cwd=test_repo))
     print(executor.exec("git config user.name Zachary", flags=WAITING, cwd=test_repo))