Skip to content

Conversation

@laoneo
Copy link
Member

@laoneo laoneo commented Sep 29, 2025

Pull Request for pr #46180.

Summary of Changes

Adds Mysql to the integration and system test matrix and proper min/max versions for the databases.

Additionally I had to migrate to the node mysql2 package, which handles the authentication properly.

When upmerged to 6, then the versions should be updated according to the requirements of Joomla 6.

Testing instructions

Verifying GitHub actions with minimum technical-requirements and max available stable version.

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.4-dev Unit/System Tests labels Sep 29, 2025
@laoneo laoneo mentioned this pull request Sep 29, 2025
4 tasks
@laoneo laoneo changed the title [5.4] Add mysql to test system matrix and proper min max db versions [5.4] Add mysql to test matrix and proper min max db versions Sep 29, 2025
@laoneo laoneo force-pushed the tests branch 3 times, most recently from af0cd94 to 6f2b73c Compare September 29, 2025 13:30
@HLeithner HLeithner added the bug label Oct 15, 2025
Copy link
Contributor

@muhme muhme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • New added GitHub actions were running multiple times successful
  • All Joomla 5.x min and current LTS versions are in use
  • Used PHP versions: 8.1, 8.2, 8.3 and 8.4
  • Used MySQL versions: 8.0.13 and 8.4
  • User MariaDB versions: 10.4 and 12.0
  • Used PostgreSQL versions: 12.0 and 18.0
  • System tests are running with Google Chrome and Microsoft Edge with all three databases min and max versions

@richard67 richard67 merged commit 7e9034a into joomla:5.4-dev Oct 23, 2025
60 checks passed
@richard67
Copy link
Member

Thanks @laoneo for the PR, and @muhme for testing/review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.4-dev Unit/System Tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants