@edwardneal Did you look at the code coverage report for "inspiration" ?

I did; Codecov aligns with it:

• SqlDataRecord.cs
• SqlDataRecord.netcore.cs
• SqlDataRecord.netfx.cs
• SqlMetaData.cs

The code coverage for SqlDataRecord is currently ~75%, where I expect this PR will bring it to close to 100% in the code coverage report. The code paths which aren't covered are ones which can't ever be called and are due to be removed; once that happens I expect these two files will hit 100%.

SqlParameter is a much larger class, and it's got a custom TypeConverter which doesn't appear to have test coverage. I've added a few tests for basic validation, but the class will need more work here.

@mdaigle would you mind triggering the Azure Pipelines bot please? It looks like CI has stopped triggering on my PRs.

/azp run

Thanks. It looks like there's a slightly deeper problem with CI - the Password and the Azure_Password variables (and maybe other secret variables?) aren't being substituted.

Thanks. It looks like there's a slightly deeper problem with CI - the Password and the Azure_Password variables (and maybe other secret variables?) aren't being substituted.

Yeah, I need to dig into this further. We had a security action item to limit forked build access to pipeline secrets and I think the trigger was pulled on that before we made accommodating changes. There's a general trend toward locked down permissions for forked builds. We need to start updating tests to use dynamically generated (rather than static) resources so that we don't rely on pipeline secrets.

@David-Engel started doing some work in this direction with#2750. I bumped the pipeline there to see if mac tests are passing with those changes.

/azp run

@edwardneal Sorry to take so long to get to this one... I think I kinda just ignored it since it has "WIP" in the title. Should this PR be reviewed and attempted to be checked in yet?

@edwardneal - More conflicts to resolve. I'll have a look once that is done.

Thanks@paulmedynski and@benrr101. I'm planning to move these tests into the new project (and to verify coverage for SqlBuffer at the same time) so I'll pull this back to draft for now.