👍

👍

Eek, actually, I think this adds back some mid-test-run noise that @iainbeeston just removed cuz skip doesn't work on 1.8. Can you stick to just omitting the defined method completely?

Good point @pd. The problem with using skip is that

1. For such a large number of tests it's very noisy ( and makes it hard to spot errors)
2. It isn't supported on the test unit that comes with 1.8.7

I just (naively) pulled the skip condition up and out of the loop.

Otherwise this looks good!

Extra commit added to not define test methods for ignored tests. I added the skipping originally because @pd wanted it. I could make it skip on 1.9+ and not define on 1.8, if y'all can come to a consensus on whether skipping is desired behaviour or not.

My vote would be for not using skip, because it makes the test run incredibly noisy (not sure if there's a way to make test unit not print out all the details for every skipped test? because I agree that skip would be more correct than not running it)

I fully agree with @iainbeeston