denisenkom / go-mssqldb

Microsoft SQL server driver written in go language
BSD 3-Clause "New" or "Revised" License
1.81k stars 493 forks source link

Add support TVP identity #771

Closed NikitaDef closed 1 year ago

codecov[bot] commented 1 year ago

Codecov Report

Merging #771 (de4b199) into master (c7ddec1) will decrease coverage by 0.03%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #771      +/-   ##
==========================================
- Coverage   71.34%   71.30%   -0.04%     
==========================================
  Files          24       24              
  Lines        5388     5405      +17     
==========================================
+ Hits         3844     3854      +10     
- Misses       1300     1305       +5     
- Partials      244      246       +2     
Impacted Files Coverage Δ
types.go 77.49% <ø> (ø)
tvp_go19.go 93.53% <100.00%> (+0.51%) :arrow_up:
tds.go 65.49% <0.00%> (-0.73%) :arrow_down:
token.go 64.21% <0.00%> (-0.15%) :arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

NikitaDef commented 1 year ago

@kardianos please check out the MR. What do you think about @identity constant? The driver detects by that const which field is the database identity

NikitaDef commented 1 year ago

I changed the test for more clarity