enable test_unicode on windows

Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
RustPython · youknowone · Feb 24, 2025 · Feb 23, 2025 · Feb 24, 2025 · Feb 24, 2025
commit f97fbc3ece94ccb05b83feb93d4851e42e26088e
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -23,7 +23,6 @@ env:
  # test_os: many failing tests
  # test_pathlib: support.rmtree() failing
  # test_posixpath: OSError: (22, 'The filename, directory name, or volume label syntax is incorrect. (os error 123)')
-  # test_unicode: AttributeError: module '_winapi' has no attribute 'GetACP'
  # test_venv: couple of failing tests
  WINDOWS_SKIPS: >-
    test_argparse
@@ -33,7 +32,6 @@ env:
    test_rlcompleter
    test_pathlib
    test_posixpath
-    test_unicode
    test_venv
  # configparser: https://github.com/RustPython/RustPython/issues/4995#issuecomment-1582397417
  # socketserver: seems related to configparser crash.

diff --git a/vm/Cargo.toml b/vm/Cargo.toml
@@ -125,6 +125,7 @@ features = [
 workspace = true
 features = [
  "Win32_Foundation",
+  "Win32_Globalization",
  "Win32_Networking_WinSock",
  "Win32_Security",
  "Win32_Storage_FileSystem",

diff --git a/vm/src/stdlib/winapi.rs b/vm/src/stdlib/winapi.rs
@@ -119,6 +119,13 @@ mod _winapi {
        Ok(HANDLE(target))
    }

+    #[pyfunction]
+    fn GetACP() -> u32 {
+        unsafe {
+            windows_sys::Win32::Globalization::GetACP()
+        }
+    }
+
    #[pyfunction]
    fn GetCurrentProcess() -> HANDLE {
        unsafe { windows::Win32::System::Threading::GetCurrentProcess() }