Merge pull request #1553 from h-mayorquin/make_base_test_setup

Make  `BaseTestRawIO` setup per class and not per test

Author: zm711

neo/test/rawiotest/common_rawio_test.py

@@ -59,14 +59,15 @@ class BaseTestRawIO:
 
     local_test_dir = get_local_testing_data_folder()
 
-    def setUp(self):
+    @classmethod
+    def setUpClass(cls):
         """
-        Set up the test fixture.  This is run for every test
+        Set up the test fixture. This is run once before all tests.
         """
-        self.shortname = self.rawioclass.__name__.lower().replace("rawio", "")
+        cls.shortname = cls.rawioclass.__name__.lower().replace("rawio", "")
 
-        if HAVE_DATALAD and self.use_network:
-            for remote_path in self.entities_to_download:
+        if HAVE_DATALAD and cls.use_network:
+            for remote_path in cls.entities_to_download:
                 download_dataset(repo=repo_for_test, remote_path=remote_path)
         else:
             raise unittest.SkipTest("Requires datalad download of data from the web")

neo/test/rawiotest/test_maxwellrawio.py

@@ -1,5 +1,4 @@
 import unittest
-import os
 
 from neo.rawio.maxwellrawio import MaxwellRawIO, auto_install_maxwell_hdf5_compression_plugin
 from neo.test.rawiotest.common_rawio_test import BaseTestRawIO
@@ -9,6 +8,7 @@ class TestMaxwellRawIO(
     BaseTestRawIO,
     unittest.TestCase,
 ):
+
     rawioclass = MaxwellRawIO
     entities_to_download = ["maxwell"]
     entities_to_test = files_to_test = [
@@ -18,7 +18,6 @@ class TestMaxwellRawIO(
 
     def setUp(self):
         auto_install_maxwell_hdf5_compression_plugin(force_download=False)
-        BaseTestRawIO.setUp(self)
 
 
 if __name__ == "__main__":