Skip to content

Commit

Permalink
create_test was not failing the create_newcase phase when project inf…
Browse files Browse the repository at this point in the history
…o was missing

The problem is that test_scheduler was passing --project None to create_newcase
when it did not have project info. create_newcase was interpreting this as a string
"None", which is not what we want. This commit changes test_scheduler to not pass
--project unless there's a project.
  • Loading branch information
jgfouca committed Aug 3, 2016
1 parent fef81df commit c3f8f84
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions utils/python/CIME/test_scheduler.py
Original file line number Diff line number Diff line change
Expand Up @@ -368,9 +368,11 @@ def _create_newcase_phase(self, test):
machine, compiler, test_mods = CIME.utils.parse_test_name(test)

create_newcase_cmd = "%s --case %s --res %s --mach %s --compiler %s --compset %s"\
" --project %s --test"%\
" --test" % \
(os.path.join(self._cime_root, "scripts", "create_newcase"),
test_dir, grid, machine, compiler, compset, self._project)
test_dir, grid, machine, compiler, compset)
if self._project is not None:
create_newcase_cmd += " --project %s " % self._project

if test_mods is not None:
files = Files()
Expand Down

0 comments on commit c3f8f84

Please sign in to comment.