From 010fd5771d7114bd032c514ac053540cffb35084 Mon Sep 17 00:00:00 2001 From: JonahStanley Date: Wed, 14 Aug 2013 16:50:28 -0400 Subject: [PATCH] Pass/Fail status updated properly Conflicts: common/djangoapps/terrain/browser.py --- common/djangoapps/terrain/browser.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/common/djangoapps/terrain/browser.py b/common/djangoapps/terrain/browser.py index 37fde93b55..52ed8d1510 100644 --- a/common/djangoapps/terrain/browser.py +++ b/common/djangoapps/terrain/browser.py @@ -174,6 +174,9 @@ def teardown_browser(total): """ Quit the browser after executing the tests. """ - if settings.MITX_FEATURES.get('USE_SAUCE') and total.scenarios_ran != total.scenarios_passed: - set_job_status(jobid, False) + if settings.MITX_FEATURES.get('USE_SAUCE'): + if total.scenarios_ran != total.scenarios_passed: + set_job_status(jobid, False) + else: + set_job_status(jobid, True) world.browser.quit()