From 61e889c0be1e7e2f58fd14285289d14ce6e7bc0f Mon Sep 17 00:00:00 2001
From: Dana Singh <Dana.Singh@noaa.gov>
Date: Wed, 18 Dec 2024 16:07:40 -0500
Subject: [PATCH]  #224 Add more compile failure tests

---
 fre/make/tests/test_create_compile.py | 54 +++++++++++++++++++++++----
 1 file changed, 46 insertions(+), 8 deletions(-)

diff --git a/fre/make/tests/test_create_compile.py b/fre/make/tests/test_create_compile.py
index e9f2a42e..cd57c918 100644
--- a/fre/make/tests/test_create_compile.py
+++ b/fre/make/tests/test_create_compile.py
@@ -3,10 +3,11 @@
 """
 import os
 import shutil
+import pytest
 from pathlib import Path
 from fre.make import create_compile_script
 
-# SET-UP
+## SET-UP
 TEST_DIR = Path("fre/make/tests")
 NM_EXAMPLE = Path("null_example")
 YAMLFILE = "null_model.yaml"
@@ -14,6 +15,13 @@
 TARGET = ["debug"]
 EXPERIMENT = "null_model_full"
 
+# Multi-plat-targ
+MULTI_TARGET = ["prod","repro"]
+
+# Bad plat/targ
+BAD_PLATFORM=["no_plat"]
+BAD_TARGET=["no_targ"]
+
 # Create output location
 OUT = f"{TEST_DIR}/compile_out"
 if Path(OUT).exists():
@@ -57,8 +65,7 @@ def test_compile_creation():
     create_compile_script.compile_create(yamlfile_path, PLATFORM, TARGET, 4, 1, False, False)
 
     # Check for creation of compile script
-    # Check for correct default HOME location set
-    assert [Path(f"{OUT}/fremake_Canopy/test/null_model_full/{plat}-{targ}/exec/compile.sh").exists()]
+    assert Path(f"{OUT}/fremake_canopy/test/null_model_full/{plat}-{targ}/exec/compile.sh").exists()
 
 def test_compile_execution():
     """
@@ -74,9 +81,40 @@ def test_compile_execution():
     # Check for creation of compile script
     # Check for FMS directory
     # Check for log.compile file
-    # Check for correct default HOME location set
-    assert [Path(f"{OUT}/fremake_Canopy/test/null_model_full/{plat}-{targ}/exec/compile.sh").exists(),
-            Path(f"{OUT}/fremake_Canopy/test/null_model_full/{plat}-{targ}/exec/FMS").is_dir(),
-            Path(f"{OUT}/fremake_Canopy/test/null_model_full/{plat}-{targ}/exec/log.compile")]
+    assert [Path(f"{OUT}/fremake_canopy/test/null_model_full/{plat}-{targ}/exec/compile.sh").exists(),
+            Path(f"{OUT}/fremake_canopy/test/null_model_full/{plat}-{targ}/exec/FMS").is_dir(),
+            Path(f"{OUT}/fremake_canopy/test/null_model_full/{plat}-{targ}/exec/log.compile")]
+
+@pytest.mark.xfail()
+def test_bad_platform():
+    """
+    Check for the failure of compile script creation
+    due to a bad platform passed.
+    """
+    yamlfile_path = f"{TEST_DIR}/{NM_EXAMPLE}/{YAMLFILE}"
+
+    # Create the compile script
+    create_compile_script.compile_create(yamlfile_path, BAD_PLATFORM, TARGET, 4, 1, False, False)
+
+@pytest.mark.xfail()
+def test_bad_target():
+    """
+    Check for the failure of compile script creation
+    due to a bad target passed.
+    """
+    yamlfile_path = f"{TEST_DIR}/{NM_EXAMPLE}/{YAMLFILE}"
+
+    # Create the compile script
+    create_compile_script.compile_create(yamlfile_path, PLATFORM, BAD_TARGET, 4, 1, False, False)
+
+def test_multi_target():
+    """
+    Check for the creation of the compile script for each target passed
+    """
+    yamlfile_path = f"{TEST_DIR}/{NM_EXAMPLE}/{YAMLFILE}"
+
+    # Create the compile script
+    create_compile_script.compile_create(yamlfile_path, PLATFORM, MULTI_TARGET, 4, 1, False, False)
 
-#TO-DO: check for failures, ETC....
+    for t in MULTI_TARGET:
+        assert Path(f"{OUT}/fremake_canopy/test/null_model_full/{PLATFORM[0]}-{t}/exec/compile.sh").exists()