Merge pull request #362 from billsacks/funitshare

Description of changes
Purposes of this test:

Ensure that the unit test infrastructure continues to work.
Confirm that the share unit tests continue to pass.
Requires the CIME changes in ESMCI/cime#4895.

Also requires share1.1.13 or later.

Specific notes
Contributors other than yourself, if any: none

Fixes: none

User interface changes?: No

Testing performed (automated tests and/or manual tests):

Ran the new test (FUNITSHARE_P1x1_Mmpi-serial.ne30pg3_t232.B1850C_LTso.derecho_intel) and confirmed that it passes.

Author: fischer-ncar

cime_config/SystemTests/funitshare.py

@@ -0,0 +1,33 @@
+"""
+Implementation of the share code FUNIT test.
+
+This "system" test runs the share code's Fortran unit tests. We're abusing the system test
+infrastructure to run these, so that a run of the test suite can result in the unit tests
+being run as well.
+
+Grid and compset are irrelevant for this test type.
+"""
+
+import os
+from CIME.SystemTests.funit import FUNIT
+from CIME.XML.standard_module_setup import *
+
+logger = logging.getLogger(__name__)
+
+
+class FUNITSHARE(FUNIT):
+    def __init__(self, case):
+        FUNIT.__init__(self, case)
+
+        # The CMake build for the share unit tests uses the MPISERIAL env var to find the
+        # mpi-serial library. On derecho, MPISERIAL is not set even if the mpi-serial
+        # module is loaded; instead, there is an NCAR_ROOT_MPI_SERIAL variable. Set the
+        # needed MPISERIAL env var for this test.
+        if "NCAR_ROOT_MPI_SERIAL" in os.environ:
+            os.environ["MPISERIAL"] = os.environ["NCAR_ROOT_MPI_SERIAL"]
+
+    def get_test_spec_dir(self):
+        return os.path.join(self._case.get_value("SRCROOT"), "share")
+
+    def get_extra_run_tests_args(self):
+        return '--cmake-args " -DUNITTESTS=ON -DUSE_CIME_MACROS=ON"'

cime_config/config_tests.xml

@@ -0,0 +1,18 @@
+<?xml version="1.0"?>
+
+<!--
+This defines various CESM-specific system tests
+-->
+
+<config_test>
+
+  <test NAME="FUNITSHARE">
+    <DESC>Run the share code's Fortran unit tests. Grid and compset (and most case settings) are ignored.</DESC>
+    <INFO_DBUG>1</INFO_DBUG>
+    <STOP_OPTION>ndays</STOP_OPTION>
+    <STOP_N>11</STOP_N>
+    <CHECK_TIMING>FALSE</CHECK_TIMING>
+    <DOUT_S>FALSE</DOUT_S>
+  </test>
+
+</config_test>

cime_config/testlist_allactive.xml

@@ -251,4 +251,13 @@
      <option name="comment">Include a short debug test using the atm grid for aoflux calculations rather than the default xgrid.</option>
    </options>
  </test>
+  <test name="FUNITSHARE_P1x1_Mmpi-serial" grid="ne30pg3_t232" compset="B1850C_LTso">
+    <machines>
+      <machine name="derecho" compiler="intel" category="prealpha"/>
+    </machines>
+    <options>
+      <option name="wallclock">00:30:00</option>
+      <option name="comment">This test runs the CESM_share Fortran unit tests. We're abusing the system test infrastructure to run these, so that a run of the test suite results in the unit tests being run as well. These unit tests are run as a GitHub action for each share PR, so this system test is less about catching code problems and more about making sure the unit testing infrastructure continues to work in each CESM tag. Grid and compset are irrelevant here. Needs to be mpi-serial in order to load the necessary modules.</option>
+    </options>
+  </test>
 </testlist>