summary: align even large values #402

[dankamongmen]

As requested by the issue template, I have not changed doctest/doctest.h directly, but instead only doctest/doctest.cpp from which it is said to be generated (btw, the issue template refers to "two headers in doctest/parts", but only one of these has a .h extension. Perhaps "two files"?).

Description

When there are one million or more tests/asserts, the summary output gets out of wack (see #402). This fixes it in my tests. In addition, the output now uses only the necessary space for the total/passed/failed columns. If this last is undesirable, just add a ', 6' to the various std::max() parameters (or request that I do so).

before:

===============================================================================
[doctest] test cases:     36 |     36 passed |      0 failed |      1 skipped
[doctest] assertions: 8509223 | 8509223 passed |      0 failed |
[doctest] Status: SUCCESS!

after:

[schwarzgerat](0) $ ./notcurses-tester -p ../data/
Running with TERM=vte-256color
[doctest] doctest version is "2.4.0"
[doctest] run with "--help" for options
DirectMode *italic*!
DirectMode *bold*!
DirectMode *underline*!
===============================================================================
[doctest] test cases:      36 |      36 passed | 0 failed |      1 skipped
[doctest] assertions: 8509223 | 8509223 passed | 0 failed |
[doctest] Status: SUCCESS!
[schwarzgerat](0) $ 

GitHub Issues

Closes #402.

[dankamongmen]

I have updated all tests affected by this change, which now pass. PTAL.

[dankamongmen]

friendly ping! =]

[onqtam]

Sorry for the huuuuUUuuge delay! I was busy July/August but I didn't get a notification for your pings - I guess I should have noted in a comment that it might take me a while to review this - will adopt this practice in the future. I'll take a look at this now, thanks!

[dankamongmen]

Sorry for the huuuuUUuuge delay! I was busy July/August but I didn't get a notification for your pings - I guess I should have noted in a comment that it might take me a while to review this - will adopt this practice in the future. I'll take a look at this now, thanks!

no worries at all man!