tests: consolidate sync and async variations of MsSql integration tests #2095
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To help reduce execution time of the MsSql integration tests, this PR combines the sync and async versions of tests. A few overall points to note:
SqlId
which is based on the command/statement text. In order to get traces for both sync and async operations, a different table name is used for some sync vs. async operations, and the SELECT command text is altered (query by first vs last name) for the operations against the pre-existingteammembers
table