Test tidy

While implementing the last few features I've found it increasingly annoying to have to write tests in the current style. This PR addresses most of those annoyances, leaving the tests short and readable.

There is one more change I would like to make but I thought it contantious so leave it to your judgement. There are javadocs all over the test classes that really don't add any value in my opinion. I know its a pain in Java compared to many languages but the test (method) name really should convey most of what is necessary to understand the subsequent code. Given the enhanced code readability , can we delete those javadocs?

PS. I made no changes to the sourcegen tests.