Skip to content

Commit b858d1f

Browse files
Merge pull request #216 from AutomationSolutionz/fix_remote_host
fix remote host
2 parents ae6809e + 686ba57 commit b858d1f

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

Framework/Built_In_Automation/Web/Selenium/BuiltInFunctions.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -381,6 +381,8 @@ def Open_Browser(dependency, window_size_X=None, window_size_Y=None, capability=
381381
)
382382
return CommonUtil.Exception_Handler(sys.exc_info(), None, ErrorMessage)
383383

384+
remote_host = None
385+
remote_browser_version = None
384386
if Shared_Resources.Test_Shared_Variables('run_time_params'): # Look for remote config in runtime params
385387
run_time_params = Shared_Resources.Get_Shared_Variables('run_time_params')
386388
remote_config = run_time_params.get("remote_config")
@@ -397,9 +399,6 @@ def Open_Browser(dependency, window_size_X=None, window_size_Y=None, capability=
397399
CommonUtil.ExecLog(
398400
sModuleInfo, "Remote host: %s is not up. Running the browser locally " % remote_config.get('host'), 3
399401
)
400-
else:
401-
remote_host = None
402-
remote_browser_version = None
403402
# try:
404403
# selenium_driver.close()
405404
# except:

0 commit comments

Comments
 (0)