New: Remove completed downloads from disk when removing from SABnzbd

(cherry picked from commit ac8283d7339b3d9e8ccf70f3d1ff031d1b0a71d1)

Fixes #947

Co-authored-by: Mark McDowall <mark@mcdowall.ca>
This commit is contained in:
servarr[bot]
2021-05-16 23:05:18 +01:00
committed by GitHub
parent ba6b52c4c9
commit 8711c5d824
2 changed files with 162 additions and 1 deletions
@@ -579,5 +579,115 @@ namespace NzbDrone.Core.Test.Download.DownloadClientTests.SabnzbdTests
result.IsValid.Should().BeFalse();
}
[Test]
public void should_remove_output_path_folder_when_deleting_a_completed_item_and_delete_data_is_true()
{
var path = @"C:\Test\Series.Title.S01E01".AsOsAgnostic();
Mocker.GetMock<IDiskProvider>()
.Setup(s => s.FolderExists(path))
.Returns(true);
_completed.Items.First().Storage = path;
GivenQueue(null);
GivenHistory(_completed);
Subject.RemoveItem(_completed.Items.First().Id, true);
Mocker.GetMock<IDiskProvider>()
.Verify(v => v.DeleteFolder(path, true), Times.Once);
Mocker.GetMock<IDiskProvider>()
.Verify(v => v.DeleteFile(path), Times.Never);
}
[Test]
public void should_remove_output_path_file_when_deleting_a_completed_item_and_delete_data_is_true()
{
var path = @"C:\Test\Series.Title.S01E01.mkv".AsOsAgnostic();
Mocker.GetMock<IDiskProvider>()
.Setup(s => s.FolderExists(path))
.Returns(false);
Mocker.GetMock<IDiskProvider>()
.Setup(s => s.FileExists(path))
.Returns(true);
_completed.Items.First().Storage = path;
GivenQueue(null);
GivenHistory(_completed);
Subject.RemoveItem(_completed.Items.First().Id, true);
Mocker.GetMock<IDiskProvider>()
.Verify(v => v.DeleteFolder(path, true), Times.Never);
Mocker.GetMock<IDiskProvider>()
.Verify(v => v.DeleteFile(path), Times.Once);
}
[Test]
public void should_not_remove_output_path_file_when_deleting_a_completed_item_and_delete_data_is_true_if_it_does_not_exist()
{
var path = @"C:\Test\Series.Title.S01E01.mkv".AsOsAgnostic();
Mocker.GetMock<IDiskProvider>()
.Setup(s => s.FolderExists(path))
.Returns(false);
Mocker.GetMock<IDiskProvider>()
.Setup(s => s.FileExists(path))
.Returns(false);
_completed.Items.First().Storage = path;
GivenQueue(null);
GivenHistory(_completed);
Subject.RemoveItem(_completed.Items.First().Id, true);
Mocker.GetMock<IDiskProvider>()
.Verify(v => v.DeleteFolder(path, true), Times.Never);
Mocker.GetMock<IDiskProvider>()
.Verify(v => v.DeleteFile(path), Times.Never);
}
[Test]
public void should_not_remove_output_path_file_when_deleting_a_completed_item_and_delete_data_is_false()
{
var path = @"C:\Test\Series.Title.S01E01.mkv".AsOsAgnostic();
Mocker.GetMock<IDiskProvider>()
.Setup(s => s.FolderExists(path))
.Returns(false);
Mocker.GetMock<IDiskProvider>()
.Setup(s => s.FileExists(path))
.Returns(false);
_completed.Items.First().Storage = path;
GivenQueue(null);
GivenHistory(_completed);
Subject.RemoveItem(_completed.Items.First().Id, false);
Mocker.GetMock<IDiskProvider>()
.Verify(v => v.FolderExists(path), Times.Never);
Mocker.GetMock<IDiskProvider>()
.Verify(v => v.FileExists(path), Times.Never);
Mocker.GetMock<IDiskProvider>()
.Verify(v => v.DeleteFolder(path, true), Times.Never);
Mocker.GetMock<IDiskProvider>()
.Verify(v => v.DeleteFile(path), Times.Never);
}
}
}