Skip to content

Commit e5eb514

Browse files
committed
fix testcase, raise exception on failure
1 parent c8ab7a0 commit e5eb514

File tree

2 files changed

+2
-7
lines changed

2 files changed

+2
-7
lines changed

src/poetry/packages/locker.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -254,10 +254,7 @@ def __walk_dependencies(
254254
)
255255

256256
if not locked_package:
257-
# Should normally be able to satisfy all requirements, but this case is
258-
# permissible eg if we encounter a dev dependency when walking the
259-
# non-dev dependencies.
260-
continue
257+
raise RuntimeError(f"Dependency walk failed at {requirement}")
261258

262259
# create dependency from locked package to retain dependency metadata
263260
# if this is not done, we can end-up with incorrect nested dependencies

tests/utils/test_exporter.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1657,8 +1657,6 @@ def test_exporter_exports_requirements_txt_with_legacy_packages_trusted_host(
16571657
(
16581658
False,
16591659
[
1660-
'bar==1.2.2 ; python_version >= "2.7" and python_version < "2.8" or'
1661-
' python_version >= "3.6" and python_version < "4.0"',
16621660
'foo==1.2.1 ; python_version >= "2.7" and python_version < "2.8" or'
16631661
' python_version >= "3.6" and python_version < "4.0"',
16641662
],
@@ -1681,7 +1679,7 @@ def test_exporter_exports_requirements_txt_with_dev_extras(
16811679
{
16821680
"name": "bar",
16831681
"version": "1.2.2",
1684-
"category": "main",
1682+
"category": "dev",
16851683
"optional": False,
16861684
"python-versions": "*",
16871685
"dependencies": {

0 commit comments

Comments
 (0)