pygeofilter icon indicating copy to clipboard operation
pygeofilter copied to clipboard

various fixes

Open jokiefer opened this issue 1 year ago • 3 comments

fixes testsuite by reenginiering it:

  • new docker container which has preinstalled python 3.9 and 3.10 based on debian bullseye with gdal 3.2.2.
  • elasticsearch container to test against. In my pov this should be deprecated. Better would be to mock elasticsearch and test against the mock instead of this system test behavior.
  • The test container run matrix tests by using tox against django version 4.0, 4.1 and 4.2 combined with python 3.9 and 3.10.
  • If all tests successes, the container will exit with code 0.

No solution found for

  • python 3.8 will raise sqlite3.Connection object has no attribute 'enable_load_extension' for all sqlite depending tests
  • django 3.2 will raise django.db.utils.OperationalError: no such function: lwgeom_version for all spatial db depending tests

fixes also

#80, #81

jokiefer avatar Nov 16 '23 14:11 jokiefer

@constantinius could you pls review, merge and bump a new realease. thx

jokiefer avatar Nov 17 '23 06:11 jokiefer

@jokiefer This PR is quite massive, I need more time to review to see if all of it is applicable.

constantinius avatar Nov 22 '23 18:11 constantinius

@constantinius take your time. It was a lot of testing against the github action/docker container; thats why there are a flood of commits.

jokiefer avatar Nov 23 '23 06:11 jokiefer