Make unit test conventions consistent

[gep13]

Some unit test have the method format ThisIsATest() and others This_Is_A_Test().

[gep13]

@akordowski @AdmiringWorm I have no real opinion on this either way. Which one would we prefer?

[akordowski]

I prefer This_Is_A_Test(). On long method names this is better readable.

[gep13]

@akordowski this comment isn't related to this issue, is it?

[AdmiringWorm]

I agree with @akordowski in this. Having underscores in the unit test names make them more readable, but this would require a change in the StyleCop rules (I think).

[gep13]

So, sounds like This_Is_A_Test() is the winner. No objections from me in adopting this.