Some cleanup of things marked for removal in v3

Fixes #5102

Co-Authored-By: Mark McDowall <markus101@users.noreply.github.com>

src/NzbDrone.Api/Indexers/ReleaseResource.cs

@@ -56,9 +56,7 @@ namespace NzbDrone.Api.Indexers
         public int? Leechers { get; set; }
         public DownloadProtocol Protocol { get; set; }
 
-        // TODO: Remove in v3
-        // Used to support the original Release Push implementation
-        // JsonIgnore so we don't serialize it, but can still parse it
+        // JsonIgnore so we don't serialize it, but can still parse it, removed in v3
         [JsonIgnore]
         public DownloadProtocol DownloadProtocol
         {

src/NzbDrone.Core.Test/Download/DownloadClientTests/SabnzbdTests/SabnzbdFixture.cs

@@ -216,7 +216,6 @@ namespace NzbDrone.Core.Test.Download.DownloadClientTests.SabnzbdTests
         [TestCase(SabnzbdDownloadStatus.Checking)]
         [TestCase(SabnzbdDownloadStatus.Downloading)]
         [TestCase(SabnzbdDownloadStatus.QuickCheck)]
-        [TestCase(SabnzbdDownloadStatus.ToPP)]
         [TestCase(SabnzbdDownloadStatus.Verifying)]
         [TestCase(SabnzbdDownloadStatus.Repairing)]
         [TestCase(SabnzbdDownloadStatus.Fetching)]

src/NzbDrone.Core/Download/Clients/Sabnzbd/SabnzbdDownloadStatus.cs

@@ -1,4 +1,4 @@
-namespace NzbDrone.Core.Download.Clients.Sabnzbd
+namespace NzbDrone.Core.Download.Clients.Sabnzbd
 {
     public enum SabnzbdDownloadStatus
     {
@@ -7,7 +7,6 @@
         Paused,
         Checking,
         Downloading,
-        ToPP,           // TODO: Remove in v3
         QuickCheck,
         Verifying,
         Repairing,