Skip to content

Commit

Permalink
Fix mac and windows tests
Browse files Browse the repository at this point in the history
  • Loading branch information
dwoz committed Mar 25, 2024
1 parent a3bebc7 commit 90b6fa4
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions tests/test_verify_build.py
Original file line number Diff line number Diff line change
Expand Up @@ -87,9 +87,11 @@ def test_imports(pyexec):
assert p.returncode == 0, f"Failed to import {mod}"


def test_pip_install_salt_git(pipexec, build, build_dir, pyexec):
if sys.platform == "win32" and get_build_version().startswith("3.11"):
pytest.xfail("Salt does not work with 3.11 on windows yet")
def test_pip_install_salt_git(pipexec, build, build_dir, pyexec, build_version):
if sys.platform == "win32" and "3.11" in build_version or "3.12" in build_version:
pytest.xfail("Salt does not work with 3.11 or 3.12 on windows yet")
if sys.platform == "darwin" and "3.12" in build_version:
pytest.xfail("Salt does not work with 3.12 on macos yet")
env = os.environ.copy()
env["RELENV_BUILDENV"] = "yes"
if sys.platform == "linux" and shutil.which("git"):
Expand Down Expand Up @@ -919,7 +921,7 @@ def test_install_with_target_cffi_versions(pipexec, pyexec, build):
env=env,
)
subprocess.run(
[str(pipexec), "install", "cffi==1.15.1", f"--target={extras}"],
[str(pipexec), "install", "cffi==1.16.0", f"--target={extras}"],
check=True,
env=env,
)
Expand All @@ -930,7 +932,7 @@ def test_install_with_target_cffi_versions(pipexec, pyexec, build):
env=env,
capture_output=True,
)
proc.stdout.decode().strip() == "1.15.1"
proc.stdout.decode().strip() == "1.16.0"


def test_install_with_target_no_ignore_installed(pipexec, pyexec, build):
Expand Down

0 comments on commit 90b6fa4

Please sign in to comment.