Skip to content
Snippets Groups Projects
Commit 816476c2 authored by Stefan Bürk's avatar Stefan Bürk
Browse files

[TASK] Upgrade to codeception/codeception 5.x

Codeception released new major version 5, which
unblocks upgrading to symfony 6 components.
This patch raises codeception only, thus acting as a
pre-patch to raise symfony requirements.

Codeception 5 is hard to impossible with core v11,
so this patch is v12 only: Codeception 5 requires
PHP 8.0, so we'd have to support both 4 and 5 in
v11 to allow testing with older PHP versions. This
would result in various headaches since some API
changes in codeception 5 are hard breaking in PHP,
especially due to changed class property type hints.
We'll thus keep codeception 4 in v11 and go with
codeception 5 in v12.

Minor acceptance test related adaptions are needed:

* Replaced config option `path.log` with `path.output`
* Added proper property type because of inheritance to
  codeception setup classes
* Ensure method return type compatibility
* Use `actor`instead of `class name` in suite config
* Create acceptance split files in proper folder to
  ensure group filtering keeps working. Adjust clean
  up scripts and gitignore for this

Used commands:

> composer req --dev -w \
  typo3/testing-framework:dev-main \
  codeception/codeception:^5.0.0 \
  codeception/module-asserts:^3.0.0 \
  codeception/module-cli:^2.0.0 \
  codeception/module-filesystem:^3.0.0 \
  codeception/module-webdriver:^3.1.2 \
  codeception/lib-asserts:^2.0.0

Resolves: #98047
Releases: main
Change-Id: I5cea60c2641988cc032b30ed1fbe3d8b0ed3b3d1
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75317


Tested-by: default avatarcore-ci <typo3@b13.com>
Tested-by: default avatarOliver Klee <typo3-coding@oliverklee.de>
Tested-by: default avatarChristian Kuhn <lolli@schwarzbu.ch>
Tested-by: default avatarStefan Bürk <stefan@buerk.tech>
Reviewed-by: default avatarChristian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: default avatarStefan Bürk <stefan@buerk.tech>
parent 25128d45
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment