Removed strict mocker from available update tests, for real this time....
This commit is contained in:
parent
1202c9e6d0
commit
54dc8ee35e
|
@ -12,10 +12,12 @@ namespace NzbDrone.Common
|
||||||
{
|
{
|
||||||
private readonly EnvironmentProvider _environmentProvider;
|
private readonly EnvironmentProvider _environmentProvider;
|
||||||
private static readonly Logger logger = LogManager.GetCurrentClassLogger();
|
private static readonly Logger logger = LogManager.GetCurrentClassLogger();
|
||||||
|
private readonly string _userAgent;
|
||||||
|
|
||||||
public HttpProvider(EnvironmentProvider environmentProvider)
|
public HttpProvider(EnvironmentProvider environmentProvider)
|
||||||
{
|
{
|
||||||
_environmentProvider = environmentProvider;
|
_environmentProvider = environmentProvider;
|
||||||
|
_userAgent = String.Format("NzbDrone {0}", _environmentProvider.Version);
|
||||||
}
|
}
|
||||||
|
|
||||||
public HttpProvider()
|
public HttpProvider()
|
||||||
|
@ -37,6 +39,7 @@ namespace NzbDrone.Common
|
||||||
try
|
try
|
||||||
{
|
{
|
||||||
var client = new WebClient { Credentials = identity };
|
var client = new WebClient { Credentials = identity };
|
||||||
|
client.Headers.Add(HttpRequestHeader.UserAgent, _userAgent);
|
||||||
return client.DownloadString(address);
|
return client.DownloadString(address);
|
||||||
}
|
}
|
||||||
catch (Exception ex)
|
catch (Exception ex)
|
||||||
|
@ -49,7 +52,7 @@ namespace NzbDrone.Common
|
||||||
public virtual Stream DownloadStream(string url, NetworkCredential credential)
|
public virtual Stream DownloadStream(string url, NetworkCredential credential)
|
||||||
{
|
{
|
||||||
var request = (HttpWebRequest)WebRequest.Create(url);
|
var request = (HttpWebRequest)WebRequest.Create(url);
|
||||||
request.UserAgent = String.Format("NzbDrone {0}", _environmentProvider.Version);
|
request.UserAgent = _userAgent;
|
||||||
|
|
||||||
request.Credentials = credential;
|
request.Credentials = credential;
|
||||||
var response = request.GetResponse();
|
var response = request.GetResponse();
|
||||||
|
@ -71,6 +74,7 @@ namespace NzbDrone.Common
|
||||||
|
|
||||||
var stopWatch = Stopwatch.StartNew();
|
var stopWatch = Stopwatch.StartNew();
|
||||||
var webClient = new WebClient();
|
var webClient = new WebClient();
|
||||||
|
webClient.Headers.Add(HttpRequestHeader.UserAgent, _userAgent);
|
||||||
webClient.DownloadFile(url, fileName);
|
webClient.DownloadFile(url, fileName);
|
||||||
stopWatch.Stop();
|
stopWatch.Stop();
|
||||||
logger.Trace("Downloading Completed. took {0:0}s", stopWatch.Elapsed.Seconds);
|
logger.Trace("Downloading Completed. took {0:0}s", stopWatch.Elapsed.Seconds);
|
||||||
|
|
|
@ -20,8 +20,6 @@ namespace NzbDrone.Core.Test.ProviderTests.UpdateProviderTests
|
||||||
[SetUp]
|
[SetUp]
|
||||||
public void Setup()
|
public void Setup()
|
||||||
{
|
{
|
||||||
WithStrictMocker();
|
|
||||||
|
|
||||||
Mocker.GetMock<ConfigProvider>().SetupGet(c => c.UpdateUrl).Returns("http://update.nzbdrone.com/_test/");
|
Mocker.GetMock<ConfigProvider>().SetupGet(c => c.UpdateUrl).Returns("http://update.nzbdrone.com/_test/");
|
||||||
Mocker.Resolve<HttpProvider>();
|
Mocker.Resolve<HttpProvider>();
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue