joomla-cms
joomla-cms copied to clipboard
[cypress] Add db_port in Installation.cy.js
Pull Request as a puzzle piece of Issue #43902
Summary of Changes
In the installation step of the system tests, support for the configuration of the database port is added.
Testing Instructions
The only isolated test that can be carried out for this change is that it does no harm.
You could see this PR is already used as hack in Joomla Branches Tester. There you can see it working in scripts/cypress.sh 44 local and running the Installation.cy.js step. Without the change (you have to comment-out the change in tests/System/integration/install/Installation.cy.js the Joomla Web Installer fails with:
Could not connect to the database. Connector returned error message: No such file or directory
Actual result BEFORE applying this Pull Request
The System Tests installation step cannot set the database port.
Expected result AFTER applying this Pull Request
The System Tests installation step can set the database port.
Link to documentations
Please select:
-
[ ] Documentation link for docs.joomla.org:
-
[X] No documentation changes for docs.joomla.org needed
-
[ ] Pull Request link for manual.joomla.org:
-
[X] No documentation changes for manual.joomla.org needed
Looks good to me.
See also my PR for the database framework: https://github.com/joomla-framework/database/pull/310 . Would be good to know if the PR here still works with the changes from my framework PR.
from my testing result with https://github.com/joomla-framework/database/pull/310
just set db_host: 'localhost:5433', proprerly in cypress.config.js
I have tested this item :white_check_mark: successfully on 9c2a993cd12a0f8b8acb7254e3b75c43c671d0e3
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43968.
Can you rebase this one to 5.2?
Rebased on 5.2-dev and checked System Tests on macOS with MariaDB passed
Thanks!