Skip to content

Commit 60ac406

Browse files
committed
Update creation of test package that must fail building
1 parent e4d9360 commit 60ac406

File tree

2 files changed

+8
-14
lines changed

2 files changed

+8
-14
lines changed

tests/functional/test_new_resolver.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2463,7 +2463,7 @@ def test_new_resolver_constraint_on_link_with_extra_indirect(
24632463
def test_new_resolver_do_not_backtrack_on_build_failure(
24642464
script: PipTestEnvironment,
24652465
) -> None:
2466-
create_basic_sdist_for_package(script, "pkg1", "2.0", fails_egg_info=True)
2466+
create_basic_sdist_for_package(script, "pkg1", "2.0", fails_build=True)
24672467
create_basic_wheel_for_package(script, "pkg1", "1.0")
24682468

24692469
result = script.pip(
@@ -2476,14 +2476,14 @@ def test_new_resolver_do_not_backtrack_on_build_failure(
24762476
expect_error=True,
24772477
)
24782478

2479-
assert "egg_info" in result.stderr
2479+
assert "Failed to build 'pkg1'" in result.stderr
24802480

24812481

24822482
def test_new_resolver_works_when_failing_package_builds_are_disallowed(
24832483
script: PipTestEnvironment,
24842484
) -> None:
24852485
create_basic_wheel_for_package(script, "pkg2", "1.0", depends=["pkg1"])
2486-
create_basic_sdist_for_package(script, "pkg1", "2.0", fails_egg_info=True)
2486+
create_basic_sdist_for_package(script, "pkg1", "2.0", fails_build=True)
24872487
create_basic_wheel_for_package(script, "pkg1", "1.0")
24882488
constraints_file = script.scratch_path / "constraints.txt"
24892489
constraints_file.write_text("pkg1 != 2.0")

tests/lib/__init__.py

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1213,8 +1213,7 @@ def create_basic_sdist_for_package(
12131213
version: str,
12141214
extra_files: dict[str, str] | None = None,
12151215
*,
1216-
fails_egg_info: bool = False,
1217-
fails_bdist_wheel: bool = False,
1216+
fails_build: bool = False,
12181217
depends: list[str] | None = None,
12191218
setup_py_prelude: str = "",
12201219
) -> pathlib.Path:
@@ -1226,14 +1225,10 @@ def create_basic_sdist_for_package(
12261225
12271226
{setup_py_prelude}
12281227
1229-
fails_bdist_wheel = {fails_bdist_wheel!r}
1230-
fails_egg_info = {fails_egg_info!r}
1228+
fails_build = {fails_build!r}
12311229
1232-
if fails_egg_info and "egg_info" in sys.argv:
1233-
raise Exception("Simulated failure for generating metadata.")
1234-
1235-
if fails_bdist_wheel and "bdist_wheel" in sys.argv:
1236-
raise Exception("Simulated failure for building a wheel.")
1230+
if fails_build:
1231+
raise Exception("Simulated build failure.")
12371232
12381233
setup(name={name!r}, version={version!r},
12391234
install_requires={depends!r})
@@ -1243,8 +1238,7 @@ def create_basic_sdist_for_package(
12431238
version=version,
12441239
depends=depends or [],
12451240
setup_py_prelude=setup_py_prelude,
1246-
fails_bdist_wheel=fails_bdist_wheel,
1247-
fails_egg_info=fails_egg_info,
1241+
fails_build=fails_build,
12481242
),
12491243
}
12501244

0 commit comments

Comments
 (0)