PostgreSQL CI - Githubissues

georust / geozero

Zero-Copy reading and writing of geospatial data.

Apache License 2.0

322 stars 30 forks source link

PostgreSQL CI #136

Closed nyurik closed 1 year ago

nyurik commented 1 year ago

Run postgresql as part of the CI
Refactor postgis tests a bit to consolidate connection code

nyurik commented 1 year ago

@pka could you take a look at the build log of this PR - I am not certain how PG tests should be executed.

P.S. feel free to make any changes to this PR and merge when ready

pka commented 1 year ago

The CI failures are at std::env::var("DATABASE_URL").unwrap(). In the CI Yaml it's _URI instead of _URL!

nyurik commented 1 year ago

@pka thx, fixed the issue, but now 4 unit tests don't pass. I fixed two of them (it seems now the shown values don't have the unneeded .000000000 ? Could you take a look again pls