arcade icon indicating copy to clipboard operation
arcade copied to clipboard

User study feedback: Would prefer to see more test failures than shared test failures

Open missymessa opened this issue 2 years ago • 5 comments

Customers would rather see more of the test failures on that are specific to their PRs rather than having a truncated subset and then a section of "shared" test failures.

missymessa avatar Sep 19 '22 19:09 missymessa

Yeah... the shared concept is... odd... especially now that we have known issues. Should we remove the "shared with target branch" functionality altogether, and let known issues handle that? That seems to be what the product teams are heading toward anyway... and it's yet another sort of thing here... We have like 5 different ways to deal with "not my fault" failures, and they all render differently, and it's really confusing.

  • Build retry
  • Local Test Retry
  • Known Issues
  • Shared Test with Target Branch
  • Infrastructure issue

I can see how our customers might be confused.

ChadNedzlek avatar Sep 19 '22 19:09 ChadNedzlek

That's an interesting idea @ChadNedzlek. Maybe we float this on the teams channel and a few known suspects?

markwilkie avatar Sep 19 '22 19:09 markwilkie

It does seem likely that as Known Issues use grows, the "Shared Test Failures" section may disappear, except on the first instance and at that point, someone's going to file a Known Issue on it.

missymessa avatar Sep 19 '22 19:09 missymessa

Yeah, so it feels like wasting space differentiating them might be more confusing than providing value at this point. It made a lot of sense before we had known issues, but those seem to be the hot ticket these days, so it makes sense to pivot to make those more central.

ChadNedzlek avatar Sep 19 '22 19:09 ChadNedzlek

Based on conversation in the Team's channel, we may remove the Shared Test Failure section entirely, but we want to have an understanding of how often it shows up. (So we need to get those metrics). Ideally, we're thinking that Known Issues will replace the need for Shared Test Failures, so that section will be unnecessary.

missymessa avatar Sep 23 '22 18:09 missymessa

Work item: Remove shared test failure section.

missymessa avatar Sep 27 '22 17:09 missymessa

PR: https://dev.azure.com/dnceng/internal/_git/dotnet-helix-service/pullrequest/26328

AlitzelMendez avatar Sep 29 '22 16:09 AlitzelMendez