Skip to content

Commit e5b4fdb

Browse files
authored
Merge pull request #41 from DavidCEllis/fix_broken_test
Fix a test that was broken due to a change in ducktools-pythonfinder
2 parents 71ca97b + 806aeb0 commit e5b4fdb

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ dependencies = [
1717
"ducktools-classbuilder>=0.7.1",
1818
"ducktools-lazyimporter>=0.7.0",
1919
"ducktools-scriptmetadata",
20-
"ducktools-pythonfinder>=0.6.0",
20+
"ducktools-pythonfinder>=0.8.1",
2121
"tomli; python_version < '3.11'",
2222
"packaging>=23.2",
2323
]

tests/conftest.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
import tempfile
2020

2121
from ducktools.pythonfinder import get_python_installs
22-
from ducktools.pythonfinder.shared import get_install_details
22+
from ducktools.pythonfinder.shared import DetailFinder
2323

2424
from ducktools.env.catalogue import TemporaryCatalogue
2525
from ducktools.env.config import Config
@@ -38,7 +38,8 @@ def available_pythons():
3838
@pytest.fixture(scope="session")
3939
def this_python():
4040
py = sys.executable
41-
details = get_install_details(py)
41+
finder = DetailFinder()
42+
details = finder.get_install_details(py)
4243
# Pretend PyPy is CPython for tests
4344
if details.implementation == "pypy":
4445
details.implementation = "cpython"

0 commit comments

Comments
 (0)