Skip to content

Commit

Permalink
Merge pull request #124 from nexB/123-fix-generic-exception
Browse files Browse the repository at this point in the history
Use proper exception subclass for invalid constraints error
  • Loading branch information
keshav-space authored Sep 12, 2023
2 parents 2f2c21a + efa6bf6 commit e90df0d
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion src/univers/version_constraint.py
Original file line number Diff line number Diff line change
Expand Up @@ -471,6 +471,10 @@ def simplify_constraints(constraints):
return sorted(set(unequal_constraints + constraints))


class InvalidConstraintsError(Exception):
pass


def contains_version(version, constraints):
"""
Return True an assertion error if the ``constraints`` list contains the
Expand Down Expand Up @@ -539,7 +543,10 @@ def contains_version(version, constraints):

else:
# this should never happen as the constraints must be valid going in
raise Exception(f"Invalid constraints sequence: {constraints }")
raise InvalidConstraintsError(
f"Invalid constraints sequence: ('{cur_comp} {cur_constraint.version}',"
f"'{nxt_comp} {nxt_constraint.version}') in {constraints!r}"
)

# If this is the last iteration and next comparator is ">" or >="
# and the "tested version" is greater than the next version
Expand Down

0 comments on commit e90df0d

Please sign in to comment.