search

microsoft / msphpsql

Microsoft Drivers for PHP for SQL Server
MIT License
1.8k stars 374 forks source link

Fix clientInfo related test cases #1407

Closed absci closed 2 years ago

codecov[bot] commented 2 years ago

Codecov Report

Merging #1407 (babb8fe) into dev (109b8bc) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev    #1407   +/-   ##
=======================================
  Coverage   83.61%   83.61%           
=======================================
  Files          22       22           
  Lines        7853     7853           
=======================================
  Hits         6566     6566           
  Misses       1287     1287
Impacted Files Coverage Δ
...php-8.0.19-src/ext/pdo_sqlsrv/php_pdo_sqlsrv_int.h
...x86/php-8.0.19-src/ext/sqlsrv/shared/core_conn.cpp
...s16/x86/php-8.0.19-src/ext/pdo_sqlsrv/pdo_stmt.cpp
...x86/php-8.0.19-src/ext/sqlsrv/shared/core_sqlsrv.h
.../php-8.0.19-src/ext/sqlsrv/shared/core_results.cpp
...s16/x86/php-8.0.19-src/ext/pdo_sqlsrv/pdo_util.cpp
...x86/php-8.0.19-src/ext/sqlsrv/shared/core_stmt.cpp
...php-8.0.19-src/ext/pdo_sqlsrv/shared/core_util.cpp
...php-8.0.19-src/ext/pdo_sqlsrv/shared/core_conn.cpp
...-8.0.19-src/ext/pdo_sqlsrv/shared/core_results.cpp
... and 34 more
absci commented 2 years ago

Due to ODBC upgraded from 17,9 to 17.10. Previously the test expected to have only 1 digit for minor version number, This PR changed that to 1 or 2 digits.