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 skipped tests #7783

Merged
merged 1 commit into from
Jan 6, 2016
Merged

Enable skipped tests #7783

merged 1 commit into from
Jan 6, 2016

Conversation

tmat
Copy link
Member

@tmat tmat commented Jan 5, 2016

Fixes #6523

@tmat
Copy link
Member Author

tmat commented Jan 5, 2016

Known issue #7618.

@tmat tmat force-pushed the SkippedTests branch 2 times, most recently from 9cd823c to b95c527 Compare January 5, 2016 18:43
@amcasey
Copy link
Member

amcasey commented Jan 5, 2016

Microsoft.CodeAnalysis.CSharp.Scripting.Hosting.UnitTests.CsiTests.CurrentWorkingDirectory_Change (from Microsoft.CodeAnalysis.CSharp.Scripting.Desktop.UnitTests)

If I'm reading the log correctly, this was the unexpected output:

(1,1): error CS1056: Unexpected character '�'
(1,2): error CS1056: Unexpected character '�'
(1,3): error CS1056: Unexpected character '�'
(1,4): error CS1040: Preprocessor directives must appear as the first non-whitespace character on a line

@amcasey
Copy link
Member

amcasey commented Jan 5, 2016

The unexpected character appears to be 0xFFFD, REPLACEMENT CHARACTER.

tmat added a commit that referenced this pull request Jan 6, 2016
@tmat
Merge pull request #7783 from tmat/SkippedTests
48e1bfa 
Enable skipped tests
@tmat tmat merged commit 48e1bfa into dotnet:master Jan 6, 2016
@tmat tmat deleted the SkippedTests branch January 6, 2016 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cla-already-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tmat @amcasey @dnfclas