From 43dfdc8bf5198ab9f7622674876471078c0b42ad Mon Sep 17 00:00:00 2001
From: Mark McDowall <mark@mcdowall.ca>
Date: Sat, 19 Aug 2023 10:52:39 -0700
Subject: [PATCH] Fix tests for ImportListSyncService

(cherry picked from commit b271b3b694081a2889d75d39dc0296d53c734aaf)

Closes #2843
---
 .../ImportListSyncServiceFixture.cs           | 32 +++++++++++++++++++
 1 file changed, 32 insertions(+)

diff --git a/src/NzbDrone.Core.Test/ImportListTests/ImportListSyncServiceFixture.cs b/src/NzbDrone.Core.Test/ImportListTests/ImportListSyncServiceFixture.cs
index 968c14636..8cf73b463 100644
--- a/src/NzbDrone.Core.Test/ImportListTests/ImportListSyncServiceFixture.cs
+++ b/src/NzbDrone.Core.Test/ImportListTests/ImportListSyncServiceFixture.cs
@@ -27,6 +27,8 @@ namespace NzbDrone.Core.Test.ImportListTests
 
             _importListReports = new List<ImportListItemInfo> { importListItem1 };
 
+            var mockImportList = new Mock<IImportList>();
+
             Mocker.GetMock<IFetchAndParseImportList>()
                 .Setup(v => v.Fetch())
                 .Returns(_importListReports);
@@ -53,6 +55,10 @@ namespace NzbDrone.Core.Test.ImportListTests
                 .Setup(v => v.Get(It.IsAny<int>()))
                 .Returns(new ImportListDefinition { ShouldMonitor = ImportListMonitorType.SpecificBook });
 
+            Mocker.GetMock<IImportListFactory>()
+                .Setup(v => v.AutomaticAddEnabled(It.IsAny<bool>()))
+                .Returns(new List<IImportList> { mockImportList.Object });
+
             Mocker.GetMock<IFetchAndParseImportList>()
                 .Setup(v => v.Fetch())
                 .Returns(_importListReports);
@@ -322,5 +328,31 @@ namespace NzbDrone.Core.Test.ImportListTests
                                                                    t.First().AddOptions.BooksToMonitor.Count == expectedBooksMonitored &&
                                                                    t.First().Monitored == expectedAuthorMonitored), false));
         }
+
+        [Test]
+        public void should_not_fetch_if_no_lists_are_enabled()
+        {
+            Mocker.GetMock<IImportListFactory>()
+                .Setup(v => v.AutomaticAddEnabled(It.IsAny<bool>()))
+                .Returns(new List<IImportList>());
+
+            Subject.Execute(new ImportListSyncCommand());
+
+            Mocker.GetMock<IFetchAndParseImportList>()
+                .Verify(v => v.Fetch(), Times.Never);
+        }
+
+        [Test]
+        public void should_not_process_if_no_items_are_returned()
+        {
+            Mocker.GetMock<IFetchAndParseImportList>()
+                .Setup(v => v.Fetch())
+                .Returns(new List<ImportListItemInfo>());
+
+            Subject.Execute(new ImportListSyncCommand());
+
+            Mocker.GetMock<IImportListExclusionService>()
+                .Verify(v => v.All(), Times.Never);
+        }
     }
 }