Close driver on failure too, misspelling fix.

semenko · semenko · commit a66b64c1d885 · 2015-09-05T19:02:55.000-05:00
Signed-off-by: Nick Semenkovich &lt;semenko@alum.mit.edu&gt;
diff --git a/test/chromium/script.py b/test/chromium/script.py
@@ -28,16 +28,16 @@ class bcolors:
 chromeOps.add_extension(sys.argv[1])
 chromeOps.add_argument("--disable-setuid-sandbox")
 
-chromdriver_path = "chromedriver"
+chromedriver_path = "chromedriver"
 
 if 'TRAVIS' in os.environ:
     chromeOps.add_argument('--no-sandbox')
-    chromdriver_path = os.path.abspath("test/chromium/chromedriver")
+    chromedriver_path = os.path.abspath("test/chromium/chromedriver")
 
 # First argument is optional, if not specified will search path.
 
 try:
-    driver = webdriver.Chrome(chromdriver_path, chrome_options=chromeOps)
+    driver = webdriver.Chrome(chromedriver_path, chrome_options=chromeOps)
 except WebDriverException as e:
     error = e.__str__()
 
@@ -55,12 +55,18 @@ class bcolors:
 
 #Page Loaded
 
+failed = False
 if driver.current_url.startswith('https'):
     print bcolors.OKGREEN + "HTTP to HTTPS redirection successful" + bcolors.ENDC
 elif driver.current_url.startswith('http'):
     print bcolors.FAIL + "HTTP to HTTPS redirection failed" + bcolors.ENDC
+    failed = True
     sys.exit(1)
 
 print '' #New line
 
 driver.quit()
+
+
+if failed:
+    sys.exit(1)
