Codecov Report

Merging #5413 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #5413      +/-   ##
==========================================
- Coverage   95.98%   95.98%   -0.01%     
==========================================
  Files         114      114              
  Lines       25523    25511      -12     
  Branches     2480     2479       -1     
==========================================
- Hits        24499    24487      -12     
  Misses        718      718              
  Partials      306      306

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 450d264...65c2a81. Read the comment docs.

I think having the location there is useful in general, but should come via ExceptionInfo then instead - if wanted.

You mean adding this to ExceptionInfo.__repr__?

entry = self.traceback[-1]
loc = ReprFileLocation(entry.path, entry.lineno + 1, self.exconly())

You mean adding this to ExceptionInfo.__repr__?

Yes - or something similar.

Adding the location gives this:

<ExceptionInfo ValueError tblen=2 location=C:\Users\bruno\pytest\testing\code\test_excinfo.py:338: ValueError: an error value>

Isn't that a too long?

Isn't that a too long?

Maybe, yes. But you have the exception type there twice.
I think it is fine for now as of this PR.