fixed newznab parsing limited Wombles rss to TV

pull/2/head
kay.one 11 years ago
parent 5dc7e03b8d
commit 074d7f2df5

@ -42,7 +42,7 @@ namespace NzbDrone.Api.Indexers
private List<ReleaseResource> GetRss() private List<ReleaseResource> GetRss()
{ {
if (results == null) //if (results == null)
{ {
var reports = _rssFetcherAndParser.Fetch(); var reports = _rssFetcherAndParser.Fetch();
var decisions = _downloadDecisionMaker.GetRssDecision(reports); var decisions = _downloadDecisionMaker.GetRssDecision(reports);

@ -4,6 +4,7 @@ using NzbDrone.Core.Indexers;
using NzbDrone.Core.Indexers.Newznab; using NzbDrone.Core.Indexers.Newznab;
using NzbDrone.Core.Indexers.NzbClub; using NzbDrone.Core.Indexers.NzbClub;
using NzbDrone.Core.Indexers.Nzbx; using NzbDrone.Core.Indexers.Nzbx;
using NzbDrone.Core.Indexers.Wombles;
using NzbDrone.Core.Parser.Model; using NzbDrone.Core.Parser.Model;
using NzbDrone.Core.Test.Framework; using NzbDrone.Core.Test.Framework;
using NUnit.Framework; using NUnit.Framework;
@ -18,7 +19,7 @@ namespace NzbDrone.Core.Test.IndexerTests.IntegrationTests
public void SetUp() public void SetUp()
{ {
UseRealHttp(); UseRealHttp();
} }
[Test] [Test]
@ -43,6 +44,17 @@ namespace NzbDrone.Core.Test.IndexerTests.IntegrationTests
[Test]
public void wombles_rss()
{
var indexer = new Wombles();
var result = Subject.FetchRss(indexer);
ValidateResult(result, skipSize: true, skipInfo: true);
}
[Test] [Test]
[Explicit("needs newznab api key")] [Explicit("needs newznab api key")]
public void nzbsorg_rss() public void nzbsorg_rss()
@ -58,25 +70,26 @@ namespace NzbDrone.Core.Test.IndexerTests.IntegrationTests
var result = Subject.FetchRss(indexer); var result = Subject.FetchRss(indexer);
result.Should().NotBeEmpty(); ValidateResult(result);
result.Should().OnlyContain(c => !string.IsNullOrWhiteSpace(c.Title));
result.Should().OnlyContain(c => !string.IsNullOrWhiteSpace(c.NzbUrl));
//TODO: uncomment these after moving to restsharp for rss
//result.Should().OnlyContain(c => !string.IsNullOrWhiteSpace(c.NzbInfoUrl));
//result.Should().OnlyContain(c => c.Size > 0);
} }
private void ValidateResult(IList<ReportInfo> reports) private void ValidateResult(IList<ReportInfo> reports, bool skipSize = false, bool skipInfo = false)
{ {
reports.Should().NotBeEmpty(); reports.Should().NotBeEmpty();
reports.Should().OnlyContain(c => !string.IsNullOrWhiteSpace(c.Title)); reports.Should().OnlyContain(c => !string.IsNullOrWhiteSpace(c.Title));
reports.Should().OnlyContain(c => !string.IsNullOrWhiteSpace(c.NzbInfoUrl));
reports.Should().OnlyContain(c => !string.IsNullOrWhiteSpace(c.NzbUrl)); reports.Should().OnlyContain(c => !string.IsNullOrWhiteSpace(c.NzbUrl));
reports.Should().OnlyContain(c => c.Size > 0);
if (!skipInfo)
{
reports.Should().OnlyContain(c => !string.IsNullOrWhiteSpace(c.NzbInfoUrl));
}
if (!skipSize)
{
reports.Should().OnlyContain(c => c.Size > 0);
}
} }
} }

@ -7,7 +7,13 @@ namespace NzbDrone.Core.Indexers.Newznab
{ {
public class Newznab : IndexerWithSetting<NewznabSettings> public class Newznab : IndexerWithSetting<NewznabSettings>
{ {
public override IParseFeed Parser
{
get
{
return new NewznabParser(this);
}
}
public override IEnumerable<IndexerDefinition> DefaultDefinitions public override IEnumerable<IndexerDefinition> DefaultDefinitions
{ {

@ -5,9 +5,17 @@ namespace NzbDrone.Core.Indexers.Wombles
{ {
public class Wombles : IndexerBase public class Wombles : IndexerBase
{ {
public override IParseFeed Parser
{
get
{
return new WomblesParser();
}
}
public override IEnumerable<string> RecentFeed public override IEnumerable<string> RecentFeed
{ {
get { yield return string.Format("http://nzb.isasecret.com/rss"); } get { yield return "http://nzb.isasecret.com/rss/?sec=TV&fr=false"; }
} }
public override string Name public override string Name

Loading…
Cancel
Save