Skip to content

Commit

Permalink
Merge branch 'trunk' into sm_mixed_output
Browse files Browse the repository at this point in the history
  • Loading branch information
diemol committed May 17, 2024
2 parents 5ab0c80 + 22386f6 commit f4d2030
Show file tree
Hide file tree
Showing 4 changed files with 18 additions and 17 deletions.
6 changes: 3 additions & 3 deletions common/mirror/selenium
Original file line number Diff line number Diff line change
Expand Up @@ -23,13 +23,13 @@
"tag_name": "nightly",
"assets": [
{
"browser_download_url": "https://github.com/SeleniumHQ/selenium/releases/download/nightly/selenium-java-4.21.0-SNAPSHOT.zip"
"browser_download_url": "https://github.com/SeleniumHQ/selenium/releases/download/nightly/selenium-java-4.22.0-SNAPSHOT.zip"
},
{
"browser_download_url": "https://github.com/SeleniumHQ/selenium/releases/download/nightly/selenium-server-4.21.0-SNAPSHOT.jar"
"browser_download_url": "https://github.com/SeleniumHQ/selenium/releases/download/nightly/selenium-server-4.22.0-SNAPSHOT.jar"
},
{
"browser_download_url": "https://github.com/SeleniumHQ/selenium/releases/download/nightly/selenium-server-4.21.0-SNAPSHOT.zip"
"browser_download_url": "https://github.com/SeleniumHQ/selenium/releases/download/nightly/selenium-server-4.22.0-SNAPSHOT.zip"
}
]
},
Expand Down
15 changes: 13 additions & 2 deletions dotnet/src/webdriver/Navigator.cs
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
// </copyright>

using System;
using System.Collections.Generic;

namespace OpenQA.Selenium
{
Expand Down Expand Up @@ -58,7 +59,17 @@ public void Forward()
/// <param name="url">String of where you want the browser to go to</param>
public void GoToUrl(string url)
{
this.driver.Url = url;
if (url == null)
{
throw new ArgumentNullException(nameof(url), "URL cannot be null.");
}

Dictionary<string, object> parameters = new Dictionary<string, object>
{
{ "url", url }
};
this.driver.InternalExecute(DriverCommand.Get, parameters);

}

/// <summary>
Expand All @@ -72,7 +83,7 @@ public void GoToUrl(Uri url)
throw new ArgumentNullException(nameof(url), "URL cannot be null.");
}

this.driver.Url = url.ToString();
this.GoToUrl(url.ToString());
}

/// <summary>
Expand Down
12 changes: 1 addition & 11 deletions dotnet/src/webdriver/WebDriver.cs
Original file line number Diff line number Diff line change
Expand Up @@ -109,17 +109,7 @@ public string Url
return commandResponse.Value.ToString();
}

set
{
if (value == null)
{
throw new ArgumentNullException(nameof(value), "Argument 'url' cannot be null.");
}

Dictionary<string, object> parameters = new Dictionary<string, object>();
parameters.Add("url", value);
this.Execute(DriverCommand.Get, parameters);
}
set => new Navigator(this).GoToUrl(value);
}

/// <summary>
Expand Down
2 changes: 1 addition & 1 deletion rb/lib/selenium/webdriver/common/logger.rb
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,7 @@ def create_logger(name, level:)
logger.progname = name
logger.level = level
logger.formatter = proc do |severity, time, progname, msg|
"#{time.strftime('%F %T')} #{severity} #{progname} #{msg}\n"
"#{time.strftime('%F %T')} #{severity} #{progname} #{msg}\n".force_encoding('UTF-8')
end

logger
Expand Down

0 comments on commit f4d2030

Please sign in to comment.