-
Notifications
You must be signed in to change notification settings - Fork 122
Open
Description
Description
Recent CI runs on PRs against 5.x branch are failing on MySQL and MariaDB tests, even though the 5.x branch itself was green as of Feb 25. These appear to be pre-existing issues unrelated to PR changes.
Failures
1. MariaDB: testMigrateAndRollback assertion failure
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'current_timestamp()'
+'CURRENT_TIMESTAMP'
/home/runner/work/migrations/migrations/tests/TestCase/MigrationsTest.php:228
The test at line 223-224 expects current_timestamp() (lowercase with parentheses) for MariaDB:
if ($driver instanceof Mysql && $driver->isMariadb()) {
$expected = 'current_timestamp()';
}But it's receiving CURRENT_TIMESTAMP (uppercase, no parentheses), suggesting either:
isMariadb()is returning false (using MySQL default)- MariaDB 11.8 changed to return the uppercase format
2. PHP 8.5: BakeMigrationDiffCommandTest errors
Error: Typed property Cake\Database\Schema\Column::$fixed must not be accessed before initialization
/vendor/cakephp/cakephp/src/Database/Schema/Column.php:530
This was reported in cakephp/cakephp#19308 and was related to cached table schema. It may be a cache clearing issue in the test environment.
Environment
- CI uses
mariadb:11.8Docker image - PHP versions: 8.2, 8.3, 8.5
- CakePHP constraint:
^5.3.0(currently resolves to 5.3.2)
Related
- PR Fix ci for mariadb #1024 fixed similar MariaDB collation issues but not this timestamp issue
- CakePHP issue #19308 about
Column::$fixedinitialization
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels