From 0113d672cb898f0b2c5d06e5063348eb3b8dabd0 Mon Sep 17 00:00:00 2001 From: Keshav Priyadarshi Date: Thu, 31 Aug 2023 21:47:15 +0530 Subject: [PATCH] Handle NoneType in VersionRange.from_string (#115) * Handle NoneType in VersionRange.from_string Signed-off-by: Keshav Priyadarshi * Address review comments Signed-off-by: Keshav Priyadarshi --------- Signed-off-by: Keshav Priyadarshi --- src/univers/version_range.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/univers/version_range.py b/src/univers/version_range.py index 37138aae..399212ad 100644 --- a/src/univers/version_range.py +++ b/src/univers/version_range.py @@ -98,6 +98,11 @@ def from_string(cls, vers, simplify=False, validate=False): Return a VersionRange built from a ``vers`` version range spec string, such as "vers:npm/1.2.3,>=2.0.0" """ + if not vers or not isinstance(vers, str) or not vers.strip(): + raise ValueError( + f"{vers!r} is not a valid argument, a valid ``vers`` string argument is required." + ) + # Spaces are not significant and removed in a canonical form. vers = remove_spaces(vers)