Pass/Fail status updated properly

Conflicts: common/djangoapps/terrain/browser.py
2013-08-14 16:50:28 -04:00
parent 28d9bbaff2
commit 010fd5771d
1 changed files with 5 additions and 2 deletions
--- 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()