fix: avoid usage of packaging

src/libecalc/common/version.py

@@ -1,7 +1,5 @@
 import re
-from typing import Optional
-
-import packaging.version
+from typing import Optional, Tuple
 
 from libecalc.dto.base import EcalcBaseModel
 
@@ -54,38 +52,42 @@ def __str__(self) -> str:
     def __repr__(self) -> str:
         return f"Major: {self.major}\nMinor: {self.minor}\nPatch: {self.patch}"
 
+    @property
+    def _version_tuple(self) -> Tuple[int, int, int]:
+        return self.major, self.minor, self.patch
+
     def __gt__(self, other):
         if not isinstance(other, Version):
             return NotImplemented
 
-        return packaging.version.Version(str(self)).__gt__(packaging.version.Version(str(other)))
+        return self._version_tuple > other._version_tuple
 
     def __ge__(self, other):
         if not isinstance(other, Version):
             return NotImplemented
 
-        return packaging.version.Version(str(self)).__ge__(packaging.version.Version(str(other)))
+        return self._version_tuple >= other._version_tuple
 
-    def __le__(self, other):
+    def __lt__(self, other):
         if not isinstance(other, Version):
             return NotImplemented
 
-        return packaging.version.Version(str(self)).__le__(packaging.version.Version(str(other)))
+        return self._version_tuple < other._version_tuple
 
-    def __lt__(self, other):
+    def __le__(self, other):
         if not isinstance(other, Version):
             return NotImplemented
 
-        return packaging.version.Version(str(self)).__lt__(packaging.version.Version(str(other)))
+        return self._version_tuple <= other._version_tuple
 
     def __eq__(self, other):
         if not isinstance(other, Version):
             return NotImplemented
 
-        return packaging.version.Version(str(self)).__eq__(packaging.version.Version(str(other)))
+        return self._version_tuple == other._version_tuple
 
     def __ne__(self, other):
         if not isinstance(other, Version):
             return NotImplemented
 
-        return packaging.version.Version(str(self)).__ne__(packaging.version.Version(str(other)))
+        return self._version_tuple != other._version_tuple

src/tests/libecalc/common/test_version.py

@@ -36,6 +36,8 @@ def test_versions(version_string, expected_version):
     ("1.1.1", "1.1.2"),
     ("1.1.2", "1.1.1"),
     ("1.1.1", "1.1.1"),
+    ("100.1.1", "1.1.1"),
+    ("1.1.1", "1.1.111"),
 ]
 
 
@@ -57,6 +59,8 @@ def to_version_obj(versions: List[Tuple[str, str]]) -> List[Tuple[Version, Versi
     True,
     False,
     False,
+    False,
+    True,
 ]
 
 
@@ -73,6 +77,8 @@ def test_comparison_less_than(first, second, expected):
     False,
     True,
     False,
+    True,
+    False,
 ]
 
 
@@ -89,6 +95,8 @@ def test_comparison_grater_than(first, second, expected):
     False,
     False,
     True,
+    False,
+    False,
 ]
 
 
@@ -110,6 +118,8 @@ def test_comparison_not_equal(first, second):
     True,
     False,
     True,
+    False,
+    True,
 ]
 
 
@@ -126,6 +136,8 @@ def test_comparison_less_than_or_equal(first, second, expected):
     False,
     True,
     True,
+    True,
+    False,
 ]