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

Show total tests as "Unknown" when test run is aborted. #979

Closed
smadala opened this issue Aug 4, 2017 · 3 comments
Closed

Show total tests as "Unknown" when test run is aborted. #979

smadala opened this issue Aug 4, 2017 · 3 comments

Comments

@smadala
Copy link
Contributor

smadala commented Aug 4, 2017

Description

  • When test were aborted the total number of tests shown as number of tests ran till that point of time.
  • Stdout won't show any tests failed , where as stderr show as Test Run Failed
  • Having no error or failure message in stdout confuses the customer who redirect the stderr and stdout different files.

Steps to reproduce

  • Create a test project with 3 tests with one as stackoverflow test.
  • Run the tests.

Expected behavior

  • Total tests: Unknown. Passed: 2. Failed: 0. Skipped: 0.?

Actual behavior

  • Stdout show as Total tests: 2. Passed: 2. Failed: 0. Skipped: 0.
  • Actual number tests are 3. and one is failed.
@codito codito changed the title Show total tests as "Unkownn" when test run is aborted. Show total tests as "Unknown" when test run is aborted. Aug 4, 2017
@pvlakshm
Copy link
Contributor

pvlakshm commented Aug 9, 2017

This would be nice to have. However, any resulting confusion is because of an intentional action by the user to redirect stdout and stderr. Therefore I am inclined to wait for more feedback on this issue to come in before we decide on the next steps.

@harshjain2
Copy link
Contributor

@Faizan2304 @pvlakshm It would be good, if we can explicitly display that the test run is aborted or cancelled by user.
To me, Unknown sounds a bit incorrect.

Displaying Total tests as unknown with Test run aborted/cancelled message would be more meaningful.

@Faizan2304
Copy link
Contributor

@harshjain2

Displaying Total tests as unknown with Test run aborted/cancelled message would be more meaningful.

We are displaying Total tests: Unknown and then displaying aborted/cancelled message.
Please see attached screen shot below

image

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

5 participants