Skip to content

Commit

Permalink
Merge branch 'jgfouca/scripts-acme/create_test_status_fix2' into next…
Browse files Browse the repository at this point in the history
… (PR #450)

wait_for_tests was reporting tests as PASS when they weren't.
This commit provides a fix for that problem.

* jgfouca/scripts-acme/create_test_status_fix2:
  create_test_impl.py: Fix bug in TestStatus file mgmt
  • Loading branch information
rljacob committed Nov 6, 2015
2 parents d045e1f + 8446a52 commit a86b028
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions cime/scripts-acme/create_test_impl.py
Original file line number Diff line number Diff line change
Expand Up @@ -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):
Expand Down

0 comments on commit a86b028

Please sign in to comment.