Properly coerce versions with leading zeroes.

A leading zero is forbidden in the SemVer spec, but could be valid under
other schemes; when coercing, it can easily be removed.

Closes rbarrois#89, thanks to Andrew Ni for the report.

Author: rbarrois

ChangeLog

@@ -4,7 +4,11 @@ ChangeLog
 2.8.4 (unreleased)
 ------------------
 
-- Nothing changed yet.
+*Bugfix:*
+
+    * `#89 <https://github.com/rbarrois/python-semanticversion/issues/89>`_:
+      Properly coerce versions with leading zeroes in components (e.g.
+      ``1.01.007``)
 
 
 2.8.3 (2019-11-21)

semantic_version/base.py

@@ -244,6 +244,14 @@ def coerce(cls, version_string, partial=False):
             while version.count('.') < 2:
                 version += '.0'
 
+        # Strip leading zeros in components
+        # Version is of the form nn, nn.pp or nn.pp.qq
+        version = '.'.join(
+            # If the part was '0', we end up with an empty string.
+            part.lstrip('0') or '0'
+            for part in version.split('.')
+        )
+
         if match.end() == len(version_string):
             return Version(version, partial=partial)
 

tests/test_base.py

@@ -569,7 +569,7 @@ def subTest(self, **kwargs):
     examples = {
         # Dict of target: [list of equivalents]
         '0.0.0': ('0', '0.0', '0.0.0', '0.0.0+', '0-'),
-        '0.1.0': ('0.1', '0.1+', '0.1-', '0.1.0'),
+        '0.1.0': ('0.1', '0.1+', '0.1-', '0.1.0', '0.01.0', '000.0001.0000000000'),
         '0.1.0+2': ('0.1.0+2', '0.1.0.2'),
         '0.1.0+2.3.4': ('0.1.0+2.3.4', '0.1.0+2+3+4', '0.1.0.2+3+4'),
         '0.1.0+2-3.4': ('0.1.0+2-3.4', '0.1.0+2-3+4', '0.1.0.2-3+4', '0.1.0.2_3+4'),