Test explorer does not handle panics or timeouts well

[firelizzard18]

What version of Go, VS Code & VS Code Go extension are you using?

• go version go1.17.1 linux/amd64
• Nightly vscode-go v2021.10.2813

Describe the bug

When the user runs a test or tests in the test explorer and one of those tests panics or times out, all tests that have not already completed, including the one that panicked or timed out, are reported as skipped.

This is a problem with GoTestRunner.

Steps to reproduce the behavior:

1. Create a test that panics or times out
2. Run the test
3. Tests show as skipped

Screenshots or recordings

[gopherbot]

Change https://golang.org/cl/373214 mentions this issue: src/goTest: fail tests if the package fails

[mgabeler-lee-6rs]

Anything that can be done to move this along? Got super confused & then frustrated by this problem recently.

I see @hyangah had some questions, but @firelizzard18 appears to have answered them ... but then also noted the CL as abandoned.

[firelizzard18]

@mgabeler-lee-6rs My day job leaves me with very little capacity to make contributions to other projects. If someone wants to submit a new CL based on my work, I would support that.

[mgabeler-lee-6rs]

Understood. I was more hoping the maintainers of this project would be able to pick things up and move this forwards :)

[firelizzard18]

My original solution (in that CL) was kind of a hack. It may be more feasible to fix this properly once golang/go#62728 has been resolved. Though I'm not sure that will handle panics well. But we might be able to add a different output type for panics.