skip venv zipapp test on windows, venv not supported on windows yet

Author: rickeylev

tests/py_zipapp/BUILD.bazel

@@ -17,14 +17,14 @@ py_binary(
         "//python/config_settings:venvs_site_packages": "yes",
     },
     main = "main.py",
-    ##target_compatible_with = NOT_WINDOWS,
+    target_compatible_with = NOT_WINDOWS,
     deps = [":some_dep"],
 )
 
 py_zipapp_binary(
     name = "venv_zipapp",
     binary = ":venv_bin",
-    ##target_compatible_with = NOT_WINDOWS,
+    target_compatible_with = NOT_WINDOWS,
 )
 
 py_test(
@@ -33,10 +33,9 @@ py_test(
     data = [":venv_zipapp"],
     env = {
         "BZLMOD_ENABLED": str(int(BZLMOD_ENABLED)),
-        # todo: for windows, the output is (launcher.exe, zipapp.pyz)
         "TEST_ZIPAPP": "$(location :venv_zipapp)",
     },
-    ##target_compatible_with = NOT_WINDOWS,
+    target_compatible_with = NOT_WINDOWS,
 )
 
 py_binary(

tests/py_zipapp/venv_zipapp_test.py

@@ -60,13 +60,13 @@ def _is_bzlmod_enabled(self):
 
     @contextlib.contextmanager
     def _open_zipapp(self, path):
-        # On windows, the main output is the launcher .exe file, and the zip
-        # file is a sibling file.
         zf = None
         try:
             try:
                 zf = zipfile.ZipFile(path, "r")
             except zipfile.BadZipFile:
+                # On windows, the main output is the launcher .exe file, and the
+                # zip file is a sibling file.
                 path = path.replace(".exe", ".zip")
                 zf = zipfile.ZipFile(path, "r")
             if zf: