Skip to content

CI: Fixing PostgreSQL service

Pavlo Solntsev requested to merge pavlosun/libgda:wip/postgres into master
  • CI file was modified to remove DB definition
  • Fixing access of potentially NULL object with a function that doesn't accept this.

Some comments here. We don't need to provide DB name in the CI file. The test will create DB by itself. The connection can be tested using psql. I tested and removed from the CI. Variables POSTGERS_USER and POSTGRES_PASSWORD works for non postgres images. My previous statement about that was wrong. The check_postgres.c test is incomplete and buggy. We need another test where all DDL operation will be tested. I start working on this.

Merge request reports