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.

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

Enable skipped tests and fix them #21335

Merged
merged 1 commit into from
Aug 8, 2017
Merged

Enable skipped tests and fix them #21335

merged 1 commit into from
Aug 8, 2017

Conversation

jcouv
Copy link
Member

@jcouv jcouv commented Aug 5, 2017

Some tests were missing a [Fact], but when the attribute was added, the tests failed and thus were skipped.
This PR enables those tests and fixes them.

Fixes #21079

@jcouv jcouv added this to the 15.5 milestone Aug 5, 2017
@jcouv jcouv self-assigned this Aug 5, 2017
@jcouv jcouv added the Test Test failures in roslyn-CI label Aug 6, 2017
@jcouv
Copy link
Member Author

jcouv commented Aug 7, 2017

AutomaticStringLiteralCompletionTests.TestProprocessor_Backspace failed in windows_debug_unit32_prtest (details)

@jcouv
Copy link
Member Author

jcouv commented Aug 7, 2017

test windows_debug_unit32_prtest please

@jcouv
Copy link
Member Author

jcouv commented Aug 7, 2017

test windows_debug_vs-integration_prtest please

1 similar comment
@jcouv
Copy link
Member Author

jcouv commented Aug 7, 2017

test windows_debug_vs-integration_prtest please

@jcouv
Copy link
Member Author

jcouv commented Aug 7, 2017

windows_release_vs-integration_prtest failed with 262 failures on a test-only PR (details)

@dotnet/roslyn-infrastructure Is this a known issue?


Test Result (262 failures / ±0)Roslyn.VisualStudio.IntegrationTests.Workspace.WorkspacesDesktop.RenamingOpenFilesRoslyn.VisualStudio.IntegrationTests.Workspace.WorkspacesNetCore.RenamingOpenFilesRoslyn.VisualStudio.IntegrationTests.Basic.BasicEncapsulateField.EncapsulateThroughCommand
--

@jcouv
Copy link
Member Author

jcouv commented Aug 7, 2017

test windows_release_vs-integration_prtest please

@jcouv
Copy link
Member Author

jcouv commented Aug 7, 2017

@dotnet/roslyn-compiler for review. Thanks

@cston
Copy link
Member

cston commented Aug 7, 2017

LGTM

@jcouv jcouv merged commit 6a2133c into dotnet:master Aug 8, 2017
@jcouv
Copy link
Member Author

jcouv commented Aug 8, 2017

Merged with single review (test-only change). Thanks

@jcouv jcouv deleted the skipped-tests branch August 8, 2017 00:07
333fred added a commit to 333fred/roslyn that referenced this pull request Aug 10, 2017
…-literal-text

* dotnet/features/ioperation:
  Fix NamedArgumentInParameterOrderWithDefaultValue test for new IOperation output.
  EnC and EE cleanup (dotnet#21226)
  Fix crash when encountering a parenthesized expression when converting an if to a switch.
  this makes OOM to crash OOP process. this won't crash VS, instead it will show info bar and notify users to close VS and re-open.
  Turn on ref assemblies in all projects (dotnet#21346)
  Re-baseline some emit tests
  Move MakeFrames logic into Analysis
  Use langver=latest for scripting (dotnet#21331)
  Enable skipped tests and fix them (dotnet#21335)
  Replace project reference with linked file
  removed left out from deleted esent code.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compiler tests that were missing [Fact] don't all pass
3 participants