Merge pull request #1532 from NetCDF-World-Domination-Council/ejh_cmake_unit_tests

Adding rest of unit_tests to CMake build, but not running them on Windows

Author: WardF

CMakeLists.txt

@@ -927,8 +927,12 @@ IF(ENABLE_TESTS)
   OPTION(ENABLE_FAILING_TESTS "Run tests which are known to fail, check to see if any have been fixed." OFF)
 
   ###
-  # Option to turn on unit testing. See https://github.com/Unidata/netcdf-c/pull/1472 for more information.
-  # Currently (August 21, 2019): Will not work with Visual Studio
+  # Option to turn on unit testing. See
+  # https://github.com/Unidata/netcdf-c/pull/1472 for more
+  # information.  Currently (August 21, 2019): Will not work with
+  # Visual Studio. The unit tests are for internal netCDF functions,
+  # so we don't want to make them external, which would be required to
+  # run on Windows.
   ###
   IF(NOT MSVC)
     OPTION(ENABLE_UNIT_TESTS "Run Unit Tests." ON)

unit_test/CMakeLists.txt

@@ -10,7 +10,7 @@
 
 # Some unit testing
 
-SET(UNIT_TESTS tst_nclist)
+SET(UNIT_TESTS tst_nclist test_ncuri test_pathcvt)
 
 IF(ENABLE_NETCDF_4)
   SET(UNIT_TESTS ${UNIT_TESTS} tst_nc4internal)