Skip to content

Commit

Permalink
Don't run stest in the python Makefile, add a note to run it by hand,…
Browse files Browse the repository at this point in the history
… fix the conflict that ended up being committed, and make sure abort is added
  • Loading branch information
ekluzek committed Mar 19, 2024
1 parent 410a9b3 commit 5150c50
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 372 deletions.
13 changes: 12 additions & 1 deletion python/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,10 @@ PYLINT_SRC = \
# ../cime_config/buildnml

all: test black lint
# ----------------------------------------------------------------
# See the stest target about this issue
@echo "Run './run_ctsm_py_tests --sys' by hand afterwards"
# ----------------------------------------------------------------
@echo
@echo
@echo "Successfully ran all standard tests"
Expand All @@ -40,7 +44,14 @@ utest: FORCE

.PHONY: stest
stest: FORCE
$(PYTHON) ./run_ctsm_py_tests $(TEST_ARGS) --sys
# ----------------------------------------------------------------
# EBK 2024-03-19: Comment out running here because of this issue:
# https://github.com/ESCOMP/CTSM/pull/2363#issuecomment-1967884908
#$(PYTHON) ./run_ctsm_py_tests $(TEST_ARGS) --sys
# Instead run by hand which seems to be working for now...
# ----------------------------------------------------------------
@echo "System tests currently don't run under Make so..."
@echo "Run './run_ctsm_py_tests --sys' by hand afterwards"

.PHONY: lint
lint: FORCE
Expand Down
Loading

0 comments on commit 5150c50

Please sign in to comment.