Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.
/ corefx Public archive

Fix SqlClient diagnostic test failures #16349

Merged
merged 3 commits into from
Feb 22, 2017

Commits on Feb 22, 2017

  1. Fix SqlClient diagnostic test failures

    Two issues:
    - Tests from the DiagnosticTests class might run concurrently with tests from other test classes.  This could cause interference between the tests.
    - The CollectStatisticsDiagnosticsAsync method was void returning, which meant the test harness would not wait for its completion before processing other tests.  This could both cause interference between the tests (including other diagnostics tests) and cause unhandled exceptions to tear down the test process.
    
    Fixes:
    - Use RemoteInvoke to launch each diagnostic test in its own process so that it doesn't experience or cause interference with other tests.
    - Ensure that all operations are appropriately waited for.
    stephentoub committed Feb 22, 2017
    Configuration menu
    Copy the full SHA
    ad23153 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    160dad0 View commit details
    Browse the repository at this point in the history
  3. Tweak RemoteExecutorTestBase slightly

    Increase fail-wait timeout in case a machine is bogged down, and add a better assert message
    stephentoub committed Feb 22, 2017
    Configuration menu
    Copy the full SHA
    bb02c8f View commit details
    Browse the repository at this point in the history