From 8446a52b1e4f491f984ae6863181e9f6f1268ae7 Mon Sep 17 00:00:00 2001 From: James Foucar Date: Fri, 6 Nov 2015 15:16:02 -0700 Subject: [PATCH] create_test: Fix bug in TestStatus file mgmt Was causing wait_for_tests to report tests as PASS when they weren't. [BFB] --- cime/scripts-acme/create_test_impl.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/cime/scripts-acme/create_test_impl.py b/cime/scripts-acme/create_test_impl.py index 684d42b3d0b3..b528a1a7440a 100644 --- a/cime/scripts-acme/create_test_impl.py +++ b/cime/scripts-acme/create_test_impl.py @@ -345,8 +345,9 @@ def _update_test_status_file(self, test_name): # into a single place. str_to_write = "" - made_it_to_phase = self._phases.index(self._get_test_phase(test_name)) - for phase in self._phases[0:made_it_to_phase+1]: + made_it_to_phase = self._get_test_phase(test_name) + made_it_to_phase_idx = self._phases.index(made_it_to_phase) + for phase in self._phases[0:made_it_to_phase_idx+1]: str_to_write += "%s %s %s\n" % (self._get_test_status(test_name, phase), test_name, phase) if (not self._no_run and not self._is_broken(test_name) and made_it_to_phase == BUILD_PHASE):