Newznab won't blow up if more than one indexer with the same URL is found during initialization.
This commit is contained in:
parent
b930eb0993
commit
a16defc5f0
|
@ -272,5 +272,28 @@ namespace NzbDrone.Core.Test.ProviderTests
|
||||||
result.Should().HaveCount(1);
|
result.Should().HaveCount(1);
|
||||||
result.First().BuiltIn.Should().BeTrue();
|
result.First().BuiltIn.Should().BeTrue();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
[Test]
|
||||||
|
public void InitializeNewznabIndexers_should_not_blow_up_if_more_than_one_indexer_with_the_same_url_is_found()
|
||||||
|
{
|
||||||
|
//Setup
|
||||||
|
var definition = Builder<NewznabDefinition>.CreateNew()
|
||||||
|
.With(d => d.Url = "http://www.nzbdrone2.com")
|
||||||
|
.With(d => d.BuiltIn = false)
|
||||||
|
.Build();
|
||||||
|
|
||||||
|
var db = TestDbHelper.GetEmptyDatabase();
|
||||||
|
Mocker.SetConstant(db);
|
||||||
|
|
||||||
|
db.Insert(definition);
|
||||||
|
db.Insert(definition);
|
||||||
|
|
||||||
|
//Act
|
||||||
|
Mocker.Resolve<NewznabProvider>().InitializeNewznabIndexers(new List<NewznabDefinition> { definition });
|
||||||
|
|
||||||
|
//Assert
|
||||||
|
var result = db.Fetch<NewznabDefinition>();
|
||||||
|
result.Should().HaveCount(2);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
|
@ -71,7 +71,7 @@ namespace NzbDrone.Core.Providers
|
||||||
{
|
{
|
||||||
NewznabDefinition indexerLocal = feedProvider;
|
NewznabDefinition indexerLocal = feedProvider;
|
||||||
var currentIndexer = currentIndexers
|
var currentIndexer = currentIndexers
|
||||||
.SingleOrDefault(c => new Uri(c.Url.ToLower()).Host == new Uri(indexerLocal.Url.ToLower()).Host);
|
.FirstOrDefault(c => new Uri(c.Url.ToLower()).Host == new Uri(indexerLocal.Url.ToLower()).Host);
|
||||||
|
|
||||||
if (currentIndexer == null)
|
if (currentIndexer == null)
|
||||||
{
|
{
|
||||||
|
|
Loading…
Reference in New Issue