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

Run omnisharp test commands from vscode #35

Closed
wants to merge 3 commits into from

Conversation

rawman
Copy link

@rawman rawman commented Feb 11, 2016

Hi @jrieken
First I would like to thank you for a great extension, I love it!
This PR introduces running omnisharp test commands inside vscode output channel. There are three commands:

  • Run Single Test (Ctrl+R, R)
  • Run Fixture Tests (Ctrl+R, F)
  • Run All Test (Ctrl+R, A)

How does it work:
Test output is displayed in standard output channel like any other task. Additionally test execution summary i.e. Total: 2, Errors: 0, Failed: 1, Skipped: 0, Time: 0.110s is displayed in status bar. If one of tests fails, output channel is opened at the end of execution, otherwise not so you are not disturbed when tests went smoothly. Currently XUnit and NUnit output format is supported. Those formats right now are hardcoded which I don't like too much, but I don't see better way to understand test output.

Let me know what do you think. I'm starting to play with TypeScript so I hope I didn't make any horrible things ;)

@DustinCampbell
Copy link
Member

Hi @rawman! Thanks for the PR! The C# team (e.g. https://github.com/dotnet/roslyn) now owns this extension as the official C# experience in Visual Studio Code. We're currently working on adding support to make the extension target .NET Core and have some pretty substantial experience changes coming soon. Please stay tuned.

@DustinCampbell
Copy link
Member

@rawman, unfortunately, we won't be able to take this PR, but we really appreciate you taking the time to submit. We will be investing in the testing experience for this extension in the coming months, so you'll definitely see improvements there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants