DUnitX icon indicating copy to clipboard operation
DUnitX copied to clipboard

Add argument 'ATrimValues: Boolean = False' to DUnitX.Attributes.TestCaseAttribute.Create

Open Nashev opened this issue 2 years ago • 1 comments

#191 very very annoying, it makes impossible to make test cases readable and comparable between each others. Please add a possibility to add spaces into the argument's list between values!

Add an argument ATrimValues: Boolean = False to the constructor DUnitX.Attributes.TestCaseAttribute.Create.

For backward compatibility I suggest to make it off by default, so we will need write a , True for every single mention of [TestCase()] declaration, but it is the lesser of the evils.

Nashev avatar Jul 12 '23 09:07 Nashev

Oops, I went to make PR, and discovered an ATrimValues argument already exists )))

I haven't seen it in a copy, provided with a Delphi 11 at "C:\Program Files (x86)\Embarcadero\Studio\22.0\source\DunitX\DUnitX.Attributes.pas"

So, are there any best practices or pitfalls on the way to replacing the files from the distribution with the latest version?

Nashev avatar Jul 12 '23 09:07 Nashev