fix version comparison operations on lazy GDAL_VERSION object (#11)

Author: joshuadavidthomas

CHANGELOG.md

@@ -17,6 +17,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 ## [Unreleased]
 
+### Fixed
+
+- Fixed GDAL version comparison by implementing custom comparison operators for the lazy `GDAL_VERSION` object, allowing Django to compare version information without loading GDAL prematurely
+
 ## [0.2.1]
 
 ### Fixed

src/django_lazy_gdal/libgdal.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import logging
+import operator
 import os
 import re
 from ctypes import CDLL
@@ -11,6 +12,7 @@
 
 from django.core.exceptions import ImproperlyConfigured
 from django.utils.functional import SimpleLazyObject
+from django.utils.functional import new_method_proxy
 
 logger = logging.getLogger("django.contrib.gis")
 
@@ -204,4 +206,9 @@ def gdal_version_info():
     return (int(major), int(minor), subminor and int(subminor))
 
 
-GDAL_VERSION = SimpleLazyObject(gdal_version_info)
+class ComparableSimpleLazyObject(SimpleLazyObject):
+    __ge__ = new_method_proxy(operator.ge)
+    __le__ = new_method_proxy(operator.le)
+
+
+GDAL_VERSION = ComparableSimpleLazyObject(gdal_version_info)

tests/test_lazy_libgdal.py

@@ -145,3 +145,19 @@ def test_load_gdal_failure(mock_find_library):
 
     with pytest.raises(ImproperlyConfigured, match="Could not find the GDAL library"):
         lazy_libgdal.lgdal.some_attribute
+
+
+def test_gdal_version_comparison():
+    import django_lazy_gdal.libgdal as lazy_libgdal
+    from django_lazy_gdal.libgdal import ComparableSimpleLazyObject
+
+    assert isinstance(lazy_libgdal.GDAL_VERSION, ComparableSimpleLazyObject)
+
+    version = ComparableSimpleLazyObject(lambda: (3, 2, 1))
+
+    assert version >= (3, 0, 0)
+    assert version > (2, 9, 9)
+    assert version < (3, 3, 0)
+    assert version <= (3, 2, 1)
+    assert version == (3, 2, 1)
+    assert version != (3, 0, 0)