Fix to Display Width Tests by Oddley · Pull Request #278 · commandlineparser/commandline
Explicitly passing MaximumDisplayWidth = 80 to tests that fail unless your Maximum Display Width is 80
… your Maximum Display Width is 80
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good. I've run into this problem before with the tests and display width being different on my machine, vs appveyor, and another user found out that running inside a default docker container would get different results as well.
I did some testing - looks like this issue actually got fixed a while back. It works as of 2.4 and later.
Fails on 2.3 and earlier with exception System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values. (Parameter 'value')
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters