mirror of
https://github.com/opensourcepos/opensourcepos.git
synced 2026-09-14 06:19:44 -04:00
jekkos
28755dfd50
* fix(tests): resolve all phpunit failures (#4626) Bring the phpunit suite from 153 failures to 0 (281 tests passing): - Employee: decouple grants block from save_value success; restructure save_employee new-employee + disallowed-grants early return - Sale: unify sales_payments_temp schema (add sale_cash_refund, reference_code) so both creators produce an identical superset table - Employees controller: provide placeholder password/hash in testing env so new-employee insert succeeds and grant logic is testable - TestDatabaseBootstrapSeeder: reset shared connection table-name cache after bootstrap reset to avoid stale listTables()/tableExists() results - Config: fix postSaveLocale validation rule syntax - Test data: use unique employee usernames to avoid UNIQUE constraint collisions latching strict-mode transStatus=false on the shared conn - Various test-file and language-string corrections * test: consolidate employee fixtures in shared trait Route test employee creation through a single EmployeeFixtureTrait that delegates to Employee::save_employee(), so fixtures exercise the same production code path instead of raw DB inserts. Removes six near-duplicate helpers across EmployeeTest, SalesControllerTest, and EmployeesControllerTest while preserving each test's specific grant set. Closes a piece of the fixture-scattering flagged in #4626. Closes #4626 * test: add global DROP/CREATE grant and commit theme fixtures * fix(ci): remove redundant symlink step, set working encryption key * fix(ci): run phpunit with --no-coverage to avoid no-driver warning * fix: address code review findings - Config: restore strict locale validation (min required|integer|>0) and fix max cross-field check with a new gte_field rule (CI4's greater_than_equal_to[field] does not resolve the field value) - Tests: assert rejection for non-numeric/zero/negative/min>max limits - .env.example: remove shared hard-coded encryption.key (auto-generates); document Docker env-var usage - phpunit.yml: scope CREATE/DROP grant to ospos_test.* and provision a per-run encryption key as an env var * feat: support ENCRYPTION_KEY env var for encryption key Read ENCRYPTION_KEY as a fallback for the encryption key when the config value is empty. This is a supported, reliable path for Docker / container deploys and CI, avoiding reliance on the raw dotted encryption.key env var. * fix: align Summary_report temp tables with Sale temp table schema Summary_report created sales_items_taxes_temp and sales_payments_temp with fewer columns than the canonical create_temp_table() in Sale.php. A later reader expecting those columns hit a schema-mismatch SQL error on the shared temp tables. Add internal_tax/sales_tax (sales_items_taxes_temp) and reference_code (sales_payments_temp) so all creators emit the identical column set.
55 lines
1.9 KiB
PHP
55 lines
1.9 KiB
PHP
<?php
|
|
|
|
namespace App\Database\Seeds;
|
|
|
|
use CodeIgniter\Database\Seeder;
|
|
use Config\Database;
|
|
|
|
class TestDatabaseBootstrapSeeder extends Seeder
|
|
{
|
|
public static function reset(): void
|
|
{
|
|
if (ENVIRONMENT !== 'testing') {
|
|
throw new \RuntimeException('TestDatabaseBootstrapSeeder can only run in the testing environment.');
|
|
}
|
|
|
|
$config = config('Database');
|
|
$group = $config->tests;
|
|
$dbName = $group['database'];
|
|
|
|
if ($dbName === '' || !str_contains(strtolower($dbName), 'test')) {
|
|
throw new \RuntimeException("Refusing to reset non-test database: {$dbName}");
|
|
}
|
|
|
|
$serverConn = Database::connect([
|
|
'hostname' => $group['hostname'],
|
|
'username' => $group['username'],
|
|
'password' => $group['password'],
|
|
'DBDriver' => $group['DBDriver'],
|
|
'database' => null,
|
|
'charset' => $group['charset'] ?? 'utf8mb4',
|
|
'DBCollat' => $group['DBCollat'] ?? 'utf8mb4_general_ci',
|
|
], false);
|
|
|
|
$serverConn->query("DROP DATABASE IF EXISTS `{$dbName}`");
|
|
$serverConn->query("CREATE DATABASE IF NOT EXISTS `{$dbName}`");
|
|
$serverConn->close();
|
|
|
|
// The application's shared 'tests' connection caches listTables()
|
|
// results in dataCache['table_names'']. Because we dropped and
|
|
// recreated the schema on a SEPARATE server connection above, that
|
|
// cache is now stale. Reset it so subsequent listTables()/tableExists()
|
|
// calls re-query the server instead of trusting a dropped schema's table
|
|
// list (which previously caused "ospos_migrations doesn't exist" and
|
|
// "column ... doesn't exist" errors for every class that followed a
|
|
// bootstrap-reset class in the suite).
|
|
$shared = Database::connect('tests');
|
|
$shared->resetDataCache();
|
|
}
|
|
|
|
public function run(): void
|
|
{
|
|
self::reset();
|
|
}
|
|
}
|