Fix ConvertException false positives from SQLSTATE[HY match

[darkspock]

Summary

ConvertException::run() matches SQLSTATE[HY in error messages to detect integrity constraint violations. However, SQLSTATE class HY is the "CLI-specific condition" class (e.g., HY000 = general error, HY001 = memory allocation error, HY010 = function sequence error). These are not integrity constraint violations and should not be wrapped as IntegrityException.

This can cause false positives where general database errors are incorrectly classified as integrity violations.

Fix

• Remove the SQLSTATE[HY string match
• Add a proper check using PDOException::errorInfo[0] (the SQLSTATE code) for class 23, which is the standard SQLSTATE class for integrity constraint violations across all DBMS
• Keep the existing SQLSTATE[23 message string match and ORA-00001 match as fallbacks

Before

return match (
    str_contains($message, self::MSG_INTEGRITY_EXCEPTION_1)
    || str_contains($message, self::MGS_INTEGRITY_EXCEPTION_2)
    || str_contains($message, self::MSG_INTEGRITY_EXCEPTION_3) // SQLSTATE[HY — too broad
) {

After

$isIntegrity = str_contains($message, self::MSG_INTEGRITY_EXCEPTION_1)
    || str_contains($message, self::MGS_INTEGRITY_EXCEPTION_2)
    || $this->isIntegrityConstraintViolation($errorInfo); // checks errorInfo[0] starts with '23'

Risk

Low — this makes detection more precise. Errors that were previously (incorrectly) classified as IntegrityException will now correctly be classified as Exception. Code that catches IntegrityException specifically will no longer catch unrelated errors.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.71%. Comparing base (dffa745) to head (fa8d9c0).

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1174      +/-   ##
============================================
+ Coverage     98.62%   98.71%   +0.09%     
- Complexity     1645     1648       +3     
============================================
  Files           120      120              
  Lines          4292     4289       -3     
============================================
+ Hits           4233     4234       +1     
+ Misses           59       55       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Tigrov]

Another PR already fixing this part of code.
#1170

If you have any suggestions regarding this, please write comment in that PR

[vjik]

Closed in favor #1170