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

Improve azure devops error reporting (for us) #5187

Open
nohwnd opened this issue Mar 7, 2025 · 4 comments
Open

Improve azure devops error reporting (for us) #5187

nohwnd opened this issue Mar 7, 2025 · 4 comments

Comments

@nohwnd
Copy link
Member

nohwnd commented Mar 7, 2025

Summary

I spend endless time looking for the right log in azdo to see where my errors are and what is their content. I need to:

  • click on the PR
  • go to build report
  • go click some of the builds
  • click the link to azdo build
  • click on the build number to land on the main page
  • click on tests, and hope it was not a crash that killed the run
  • look at errors that are reported
  • if there is a crash, look for the log and find where the log is, hope it is uploaded
  • then hope that the error message has enough information / assertion failure is not just Assert.True expected true but was false.

I would much prefer to see the errors directly in the source code, under the line that failed.

Image

I am not sure if this is problem with out arcade based setup, with MTP of mix of both. I do think that VSTest did this, but I don't see any reference to the ## api in vstest code, so maybe I am imagining stuff or it was vstesttask post processing TRX.

https://grep.app/search?f.repo.pattern=vstest&q=logissue

We do have similar integration in Pester pester/Pester#1996 where it works well.

https://github.com/pester/Pester/blame/main/src/functions/Output.ps1

@nohwnd
Copy link
Member Author

nohwnd commented Mar 17, 2025

https://github.com/nohwnd/EvertythingIsBroken/pull/1/files started on it here.

  • paths need to be relative and Linuxed.
  • vso task issue needs to be reported for it to show up in the PR.
  • newlines need to be escaped to show up correctly.

@nohwnd
Copy link
Member Author

nohwnd commented Mar 17, 2025

multiline error:
Image

unchanged file:
Image

@nohwnd
Copy link
Member Author

nohwnd commented Mar 17, 2025

@nohwnd
Copy link
Member Author

nohwnd commented Mar 17, 2025

hardcoded into terminal reporter but this is not the desired state ofc. should be an extension

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

1 participant