From 126cb65113f3b461dd2be9b2934b1d2293fffa9c Mon Sep 17 00:00:00 2001 From: Paul Romano Date: Tue, 23 Jan 2018 07:18:33 -0600 Subject: [PATCH] Don't check returncode in tests --- openmc/executor.py | 21 ++++++++++++++++++- tests/test_mg_convert/test_mg_convert.py | 7 ++----- .../test_mgxs_library_ce_to_mg.py | 13 ++++-------- tests/test_plot/test_plot.py | 3 +-- .../test_statepoint_restart.py | 10 +++------ tests/testing_harness.py | 16 +++++--------- 6 files changed, 35 insertions(+), 35 deletions(-) diff --git a/openmc/executor.py b/openmc/executor.py index 46ad53ec29..ada662a12f 100644 --- a/openmc/executor.py +++ b/openmc/executor.py @@ -45,6 +45,11 @@ def plot_geometry(output=True, openmc_exec='openmc', cwd='.'): cwd : str, optional Path to working directory to run in + Raises + ------ + subprocess.CalledProcessError + If the `openmc` executable returns a non-zero status + """ _run([openmc_exec, '-p'], output, cwd) @@ -67,6 +72,11 @@ def plot_inline(plots, openmc_exec='openmc', cwd='.', convert_exec='convert'): convert_exec : str, optional Command that can convert PPM files into PNG files + Raises + ------ + subprocess.CalledProcessError + If the `openmc` executable returns a non-zero status + """ from IPython.display import Image, display @@ -125,6 +135,11 @@ def calculate_volumes(threads=None, output=True, cwd='.', Path to working directory to run in. Defaults to the current working directory. + Raises + ------ + subprocess.CalledProcessError + If the `openmc` executable returns a non-zero status + See Also -------- openmc.VolumeCalculation @@ -171,8 +186,12 @@ def run(particles=None, threads=None, geometry_debug=False, MPI execute command and any additional MPI arguments to pass, e.g. ['mpiexec', '-n', '8']. - """ + Raises + ------ + subprocess.CalledProcessError + If the `openmc` executable returns a non-zero status + """ args = [openmc_exec] if isinstance(particles, Integral) and particles > 0: diff --git a/tests/test_mg_convert/test_mg_convert.py b/tests/test_mg_convert/test_mg_convert.py index 9d13714580..36bfe5338b 100755 --- a/tests/test_mg_convert/test_mg_convert.py +++ b/tests/test_mg_convert/test_mg_convert.py @@ -139,13 +139,10 @@ class MGXSTestHarness(PyAPITestHarness): if self._opts.mpi_exec is not None: mpi_args = [self._opts.mpi_exec, '-n', self._opts.mpi_np] - returncode = openmc.run(openmc_exec=self._opts.exe, - mpi_args=mpi_args) + openmc.run(openmc_exec=self._opts.exe, mpi_args=mpi_args) else: - returncode = openmc.run(openmc_exec=self._opts.exe) - - assert returncode == 0, 'OpenMC did not exit successfully.' + openmc.run(openmc_exec=self._opts.exe) sp = openmc.StatePoint('statepoint.' + str(batches) + '.h5') diff --git a/tests/test_mgxs_library_ce_to_mg/test_mgxs_library_ce_to_mg.py b/tests/test_mgxs_library_ce_to_mg/test_mgxs_library_ce_to_mg.py index aaa83a70db..7ae47636e6 100644 --- a/tests/test_mgxs_library_ce_to_mg/test_mgxs_library_ce_to_mg.py +++ b/tests/test_mgxs_library_ce_to_mg/test_mgxs_library_ce_to_mg.py @@ -36,13 +36,9 @@ class MGXSTestHarness(PyAPITestHarness): # Initial run if self._opts.mpi_exec is not None: mpi_args = [self._opts.mpi_exec, '-n', self._opts.mpi_np] - returncode = openmc.run(openmc_exec=self._opts.exe, - mpi_args=mpi_args) + openmc.run(openmc_exec=self._opts.exe, mpi_args=mpi_args) else: - returncode = openmc.run(openmc_exec=self._opts.exe) - - assert returncode == 0, 'CE OpenMC calculation did not exit' \ - 'successfully.' + openmc.run(openmc_exec=self._opts.exe) # Build MG Inputs # Get data needed to execute Library calculations. @@ -73,10 +69,9 @@ class MGXSTestHarness(PyAPITestHarness): # Re-run MG mode. if self._opts.mpi_exec is not None: mpi_args = [self._opts.mpi_exec, '-n', self._opts.mpi_np] - returncode = openmc.run(openmc_exec=self._opts.exe, - mpi_args=mpi_args) + openmc.run(openmc_exec=self._opts.exe, mpi_args=mpi_args) else: - returncode = openmc.run(openmc_exec=self._opts.exe) + openmc.run(openmc_exec=self._opts.exe) def _cleanup(self): super(MGXSTestHarness, self)._cleanup() diff --git a/tests/test_plot/test_plot.py b/tests/test_plot/test_plot.py index d484925cb4..1a99c24886 100644 --- a/tests/test_plot/test_plot.py +++ b/tests/test_plot/test_plot.py @@ -19,8 +19,7 @@ class PlotTestHarness(TestHarness): self._plot_names = plot_names def _run_openmc(self): - returncode = openmc.plot_geometry(openmc_exec=self._opts.exe) - assert returncode == 0, 'OpenMC did not exit successfully.' + openmc.plot_geometry(openmc_exec=self._opts.exe) def _test_output_created(self): """Make sure *.ppm has been created.""" diff --git a/tests/test_statepoint_restart/test_statepoint_restart.py b/tests/test_statepoint_restart/test_statepoint_restart.py index 2c1c444958..9c10551dea 100644 --- a/tests/test_statepoint_restart/test_statepoint_restart.py +++ b/tests/test_statepoint_restart/test_statepoint_restart.py @@ -50,14 +50,10 @@ class StatepointRestartTestHarness(TestHarness): # Run OpenMC if self._opts.mpi_exec is not None: mpi_args = [self._opts.mpi_exec, '-n', self._opts.mpi_np] - returncode = openmc.run(restart_file=statepoint, - openmc_exec=self._opts.exe, - mpi_args=mpi_args) + openmc.run(restart_file=statepoint, openmc_exec=self._opts.exe, + mpi_args=mpi_args) else: - returncode = openmc.run(openmc_exec=self._opts.exe, - restart_file=statepoint) - - assert returncode == 0, 'OpenMC did not exit successfully.' + openmc.run(openmc_exec=self._opts.exe, restart_file=statepoint) if __name__ == '__main__': diff --git a/tests/testing_harness.py b/tests/testing_harness.py index 428bf5c4b7..ad3b8ac1ec 100644 --- a/tests/testing_harness.py +++ b/tests/testing_harness.py @@ -62,14 +62,10 @@ class TestHarness(object): def _run_openmc(self): if self._opts.mpi_exec is not None: - returncode = openmc.run( - openmc_exec=self._opts.exe, - mpi_args=[self._opts.mpi_exec, '-n', self._opts.mpi_np]) - + openmc.run(openmc_exec=self._opts.exe, + mpi_args=[self._opts.mpi_exec, '-n', self._opts.mpi_np]) else: - returncode = openmc.run(openmc_exec=self._opts.exe) - - assert returncode == 0, 'OpenMC did not exit successfully.' + openmc.run(openmc_exec=self._opts.exe) def _test_output_created(self): """Make sure statepoint.* and tallies.out have been created.""" @@ -189,13 +185,11 @@ class ParticleRestartTestHarness(TestHarness): args['mpi_args'] = [self._opts.mpi_exec, '-n', self._opts.mpi_np] # Initial run - returncode = openmc.run(**args) - assert returncode == 0, 'OpenMC did not exit successfully.' + openmc.run(**args) # Run particle restart args.update({'restart_file': self._sp_name}) - returncode = openmc.run(**args) - assert returncode == 0, 'OpenMC did not exit successfully.' + openmc.run(**args) def _test_output_created(self): """Make sure the restart file has been created."""