Skip to content

[4.x] fix connection pgsql (ci)#365

Open
heelc29 wants to merge 2 commits into
joomla-framework:4.x-devfrom
heelc29:4.x/fix-pgsql-ci
Open

[4.x] fix connection pgsql (ci)#365
heelc29 wants to merge 2 commits into
joomla-framework:4.x-devfrom
heelc29:4.x/fix-pgsql-ci

Conversation

@heelc29

@heelc29 heelc29 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Pull Request for Issue #

Summary of Changes

  • JOOMLA_TEST_DB_HOST should match service name of workflow file
  • JOOMLA_TEST_DB_PORTd efault postgres port is 5432
  • JOOMLA_TEST_DB_USER should match environment variable POSTGRES_USER
  • JOOMLA_TEST_DB_PASSWORD should match environment variable POSTGRES_PASSWORD

postgres was handle different in travis (drone im not sure it worked).
The database base was created with commands and manual import the test data. In GitHub Actions we set environment variable POSTGRES_DB to create the database.

database/.travis.yml

Lines 229 to 231 in 1944153

before_script:
- psql -U postgres -c 'create database joomla_ut;'
- psql -U postgres -d joomla_ut -a -f Tests/Stubs/Schema/pgsql.sql

database/.drone.jsonnet

Lines 170 to 173 in ff7ee96

commands: [
'psql -U postgres -c ',
'psql -U postgres -d joomla_ut -a -f Tests/Stubs/Schema/pgsql.sql',
],

This PR adjust the handling of test data like mysql/mysqli.

Testing Instructions

ci/cd

Documentation Changes Required

no

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.

1 participant