Build | Correct references to Microsoft.SqlServer.Server in Visual Studio builds

[edwardneal]

Description

This follows up my previous PR - I can't currently build via Visual Studio. Upon trying to do so, the net8.0, net9.0 and net10.0 targets for ManualTests fail with the error:

The type 'IBinarySerialize' is defined in an assembly that is not referenced. You must add a reference to assembly 'Microsoft.SqlServer.Server, Version=7.0.0.0, Culture=neutral, PublicKeyToken=null'.

The versioning for Microsoft.SqlServer.Server is also wrong because the package is versioned independently of SqlClient and its associated libraries, but this doesn't break the build - and more importantly, threading the package-specific versions into Microsoft.SqlServer.Server just changes the version in the error message:

The type 'IBinarySerialize' is defined in an assembly that is not referenced. You must add a reference to assembly 'Microsoft.SqlServer.Server, Version=1.0.0.0, Culture=neutral, PublicKeyToken=null'.

Converting netcore's unconditional PackageReference into the newer style of PackageReference/ProjectReference fixes this problem.

Just to neaten up the version numbers and dependencies, we then return to Microsoft.SqlServer.Server and bring the build configuration into the same style as the Abstractions project file, adding the various version-related properties and making sure that these are threaded into the nuspec.

Issues

None.

Testing

Visual Studio builds from scratch. build.proj continues to function properly, setting correct dependencies on Microsoft.SqlServer.Server. I can continue to execute UDT-based tests in ManualTests, so the updated DLL loads properly.

[paulmedynski]

I'll trigger the pipelines once you add the SqlServer version to the several place the YAML needs it.

[mdaigle]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 2 pipeline(s).

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.92%. Comparing base (3303d80) to head (ae2e17f).
⚠️ Report is 8 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (3303d80) and HEAD (ae2e17f). Click for more details.

HEAD has 2 uploads less than BASE

Flag	BASE (3303d80)	HEAD (ae2e17f)
CI-SqlClient	2	0

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4003      +/-   ##
==========================================
- Coverage   74.38%   64.92%   -9.47%     
==========================================
  Files         287      282       -5     
  Lines       43982    66073   +22091     
==========================================
+ Hits        32717    42896   +10179     
- Misses      11265    23177   +11912     

Flag	Coverage Δ
CI-SqlClient	?
PR-SqlClient-Project	64.92% <ø> (?)

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.

[paulmedynski]

@edwardneal The pipeline errors appear to be caused by a missing property value on the nuget command line.

[edwardneal]

I think I see the problem, thanks @paulmedynski. Does the pipeline configuration need to define sqlServerArtifactsName and sqlServerPackageVersion variables? It looks like the root cause is that these aren't being substituted by the pipeline.

[mdaigle]

I believe those need to be defined in the variables yml file. Look for the other version variables, they should be defined in the same spot.

[edwardneal]

Thanks - I think I've found them, could you rerun CI please?