Skip to content

Commit a618b45

Browse files
committed
fix testcase, raise exception on failure
1 parent d76d1ce commit a618b45

2 files changed

Lines changed: 2 additions & 7 deletions

File tree

src/poetry/packages/locker.py

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

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

263260
# create dependency from locked package to retain dependency metadata
264261
# 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
@@ -1664,8 +1664,6 @@ def test_exporter_exports_requirements_txt_with_legacy_packages_trusted_host(
16641664
(
16651665
False,
16661666
[
1667-
'bar==1.2.2 ; python_version >= "2.7" and python_version < "2.8" or'
1668-
' python_version >= "3.6" and python_version < "4.0"',
16691667
'foo==1.2.1 ; python_version >= "2.7" and python_version < "2.8" or'
16701668
' python_version >= "3.6" and python_version < "4.0"',
16711669
],
@@ -1688,7 +1686,7 @@ def test_exporter_exports_requirements_txt_with_dev_extras(
16881686
{
16891687
"name": "bar",
16901688
"version": "1.2.2",
1691-
"category": "main",
1689+
"category": "dev",
16921690
"optional": False,
16931691
"python-versions": "*",
16941692
"dependencies": {

0 commit comments

Comments
 (0)