Merge pull request #3661 from heplesser/helveg_3510

Simplify test implementation and remove complex reusable code for better readability (replacement for #3510)

Author: terhorstd

testsuite/pytests/conftest.py

@@ -31,7 +31,6 @@ def test_gsl():
         pass
 """
 
-import dataclasses
 import os
 import pathlib
 import subprocess
@@ -45,7 +44,6 @@ def test_gsl():
 # Ignore it during test collection
 collect_ignore = ["utilities"]
 
-import testsimulation  # noqa
 import testutil  # noqa
 
 
@@ -198,24 +196,3 @@ def skipif_incompatible_mpi(request, subprocess_compatible_mpi):
 
     if not subprocess_compatible_mpi and request.node.get_closest_marker("skipif_incompatible_mpi"):
         pytest.skip("skipped because MPI is incompatible with subprocess")
-
-
-@pytest.fixture(autouse=True)
-def simulation_class(request):
-    return getattr(request, "param", testsimulation.Simulation)
-
-
-@pytest.fixture
-def simulation(request):
-    marker = request.node.get_closest_marker("simulation")
-    sim_cls = marker.args[0] if marker else testsimulation.Simulation
-    sim = sim_cls(*(request.getfixturevalue(field.name) for field in dataclasses.fields(sim_cls)))
-    nest.ResetKernel()
-    if getattr(sim, "set_resolution", True):
-        nest.resolution = sim.resolution
-    nest.local_num_threads = sim.local_num_threads
-    return sim
-
-
-# Inject the root simulation fixtures into this module to be always available.
-testutil.create_dataclass_fixtures(testsimulation.Simulation, __name__)