-
Notifications
You must be signed in to change notification settings - Fork 453
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Correct DeleteWithOutputAsync() function name #2967
Conversation
Well, looks like we do not have regression test for this function ;) |
pish posh. 😁 |
And missed again. Tested on Sync version. |
Yes, all of the tests were only done w/ the sync version. Only added sync to match the other sync tests. I can double all of the tests for the async versions if you want? Seems somewhat extraneous, but... |
Believe me it worth efforts. We can accidentally change something and async versions may fail. |
PS: Blame @MaceWindu - he's the one who wrote the InsertWithOutput tests that I copied from originally... 😉 |
It's me ;) Really messed up with count of new methods. |
There. Punk... :) |
/azp run test-all |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tests fail, probably something wrong with asserts
/azp run test-all |
Azure Pipelines successfully started running 1 pipeline(s). |
Test baselines changed by this PR. Don't forget to merge/close baselines PR after this pr merged/closed. |
* [Windows / NET472 / SQL Server 2019 (Microsoft.Data.SqlClient)] baselines * [Windows / NET472 / SQL Server 2008 (System.Data.SqlClient)] baselines * [Windows / NET472 / SQL Server 2017 (System.Data.SqlClient)] baselines * [Windows / NET472 / SQL Server 2014 (System.Data.SqlClient)] baselines * [Windows / NET472 / SQL Server 2012 (System.Data.SqlClient)] baselines * [Windows / NETCOREAPP2.1 / SQL Server 2012 (System.Data.SqlClient)] baselines * [Windows / NETCOREAPP2.1 / SQL Server 2014 (System.Data.SqlClient)] baselines * [Linux / NET5.0 / SQL Server 2019 (Microsoft.Data.SqlClient)] baselines * [Windows / NET 5.0 / SQL Server 2019 (Microsoft.Data.SqlClient)] baselines * [Windows / NETCOREAPP2.1 / SQL Server 2016 (System.Data.SqlClient)] baselines * [Windows / NETCOREAPP2.1 / SQL Server 2008 (System.Data.SqlClient)] baselines * [Windows / NETCOREAPP2.1 / SQL Server 2017 (System.Data.SqlClient)] baselines * [Windows / NET472 / SQL Server 2016 (System.Data.SqlClient)] baselines Co-authored-by: Azure Pipelines Bot <azp@linq2db.com>
No description provided.