Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update pytest.mark.network markers #9212

Merged
merged 1 commit into from
Mar 23, 2024
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
update pytest.mark.network markers
dimbleby committed Mar 23, 2024
commit 0d5de1c3c55c7ae1152d169d33f375f5e32592ad
4 changes: 0 additions & 4 deletions tests/inspection/test_info.py
Original file line number Diff line number Diff line change
@@ -315,7 +315,6 @@ def test_info_setup_simple(mocker: MockerFixture, demo_setup: Path) -> None:
demo_check_info(info, requires_dist={"package"})


@pytest.mark.network
def test_info_setup_complex(demo_setup_complex: Path) -> None:
info = PackageInfo.from_directory(demo_setup_complex)
demo_check_info(info, requires_dist={"package"})
@@ -334,22 +333,19 @@ def test_info_setup_complex_pep517_error(
PackageInfo.from_directory(demo_setup_complex)


@pytest.mark.network
def test_info_setup_complex_pep517_legacy(
demo_setup_complex_pep517_legacy: Path,
) -> None:
info = PackageInfo.from_directory(demo_setup_complex_pep517_legacy)
demo_check_info(info, requires_dist={"package"})


@pytest.mark.network
def test_info_setup_complex_calls_script(demo_setup_complex_calls_script: Path) -> None:
"""Building the project requires calling a script from its build_requires."""
info = PackageInfo.from_directory(demo_setup_complex_calls_script)
demo_check_info(info, requires_dist={"package"})


@pytest.mark.network
@pytest.mark.parametrize("missing", ["version", "name"])
def test_info_setup_missing_mandatory_should_trigger_pep517(
mocker: MockerFixture, source_dir: Path, missing: str
2 changes: 0 additions & 2 deletions tests/installation/test_chef.py
Original file line number Diff line number Diff line change
@@ -80,7 +80,6 @@ def test_prepare_directory(
os.unlink(wheel)


@pytest.mark.network
def test_prepare_directory_with_extensions(
config: Config,
config_cache_dir: Path,
@@ -126,7 +125,6 @@ def test_prepare_directory_editable(
os.unlink(wheel)


@pytest.mark.network
def test_prepare_directory_script(
config: Config,
config_cache_dir: Path,
5 changes: 0 additions & 5 deletions tests/installation/test_executor.py
Original file line number Diff line number Diff line change
@@ -1547,11 +1547,6 @@ def test_executor_known_hashes(
io: BufferedIO,
fixture_dir: FixtureDirGetter,
) -> None:
# when installing sdist, an isolated build environment is required to extract metadata
# this will install any build system requirements into the environment, to avoid failures when
# network is not available we enable mock_file_downloads fixture here
# see: https://github.com/python-poetry/poetry/issues/9114

package_source_url: Path = (
fixture_dir("distributions") / "demo-0.1.0.tar.gz"
).resolve()