comment out the automated answer-tests for the new-style Checkpoint-Restart functionality (until PR #313 is merged)

Author: mabruzzo

test/answer_tests/test_ckpt_restart.py

@@ -25,43 +25,45 @@
     os.path.abspath(os.path.join(os.path.dirname(__file__), "../../input"))
 )
 
-# This is the parameterized test for the old-style checkpoint-restart tests
+# This is the parameterized test for the new-style checkpoint-restart tests
 # - these use parameter files from input/Checkpoint directory (it should not
 #   consider stuff from the legacy subdirectory)
+#
+# The following should be uncommented once PR #313 is merged
 
-@pytest.mark.parametrize("nominal_input,uses_grackle",
-                         [('checkpoint_boundary.in', False),
-                          ('checkpoint_grackle.in', True),
-                          ('checkpoint_ppm.in', False)])
-def test_ckpt_restart(nominal_input, uses_grackle):
-    nominal_input = os.path.abspath(f'{_input_dir}/Checkpoint/{nominal_input}')
-
-    symlink_srcs = list(get_symlink_targets(grackle_files = uses_grackle))
-
-    # the following context manager class creates a temporary directory,
-    # stores the path to it in working_dir, and then always cleans up the
-    # contents (even if the program encounters an error) when we exit the
-    # context
-    #
-    # TODO: it would be REALLY nice to have a unified approach (shared across
-    #       all tests) for doing this, that can be configured to:
-    #       - name the temporary-directory in a deterministic manner after the
-    #         current test (maybe let the user specify an empty directory that
-    #         all tests get run inside of)
-    #       - conditionally decide whether to delete the temporary directory.
-    #         One could imagine 3 options:
-    #            1. Always delete the temporary directory (current behavior)
-    #            2. Only delete the temporary directory on test failure
-    #            3. Never delete the temporary directory
-    with tempfile.TemporaryDirectory() as working_dir:
-
-        run_ckpt_restart_test(os.path.join(_input_dir, nominal_input),
-                              working_dir = working_dir,
-                              enzoe_driver = cached_opts().enzoe_driver,
-                              nproc = 1, ckpt_cycle = 2, stop_cycle = 4,
-                              symlink_srcs = symlink_srcs,
-                              sim_name_prefix = None,
-                              buffer_outputs_on_disk = False)
+#@pytest.mark.parametrize("nominal_input,uses_grackle",
+#                         [('checkpoint_boundary.in', False),
+#                          ('checkpoint_grackle.in', True),
+#                          ('checkpoint_ppm.in', False)])
+#def test_ckpt_restart(nominal_input, uses_grackle):
+#    nominal_input = os.path.abspath(f'{_input_dir}/Checkpoint/{nominal_input}')
+#
+#    symlink_srcs = list(get_symlink_targets(grackle_files = uses_grackle))
+#
+#    # the following context manager class creates a temporary directory,
+#    # stores the path to it in working_dir, and then always cleans up the
+#    # contents (even if the program encounters an error) when we exit the
+#    # context
+#    #
+#    # TODO: it would be REALLY nice to have a unified approach (shared across
+#    #       all tests) for doing this, that can be configured to:
+#    #       - name the temporary-directory in a deterministic manner after the
+#    #         current test (maybe let the user specify an empty directory that
+#    #         all tests get run inside of)
+#    #       - conditionally decide whether to delete the temporary directory.
+#    #         One could imagine 3 options:
+#    #            1. Always delete the temporary directory (current behavior)
+#    #            2. Only delete the temporary directory on test failure
+#    #            3. Never delete the temporary directory
+#    with tempfile.TemporaryDirectory() as working_dir:
+#
+#        run_ckpt_restart_test(os.path.join(_input_dir, nominal_input),
+#                              working_dir = working_dir,
+#                              enzoe_driver = cached_opts().enzoe_driver,
+#                              nproc = 1, ckpt_cycle = 2, stop_cycle = 4,
+#                              symlink_srcs = symlink_srcs,
+#                              sim_name_prefix = None,
+#                              buffer_outputs_on_disk = False)
 
 
 # This is the parameterized test for the old-style checkpoint-restart tests