mirror of
https://github.com/Vateron-Media/XC_VM.git
synced 2026-10-29 19:03:29 +01:00
Two regressions surfaced when running the suite (9 + 3 errors): - setDb(DatabaseHandler) rejects the SQLite double: the DatabaseAware refactor gave setDb() a strict DatabaseHandler hint, but TestDb was a standalone class. Make TestDb extend DatabaseHandler (a real subtype) so it satisfies the seam; its own constructor wires sqlite::memory: and never calls the MySQL-connecting parent constructor. Overridden methods widen parameter types (contravariant) and add return types, so LSP holds. - validateHMAC(int|string $rExpiry) rejects null: HmacTokenTest passed null for an intentionally-empty expiry. Pass '' — identical HMAC input, and the strict type stays correct (the sole production caller always passes a value). Suite is green again: 580 tests, 0 errors.