Skip to content

joomla-cypress test suite#39

Merged
laoneo merged 8 commits intojoomla-projects:mainfrom
muhme:joomla-cypress-test-suite
Dec 20, 2024
Merged

joomla-cypress test suite#39
laoneo merged 8 commits intojoomla-projects:mainfrom
muhme:joomla-cypress-test-suite

Conversation

@muhme
Copy link
Copy Markdown
Contributor

@muhme muhme commented Dec 15, 2024

Pull Request for Issue #14

Cypress-based test suite for joomla-cypress NPM module. Each Cypress custom command is tested at least once.

Update to Cypress 13.16.0 is needed to use defaultBrowser in cypress config.

The joomla-cypress test suite was tested headless with:

  • Own target

    • Windows 11 24H2 + Laragon + IPv4 + Joomla 5.2-dev
    • macOS 15.1 Intel + MariaDB + Apache + IPv4 + Joomla 5.2-dev
  • JBT

    • Windows 11 24H2 + WSL 2 Ubuntu + MariaDB-MySQLi + IPv4 + Joomla 4.4-dev, 5.2-dev, 5.3-dev, 6.0-dev
    • Ubuntu 24.04 + MariaDB-MySQLi + IPv4 + Joomla 4.4-dev, 5.2-dev, 5.3-dev, 6.0-dev
    • macOS 15.1 Apple Silicon
      • IPv4 + MariaDB-MySQLi + Joomla 4.4-dev, 5.2-dev, 5.3-dev, 6.0-dev
      • IPv6 + Joomla 5.2-dev + MySQL-MySQLi, MySQL-PDO, MariaDB-MySQLi, MariaDB-PDO, PostgreSQL-PDO, MySQL-MySQLi-Socket, MySQL-PDO-Socket, MariaDB-MySQLi-Socket, Mariadb-PDO-Socket, PostgreSQL-PDO-Socket
    • macOS 15.1 Intel + MariaDB-MySQLi + IPv4 + Joomla 4.4-dev, 5.2-dev, 5.3-dev, 6.0-dev

    ⚠️ You have to install patch Fixing installJoomlaMultilingualSite() #37 first or installJoomlaMultilingualSite and a lot of sub-sequent tests will fail.


🔔 19 Dec 2024 PM: Directory level tests inserted before the Cypress tests, directory cypress removed and tests are moved into e2e sub-folder. JBT adopted too and you have to use version >= 2.1.16.

Update to Cypress 13.16.0 is needed to use defaultBrowser in cypress config

Tested joomla-cypress test suite headless with:
* Own target
  * Windows 11 24H2 + Laragon + IPv4 + Joomla 5.2-dev
  * macOS 15.1 Intel + MariaDB + Apache + IPv4 + Joomla 5.2-dev
* JBT
  * Windows 11 24H2 + WSL 2 Ubuntu + MariaDB-MySQLi + IPv4 + Joomla 5.2-dev
  * Ubuntu 24.04 + MariaDB-MySQLi + IPv4 + Joomla 4.4-dev, 5.2-dev, 5.3-dev, 6.0-dev
  * macOS 15.1 Apple Silicon
    * IPv4 + MariaDB-MySQLi + Joomla 4.4-dev, 5.2-dev, 5.3-dev, 6.0-dev
    * IPv6 + Joomla 5.2-dev + MySQL-MySQLi, MySQL-PDO, MariaDB-MySQLi, MariaDB-PDO, PostgreSQL-PDO, MySQL-MySQLi-Socket, MySQL-PDO-Socket, MariaDB-MySQLi-Socket, Mariadb-PDO-Socket, PostgreSQL-PDO-Socket
  * macOS 15.1 Intel + MariaDB-MySQLi + IPv4 + Joomla 4.4-dev, 5.2-dev, 5.3-dev, 6.0-dev
@laoneo
Copy link
Copy Markdown
Collaborator

laoneo commented Dec 16, 2024

Thanks, should the cypress folder not be named tests?

@muhme
Copy link
Copy Markdown
Contributor Author

muhme commented Dec 16, 2024

I did indeed start with tests. Later I renamed it to cypress because cypress/fixtures and cypress/screenshots were needed. Renaming is not possible, I think. We can leave it at cypress or move cypress to tests/cypress and ./cypress.config.* to tests?

@laoneo
Copy link
Copy Markdown
Collaborator

laoneo commented Dec 17, 2024

A common setup in many projects is that you have a src folder with the code and a tests folder with all the testing stuff. I would like to keep it that way. How it is now, it is not possible to distinguish what is testing code and what is library code.

@laoneo laoneo merged commit 5e3376f into joomla-projects:main Dec 20, 2024
@laoneo
Copy link
Copy Markdown
Collaborator

laoneo commented Dec 20, 2024

Thanks you very much!

@muhme muhme deleted the joomla-cypress-test-suite branch December 20, 2024 08:45
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.

2 participants