Fixed: Skip torrents in Deluge api that don't have hashes.
closes #2566 closes #2567 closes #2664
This commit is contained in:
parent
d7a054f637
commit
2a1421f488
src
NzbDrone.Core.Test/Download/DownloadClientTests/DelugeTests
NzbDrone.Core/Download/Clients/Deluge
|
@ -290,6 +290,24 @@ namespace NzbDrone.Core.Test.Download.DownloadClientTests.DelugeTests
|
||||||
item.CanBeRemoved.Should().Be(canBeRemoved);
|
item.CanBeRemoved.Should().Be(canBeRemoved);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
[Test]
|
||||||
|
public void GetItems_should_ignore_items_without_hash()
|
||||||
|
{
|
||||||
|
_downloading.Hash = null;
|
||||||
|
|
||||||
|
GivenTorrents(new List<DelugeTorrent>
|
||||||
|
{
|
||||||
|
_downloading,
|
||||||
|
_queued
|
||||||
|
});
|
||||||
|
|
||||||
|
var items = Subject.GetItems();
|
||||||
|
|
||||||
|
items.Should().HaveCount(1);
|
||||||
|
|
||||||
|
items.First().Status.Should().Be(DownloadItemStatus.Queued);
|
||||||
|
}
|
||||||
|
|
||||||
[Test]
|
[Test]
|
||||||
public void should_return_status_with_outputdirs()
|
public void should_return_status_with_outputdirs()
|
||||||
{
|
{
|
||||||
|
|
|
@ -105,6 +105,7 @@ namespace NzbDrone.Core.Download.Clients.Deluge
|
||||||
foreach (var torrent in torrents)
|
foreach (var torrent in torrents)
|
||||||
{
|
{
|
||||||
if (torrent.Hash == null) continue;
|
if (torrent.Hash == null) continue;
|
||||||
|
|
||||||
var item = new DownloadClientItem();
|
var item = new DownloadClientItem();
|
||||||
item.DownloadId = torrent.Hash.ToUpper();
|
item.DownloadId = torrent.Hash.ToUpper();
|
||||||
item.Title = torrent.Name;
|
item.Title = torrent.Name;
|
||||||
|
|
Loading…
Reference in New Issue