Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Enable tests for code fix behavior with alternate indentation #1135

Merged
merged 2 commits into from
Aug 6, 2015
Merged

Enable tests for code fix behavior with alternate indentation #1135

merged 2 commits into from
Aug 6, 2015

Commits on Aug 6, 2015

  1. Enable tests for code fix behavior with alternate indentation

    @sharwell
    sharwell committed Aug 6, 2015
    Configuration menu
    Copy the full SHA
    6c9d1af View commit details
    Browse the repository at this point in the history

  2. Disable test parallelization due to shared state problems 

    The per-language options used by the AdhocWorkspace are currently shared.
This is caused by the default constructor of AdhocWorkspace using
MefHostServices.DefaultHost to provide workspace services, combined with
Workspace.WithChangedOption setting values back to the host.

In theory, all tests which run using the default options could be run in
parallel, and only serialize the tests which use custom options. However,
we don't have a mechanism in place to handle this currently, so disabling
test parallelization allows us to improve overall test quality immediately
(by testing alternate indentation options in some scenarios).
    @sharwell
    sharwell committed Aug 6, 2015
    Configuration menu
    Copy the full SHA
    5b2cbf8 View commit details
    Browse the repository at this point in the history