mirror of
https://github.com/opensourcepos/opensourcepos.git
synced 2026-10-28 19:18:58 -04:00
e20fb2a792
Add MySQL 8.4 compatibility, including a utf8mb4 conversion migration, hardened foreign key handling, and expanded CI database coverage. Migrations: - Add Mysql84Utf8mb4Conversion migration using sqlscripts/3.4.3_mysql84_utf8mb4_conversion.sql via executeScript() - Drop ospos_people composite index before conversion, since its prefix length exceeds the 3072-byte key limit under utf8mb4 - Add assertNoCollationCollisions() pre-flight check to fail fast on unique values that collide under utf8mb4_unicode_520_ci, instead of hitting "Duplicate entry" partway through ALTER TABLE - Wrap the conversion in try/catch so foreign keys are recreated on failure; log restore errors separately so they don't mask the original exception - Throw RuntimeException if re-adding the people index fails - Remove the re-add of the sales_items_taxes FK in down(), which MySQL 8.4 rejects without a unique key - Revert edits to historical migrations; fix FK handling and table names migration_helper: - Add shared dropAllForeignKeyConstraints() and recreateForeignKeyConstraints(); table/column now optional, dropping all FKs in the schema when omitted - Match CONSTRAINT_SCHEMA when joining KEY_COLUMN_USAGE to REFERENTIAL_CONSTRAINTS to avoid cross-schema name collisions - Rebuild FK lookup with the query builder and bound values instead of interpolated SQL CI/tests: - Add mysql:5.7, mariadb:10.6 and mariadb:10.11 to PHPUnit matrix - Use master instead of latest image tag; improve CI robustness - Mock table() in FailedQueryConnectionTrait so query builder chains work against the failing connection Signed-off-by: objecttothis <17935339+objecttothis@users.noreply.github.com> Co-authored-by: aliworksx08 <57456290+aliworksx08@users.noreply.github.com> Co-authored-by: jekkos <jeroen.peelaerts@gmail.com> Co-authored-by: objecttothis <17935339+objecttothis@users.noreply.github.com> Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
45 lines
1.2 KiB
PHP
45 lines
1.2 KiB
PHP
<?php
|
|
|
|
namespace Tests\Support;
|
|
|
|
use CodeIgniter\BaseModel;
|
|
use CodeIgniter\Database\BaseConnection;
|
|
|
|
trait FailedQueryConnectionTrait
|
|
{
|
|
protected function failingConnection(): BaseConnection
|
|
{
|
|
$realConnection = \Config\Database::connect($this->DBGroup);
|
|
$driverClass = get_class($realConnection);
|
|
$builderClass = str_replace('Connection', 'Builder', $driverClass);
|
|
|
|
$db = $this->getMockBuilder($driverClass)
|
|
->disableOriginalConstructor()
|
|
->onlyMethods(['query', 'error', 'table'])
|
|
->getMock();
|
|
$db->method('query')->willReturn(false);
|
|
$db->method('error')->willReturn(['code' => 1, 'message' => 'simulated query failure']);
|
|
$db->method('table')->willReturnCallback(static fn (string $tableName) => new $builderClass($tableName, $db));
|
|
|
|
return $db;
|
|
}
|
|
|
|
/**
|
|
* @template T of object
|
|
*
|
|
* @param class-string<T> $class
|
|
*
|
|
* @return T
|
|
*/
|
|
protected function modelWithFailingQuery(string $class)
|
|
{
|
|
$model = new $class();
|
|
|
|
$prop = new \ReflectionProperty(BaseModel::class, 'db');
|
|
$prop->setAccessible(true);
|
|
$prop->setValue($model, $this->failingConnection());
|
|
|
|
return $model;
|
|
}
|
|
}
|