On 11/12/2016 05:53 AM, Luis Machado wrote:
This fixes the offender testcases that have test names starting with
uppercase.
I also tried to catch cases where we set the test names via variables.
We should probably adopt a policy and stick with it. If the way to go is
all lowercase, then the following patch should hopefully help clean some
of the inconsistencies up. Otherwise, we should go the other way and adjust
the test names to start with uppercase.
I agree with lowercase. It's what we use most pervasively, and I've
asked people to lowercase messages in review in the past. The fewer
(bad) examples people can crib from, the better. So thanks for doing this.