fix: pass columnType to readDateTimeString

[steveetm]

See issue: #3699

[steveetm]

I could not add test or run those as locally running them always stuck after a few seconds. If you can help me sorting that out I would like to add some tests as well.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.09%. Comparing base (9dd0202) to head (d40c649).
Report is 3 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #3700   +/-   ##
=======================================
  Coverage   89.08%   89.09%           
=======================================
  Files          86       86           
  Lines       13538    13544    +6     
  Branches     1569     1571    +2     
=======================================
+ Hits        12061    12067    +6     
  Misses       1477     1477           

Flag	Coverage Δ
compression-0	89.09% <100.00%> (+<0.01%)	⬆️
compression-1	89.09% <100.00%> (+<0.01%)	⬆️
static-parser-0	86.65% <62.50%> (-0.01%)	⬇️
static-parser-1	87.42% <37.50%> (-0.03%)	⬇️
tls-0	88.51% <100.00%> (+<0.01%)	⬆️
tls-1	88.86% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[wellwelwel]

Curiously, some scenarios wait for the datetime and others pass with just the date:

1) test/esm/integration/parsers/typecast-field-datetime.test.mjs

  🔬 typeCast field.datetime
  ✘ DATETIME
        File test/esm/integration/parsers/typecast-field-datetime.test.mjs:37:8
        Code ERR_ASSERTION
    Operator ==
    Actual:
    2025-07-09
    Expected:
    2025-07-09 00:00:00

[steveetm]

Curiously, some scenarios wait for the datetime and others pass with just the date:

The expection there is to .query and .execute results should match.
This failure is the exact same what I get prior this PR, and no matter what I try locally(tried different node, mysql versions and configurations) it is always passes with the PR(and fails without it, obviously)

I don't know how, but it seems like the change is not applied for those tests, but that shouldn't be possible.

[wellwelwel]

@steveetm, thanks a lot! Today marks exactly one year since this error was reported on Sequelize (sequelize/sequelize#17141 (comment)).