8000 joomla-cypress test suite by muhme · Pull Request #39 · joomla-projects/joomla-cypress · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

joomla-cypress test suite #39

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Dec 20, 2024

Conversation

muhme
Copy link
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
Contributor
laoneo commented Dec 16, 2024

Thanks, should the cypress folder not be named tests?

@muhme
Copy link
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
Contributor
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
Contributor
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
0