Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

various fixes #82

Open
wants to merge 31 commits into
base: main
Choose a base branch
from
Open

various fixes #82

wants to merge 31 commits into from

Conversation

jokiefer
Copy link
Contributor

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
Copy link
Contributor Author

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

@constantinius
Copy link
Contributor

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

@jokiefer
Copy link
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants