Merge pull request #102 from urschrei/shugel/push-oxsqwlokvytw

Add loop test

Author: urschrei

.github/workflows/wheels.yml

@@ -56,7 +56,7 @@ jobs:
           id: windows
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
         with:
           fetch-depth: 0  # Optional, use if you use setuptools_scm
           submodules: true  # Optional, use if you have submodules
@@ -133,7 +133,7 @@ jobs:
     name: Make SDist
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v5
       with:
         fetch-depth: 0  # Optional, use if you use setuptools_scm
         submodules: true  # Optional, use if you have submodules
@@ -157,7 +157,7 @@ jobs:
       contents: write
 
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v5
       with:
         fetch-depth: 0  # Optional, use if you use setuptools_scm
         submodules: true  # Optional, use if you have submodules

tests/test_convertbng.py

@@ -1,21 +1,23 @@
-import unittest
-import numpy as np
 import array
 import math
+import unittest
+from ctypes import ArgumentError
+from random import randint, randrange
+
+import numpy as np
+from convertbng.cutil import convert_bng as cconvert_bng
+
 from convertbng.util import (
     convert_bng,
+    convert_epsg3857_to_wgs84,
+    convert_etrs89_to_lonlat,
+    convert_etrs89_to_osgb36,
     convert_lonlat,
-    convert_to_osgb36,
-    convert_osgb36_to_lonlat,
     convert_osgb36_to_etrs89,
-    convert_etrs89_to_osgb36,
-    convert_etrs89_to_lonlat,
-    convert_epsg3857_to_wgs84,
+    convert_osgb36_to_lonlat,
     convert_to_etrs89,
+    convert_to_osgb36,
 )
-from convertbng.cutil import convert_bng as cconvert_bng
-from ctypes import ArgumentError
-from random import randrange
 
 
 class ConvertbngTests(unittest.TestCase):
@@ -58,6 +60,16 @@ def testCythonConvertList(self):
         )
         self.assertEqual(expected[0][0], result[0][0])
 
+    def test_loop(self):
+        """See https://github.com/urschrei/convertbng/issues/101"""
+
+        def test_conversion(easting, northing):
+            res = convert_lonlat(easting, northing)
+            return [res[0][0], res[1][0]]
+
+        for i in range(500000):
+            test_conversion([randint(10000, 50000)], [randint(10000, 50000)])
+
     def testConvertBNG(self):
         """Test multithreaded BNG --> lon, lat function"""
         expected = ([-0.32822654, -2.01831267], [51.44533145, 54.58910534])