mirror of https://github.com/Sonarr/Sonarr
New: Use better page size for Newznab/Torznab (up to 100) when supported by the indexer
Closes #5373
This commit is contained in:
parent
c1e43e9389
commit
ddb25b1095
|
@ -94,12 +94,21 @@ namespace NzbDrone.Core.Test.IndexerTests.NewznabTests
|
||||||
}
|
}
|
||||||
|
|
||||||
[Test]
|
[Test]
|
||||||
public void should_use_pagesize_reported_by_caps()
|
public void should_use_best_pagesize_reported_by_caps()
|
||||||
{
|
{
|
||||||
_caps.MaxPageSize = 30;
|
_caps.MaxPageSize = 30;
|
||||||
_caps.DefaultPageSize = 25;
|
_caps.DefaultPageSize = 25;
|
||||||
|
|
||||||
Subject.PageSize.Should().Be(25);
|
Subject.PageSize.Should().Be(30);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Test]
|
||||||
|
public void should_not_use_pagesize_over_100_even_if_reported_in_caps()
|
||||||
|
{
|
||||||
|
_caps.MaxPageSize = 250;
|
||||||
|
_caps.DefaultPageSize = 25;
|
||||||
|
|
||||||
|
Subject.PageSize.Should().Be(100);
|
||||||
}
|
}
|
||||||
|
|
||||||
[Test]
|
[Test]
|
||||||
|
|
|
@ -136,12 +136,21 @@ namespace NzbDrone.Core.Test.IndexerTests.TorznabTests
|
||||||
}
|
}
|
||||||
|
|
||||||
[Test]
|
[Test]
|
||||||
public void should_use_pagesize_reported_by_caps()
|
public void should_use_best_pagesize_reported_by_caps()
|
||||||
{
|
{
|
||||||
_caps.MaxPageSize = 30;
|
_caps.MaxPageSize = 30;
|
||||||
_caps.DefaultPageSize = 25;
|
_caps.DefaultPageSize = 25;
|
||||||
|
|
||||||
Subject.PageSize.Should().Be(25);
|
Subject.PageSize.Should().Be(30);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Test]
|
||||||
|
public void should_not_use_pagesize_over_100_even_if_reported_in_caps()
|
||||||
|
{
|
||||||
|
_caps.MaxPageSize = 250;
|
||||||
|
_caps.DefaultPageSize = 25;
|
||||||
|
|
||||||
|
Subject.PageSize.Should().Be(100);
|
||||||
}
|
}
|
||||||
|
|
||||||
[TestCase("http://localhost:9117/", "/api")]
|
[TestCase("http://localhost:9117/", "/api")]
|
||||||
|
|
|
@ -21,7 +21,7 @@ namespace NzbDrone.Core.Indexers.Newznab
|
||||||
|
|
||||||
public override DownloadProtocol Protocol => DownloadProtocol.Usenet;
|
public override DownloadProtocol Protocol => DownloadProtocol.Usenet;
|
||||||
|
|
||||||
public override int PageSize => _capabilitiesProvider.GetCapabilities(Settings).DefaultPageSize;
|
public override int PageSize => Math.Min(100, Math.Max(_capabilitiesProvider.GetCapabilities(Settings).DefaultPageSize, _capabilitiesProvider.GetCapabilities(Settings).MaxPageSize));
|
||||||
|
|
||||||
public override IIndexerRequestGenerator GetRequestGenerator()
|
public override IIndexerRequestGenerator GetRequestGenerator()
|
||||||
{
|
{
|
||||||
|
|
|
@ -20,7 +20,7 @@ namespace NzbDrone.Core.Indexers.Torznab
|
||||||
public override string Name => "Torznab";
|
public override string Name => "Torznab";
|
||||||
|
|
||||||
public override DownloadProtocol Protocol => DownloadProtocol.Torrent;
|
public override DownloadProtocol Protocol => DownloadProtocol.Torrent;
|
||||||
public override int PageSize => _capabilitiesProvider.GetCapabilities(Settings).DefaultPageSize;
|
public override int PageSize => Math.Min(100, Math.Max(_capabilitiesProvider.GetCapabilities(Settings).DefaultPageSize, _capabilitiesProvider.GetCapabilities(Settings).MaxPageSize));
|
||||||
|
|
||||||
public override IIndexerRequestGenerator GetRequestGenerator()
|
public override IIndexerRequestGenerator GetRequestGenerator()
|
||||||
{
|
{
|
||||||
|
|
Loading…
Reference in New Issue