-
Notifications
You must be signed in to change notification settings - Fork 22
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
[TASK] Switch to the core testing framework #361
Conversation
063b666
to
96e03a0
Compare
3970f6b
to
01eeab2
Compare
I don't know current state, but https://github.com/sabbelasichon/typo3-rector/blob/main/src/Set/Extension/NimutTestingFrameworkSetList.php might be helpful. We used the set during a project update from V8 to V10 to also switch the framework, and it did the job. |
@DanielSiepmann Thanks! |
53678ed
to
571ef3b
Compare
I got totally different errors locally in functional tests. |
This PR needs rebasing. |
e6ab9a8
to
01b414d
Compare
$connection = $this->getConnectionPool() | ||
->getConnectionForTable('tx_tea_domain_model_product_tea'); | ||
$databaseRow = $connection->select(['*'], 'tx_tea_domain_model_product_tea', ['uid' => $model->getUid()]) | ||
->fetchAssociative(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To be compatible with TYPO3 10LTS with lowest dependencies, we need to use the deprecated method here (maybe with a comment).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are the requirements in the composer.json
of typo3/cms-core
:
- V10:
"doctrine/dbal": "~2.10.0 || ~2.11.2 || ~2.13.1",
- V11:
"doctrine/dbal": "^2.13.5",
- main:
"doctrine/dbal": "^3.2",
f868a25
to
dea0962
Compare
The TYPO3 core testing framework has more person-power for maintenance behind it compared to the nimut testing framework. So we should use that.
We need to have the paths of the TYPO3 testing framework now.
dea0962
to
90a248c
Compare
`fetchAssociative()` is not available yet in doctrine/dbal 2.10 (which is thge lowest requirement for TYPO3 V10).
90a248c
to
2e4f829
Compare
Thus, we can avoid using a function that was removed in newer versions of DBAL.
The TYPO3 core testing framework has more person-power for maintenance
behind it compared to the nimut testing framework. So we should use that.