Make BaseTestRawIO setup per class and not per test

[h-mayorquin]

Related to the discussion in #1550

[zm711]

Looks like maxwell testing relies directly on setUp? Do you want to dig into or you want me to?

FAILED neo/test/rawiotest/test_maxwellrawio.py::TestMaxwellRawIO::test_read_all - AttributeError: type object 'BaseTestRawIO' has no attribute 'setUp'

[h-mayorquin]

Fixed.

[zm711]

@nikhilchandra, want to test to see if this fixes your issues using the testing locally?

[h-mayorquin]

I want to add that I think this is a good improvement on its own for tests with multiple cases so the download is not attempted twice anyway.

[zm711]

I want to add that I think this is a good improvement on its own for tests with multiple cases so the download is not attempted twice anyway.

I agree. I was hoping for a magic speed-up. But it looks like we go from 7-8 min to 6-7 min. So a slight (although not statistically tested) improvement in test speed.

Sam can confirm he is okay with it tomorrow either way and we can merge then.

[h-mayorquin]

1 minute is great : )

[nikhilchandra]

@zm711 will give it a shot in a bit, thanks!

[samuelgarcia]

Genial