fix: pass dataLength to getMssqlType in valueCorrection

[dhensby]

Problem

valueCorrection() calls getMssqlType(metadata.type) without passing the length argument. The getMssqlType function uses length to disambiguate nullable N-types (IntN, FloatN, MoneyN, DateTimeN) — without it, they always resolve to the smallest variant:

N-Type	Expected (with length)	Actual (without length)
IntN (len=8)	BigInt	TinyInt
IntN (len=4)	Int	TinyInt
IntN (len=2)	SmallInt	TinyInt
FloatN (len=8)	Float	Real
MoneyN (len=8)	Money	SmallMoney
DateTimeN (len=8)	DateTime	SmallDateTime

This means valueHandler entries registered for BigInt, Int, SmallInt, Float, Money, or DateTime are never invoked — only the smallest variant handler fires.

Note: column metadata types (via createColumns) are not affected — that call site already passes column.dataLength correctly.

Closes #1853

Fix

One-liner: pass metadata.dataLength as the second argument to getMssqlType() in valueCorrection().

Tests

Added an integration test that registers separate value handlers for BigInt, Int, and TinyInt, then queries all three types. Without the fix, only the TinyInt handler fires; with the fix, each handler fires exactly once.

[Copilot]

Pull request overview

Fixes a driver-specific bug in the tedious implementation where valueCorrection() failed to pass dataLength into getMssqlType(), causing IntN/FloatN/MoneyN/DateTimeN values to be misclassified as their smallest variants and therefore dispatch to the wrong (or no) valueHandler.

Changes:

• Passes metadata.dataLength to getMssqlType() in lib/tedious/request.js so N-types resolve to the correct sql.TYPES.*.
• Adds an integration test to ensure valueHandler dispatch works correctly for BigInt, Int, and TinyInt.
• Wires the new test into both the tedious and msnodesqlv8 integration test runners.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
lib/tedious/request.js	Fixes N-type disambiguation for valueHandler dispatch by providing metadata.dataLength to getMssqlType().
test/common/tests.js	Adds an integration test asserting correct valueHandler routing for IntN-family integer types.
test/tedious/tedious.js	Registers the new shared integration test in the tedious test suite.
test/msnodesqlv8/msnodesqlv8.js	Registers the new shared integration test in the msnodesqlv8 test suite.

[github-actions]

🎉 This PR is included in version 12.5.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀