remove openmpi tests

Author: jan-janssen

.github/workflows/pipeline.yml

@@ -296,38 +296,6 @@ jobs:
         python -m unittest test_slurmclusterexecutor.py
         python -m unittest test_slurmjobexecutor.py
 
-  unittest_slurm_openmpi:
-    needs: [black]
-    runs-on: ubuntu-latest
-    services:
-      mysql:
-        image: mysql:8.0
-        env:
-          MYSQL_ROOT_PASSWORD: root
-        ports:
-          - "8888:3306"
-        options: --health-cmd="mysqladmin ping" --health-interval=10s --health-timeout=5s --health-retries=3
-    steps:
-    - uses: actions/checkout@v4
-    - uses: koesterlab/setup-slurm-action@v1
-    - name: Conda config
-      shell: bash -l {0}
-      run: echo -e "channels:\n  - conda-forge\n" > .condarc
-    - uses: conda-incubator/setup-miniconda@v3
-      with:
-        python-version: '3.13'
-        miniforge-version: latest
-        condarc-file: .condarc
-        environment-file: .ci_support/environment-openmpi.yml
-    - name: Test
-      shell: bash -l {0}
-      timeout-minutes: 5
-      run: |
-        pip install . --no-deps --no-build-isolation
-        cd tests
-        python -m unittest test_slurmclusterexecutor.py
-        python -m unittest test_slurmjobexecutor.py
-
   unittest_mpich:
     needs: [black]
     runs-on: ${{ matrix.operating-system }}
@@ -464,7 +432,7 @@ jobs:
           GH_TOKEN: ${{secrets.GITHUB_TOKEN}}
 
   uml:
-    needs: [unittest_slurm_mpich, unittest_slurm_openmpi, unittest_old, unittest_win, unittest_openmpi, unittest_mpich, unittest_flux_openmpi, unittest_flux_mpich, notebooks, benchmark, minimal, pip_check, mypy]
+    needs: [unittest_slurm_mpich, unittest_old, unittest_win, unittest_openmpi, unittest_mpich, unittest_flux_openmpi, unittest_flux_mpich, notebooks, benchmark, minimal, pip_check, mypy]
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v4

tests/test_slurmjobexecutor.py

@@ -39,7 +39,6 @@ def set_global():
 class TestSlurmBackend(unittest.TestCase):
     def test_slurm_executor_serial(self):
         with SlurmJobExecutor(
-            max_cores=2,
             block_allocation=True,
         ) as exe:
             fs_1 = exe.submit(calc, 1)
@@ -51,7 +50,6 @@ def test_slurm_executor_serial(self):
 
     def test_slurm_executor_serial_no_depencies(self):
         with SlurmJobExecutor(
-            max_cores=2,
             block_allocation=True,
             disable_dependencies=True,
         ) as exe:
@@ -64,7 +62,6 @@ def test_slurm_executor_serial_no_depencies(self):
 
     def test_slurm_executor_threads(self):
         with SlurmJobExecutor(
-            max_cores=1,
             resource_dict={"threads_per_core": 2},
             block_allocation=True,
         ) as exe:
@@ -77,7 +74,6 @@ def test_slurm_executor_threads(self):
 
     def test_slurm_executor_parallel(self):
         with SlurmJobExecutor(
-            max_cores=2,
             resource_dict={"cores": 2},
             block_allocation=True,
         ) as exe:
@@ -87,7 +83,6 @@ def test_slurm_executor_parallel(self):
 
     def test_single_task(self):
         with SlurmJobExecutor(
-            max_cores=2,
             resource_dict={"cores": 2},
             block_allocation=True,
         ) as p:
@@ -99,7 +94,6 @@ def test_single_task(self):
 
     def test_internal_memory(self):
         with SlurmJobExecutor(
-            max_cores=1,
             resource_dict={"cores": 1},
             init_function=set_global,
             block_allocation=True,
@@ -108,11 +102,3 @@ def test_internal_memory(self):
             self.assertFalse(f.done())
             self.assertEqual(f.result(), np.array([5]))
             self.assertTrue(f.done())
-
-    def test_validate_max_workers(self):
-        with self.assertRaises(ValueError):
-            SlurmJobExecutor(
-                max_workers=10,
-                resource_dict={"cores": 10, "threads_per_core": 10},
-                block_allocation=True,
-            )