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

Compressed test results #2040

Closed
adamralph opened this issue Jun 1, 2019 · 4 comments
Closed

Compressed test results #2040

adamralph opened this issue Jun 1, 2019 · 4 comments

Comments

@adamralph
Copy link
Contributor

Description

The test results output in the command line spans several lines and considerably bloats the line count, especially when multiple test runs form part of a single build.

Steps to reproduce

Run any test project from the command line.

Expected behavior

Something like:

Test run successful. Total tests: 63 (passed: 63, failed: 0, skipped: 0). Total time: 1.6310 seconds.

Actual behavior

Test Run Successful.
Total tests: 63
     Passed: 63
 Total time: 1.6310 Seconds

Environment

Windows 10, dotnet 2.2.300.

@ShreyasRmsft
Copy link
Member

@adamralph we arrived with this format after much thought and it is a lot more readable than the compressed form.

However the vstestplatform is extremely extensible and you have the option to extend the existing logger and create a custom version of it that you use for your test runs.

https://blogs.msdn.microsoft.com/vikramagrawal/2012/07/26/writing-loggers-for-command-line-test-runner-vstest-console-exe/

Existing console logger that you can use as a base:
https://github.com/microsoft/vstest/blob/master/src/vstest.console/Internal/ConsoleLogger.cs

@adamralph
Copy link
Contributor Author

@ShreyasRmsft how was this decision made? Was it discussed publicly?

I see that this is a recent thing, since older versions of the SDK have a more compact output.

@ShreyasRmsft
Copy link
Member

@cltshivash @singhsarab to give reasoning behind the change.

@singhsarab
Copy link
Contributor

@adamralph Here is the link to the issue #1836

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

No branches or pull requests

3 participants