raise InvalidMarker for invalid marker

Author: dimbleby

src/poetry/core/version/markers.py

@@ -12,6 +12,7 @@
 from typing import Iterable
 
 from poetry.core.constraints.version import VersionConstraint
+from poetry.core.constraints.version.exceptions import ParseConstraintError
 from poetry.core.version.grammars import GRAMMAR_PEP_508_MARKERS
 from poetry.core.version.parser import Parser
 
@@ -212,12 +213,12 @@ def __init__(
         self._constraint: BaseConstraint | VersionConstraint
         self._parser: Callable[[str], BaseConstraint | VersionConstraint]
         self._name = ALIASES.get(name, name)
-        constraint_string = str(constraint)
+        original_constraint_string = constraint_string = str(constraint)
 
         # Extract operator and value
         m = self._CONSTRAINT_RE.match(constraint_string)
         if m is None:
-            raise InvalidMarker(f"Invalid marker '{constraint_string}'")
+            raise InvalidMarker(f"Invalid marker: {constraint_string}")
 
         self._operator = m.group(1)
         if self._operator is None:
@@ -245,9 +246,7 @@ def __init__(
                 if self._operator == "in":
                     glue = " || "
 
-                self._constraint = self._parser(glue.join(versions))
-            else:
-                self._constraint = self._parser(constraint_string)
+                constraint_string = glue.join(versions)
         else:
             # if we have a in/not in operator we split the constraint
             # into a union/multi-constraint of single constraint
@@ -256,7 +255,10 @@ def __init__(
                 values = re.split("[ ,]+", self._value)
                 constraint_string = glue.join(f"{op} {value}" for value in values)
 
+        try:
             self._constraint = self._parser(constraint_string)
+        except ParseConstraintError as e:
+            raise InvalidMarker(f"Invalid marker: {original_constraint_string}") from e
 
     @property
     def name(self) -> str:

tests/packages/test_dependency.py

@@ -6,7 +6,9 @@
 
 from poetry.core.constraints.version.exceptions import ParseConstraintError
 from poetry.core.packages.dependency import Dependency
+from poetry.core.version.markers import InvalidMarker
 from poetry.core.version.markers import parse_marker
+from poetry.core.version.requirements import InvalidRequirement
 
 
 @pytest.mark.parametrize(
@@ -184,6 +186,29 @@ def test_to_pep_508_combination() -> None:
     assert dependency.to_pep_508() == "foo (>=1.2,<1.3,!=1.2.5)"
 
 
+@pytest.mark.parametrize(
+    "requirement",
+    [
+        "enum34; extra == ':python_version < \"3.4\"'",
+        "enum34; extra == \":python_version < '3.4'\"",
+    ],
+)
+def test_to_pep_508_with_invalid_marker(requirement: str) -> None:
+    with pytest.raises(InvalidMarker):
+        _ = Dependency.create_from_pep_508(requirement)
+
+
+@pytest.mark.parametrize(
+    "requirement",
+    [
+        'enum34; extra == ":python_version < "3.4""',
+    ],
+)
+def test_to_pep_508_with_invalid_requirement(requirement: str) -> None:
+    with pytest.raises(InvalidRequirement):
+        _ = Dependency.create_from_pep_508(requirement)
+
+
 def test_complete_name() -> None:
     assert Dependency("foo", ">=1.2.3").complete_name == "foo"
     assert (