Throw when unable to move file

NzbDrone.Core.Test/MediaFileTests/RenameEpisodeFileServiceFixture.cs

@@ -80,6 +80,10 @@ namespace NzbDrone.Core.Test.MediaFileTests
         {
             GivenEpisodeFiles();
 
+            Mocker.GetMock<IMoveEpisodeFiles>()
+                  .Setup(s => s.MoveEpisodeFile(It.IsAny<EpisodeFile>(), It.IsAny<Series>()))
+                  .Throws(new SameFilenameException("Same file name", "Filename"));
+
             Subject.Execute(new RenameSeriesCommand(_series.Id));
 
             Mocker.GetMock<IMessageAggregator>()

NzbDrone.Core/MediaFiles/EpisodeFileMovingService.cs

@@ -62,15 +62,12 @@ namespace NzbDrone.Core.MediaFiles
         {
             if (!_diskProvider.FileExists(episodeFile.Path))
             {
-                _logger.Error("Episode file path does not exist, {0}", episodeFile.Path);
-                return null;
+                throw new FileNotFoundException("Episode file path does not exist", episodeFile.Path);
             }
 
-            //Only rename if existing and new filenames don't match
             if (DiskProvider.PathEquals(episodeFile.Path, destinationFilename))
             {
-                _logger.Debug("Skipping file rename, source and destination are the same: {0}", episodeFile.Path);
-                return null;
+                throw new SameFilenameException("File not moved, source and destination are the same", episodeFile.Path);
             }
 
             _diskProvider.CreateFolder(new FileInfo(destinationFilename).DirectoryName);

NzbDrone.Core/MediaFiles/EpisodeImport/ImportApprovedEpisodes.cs

@@ -68,13 +68,6 @@ namespace NzbDrone.Core.MediaFiles.EpisodeImport
                     if (newDownload)
                     {
                         episodeFile = _episodeFileUpgrader.UpgradeEpisodeFile(episodeFile, localEpisode);
-
-                        if (episodeFile == null)
-                        {
-                            _logger.Error("Failed to move [{0}], aborting processing", localEpisode);
-                            continue;
-                        }
-
                         _messageAggregator.PublishEvent(new EpisodeImportedEvent(episodeFile));
                     }
                     

NzbDrone.Core/MediaFiles/RenameEpisodeFileService.cs

@@ -1,4 +1,5 @@
-using System.Collections.Generic;
+using System;
+using System.Collections.Generic;
 using System.Linq;
 using NLog;
 using NzbDrone.Common.Messaging;
@@ -35,18 +36,26 @@ namespace NzbDrone.Core.MediaFiles
 
             foreach (var file in episodeFiles)
             {
-                var episodeFile = file;
+                try
+                {
+                    var episodeFile = file;
 
-                _logger.Trace("Renaming episode file: {0}", episodeFile);
-                episodeFile = _episodeFileMover.MoveEpisodeFile(episodeFile, series);
+                    _logger.Trace("Renaming episode file: {0}", episodeFile);
+                    episodeFile = _episodeFileMover.MoveEpisodeFile(episodeFile, series);
 
-                if (episodeFile != null)
-                {
                     _mediaFileService.Update(episodeFile);
                     renamed.Add(episodeFile);
-                }
 
-                _logger.Trace("Renamed episode file: {0}", episodeFile);
+                    _logger.Trace("Renamed episode file: {0}", episodeFile);
+                }
+                catch (SameFilenameException ex)
+                {
+                    _logger.Trace("File not renamed, source and destination are the same: {0}", ex.Filename);
+                }
+                catch (Exception ex)
+                {
+                    _logger.ErrorException("Failed to rename file: " + file.Path, ex);
+                }
             }
 
             if (renamed.Any())

NzbDrone.Core/MediaFiles/SameFilenameException.cs

@@ -0,0 +1,14 @@
+using System;
+
+namespace NzbDrone.Core.MediaFiles
+{
+    public class SameFilenameException : Exception
+    {
+        public String Filename { get; set; }
+
+        public SameFilenameException(string message, string filename) : base(message)
+        {
+            Filename = filename;
+        }
+    }
+}

NzbDrone.Core/NzbDrone.Core.csproj

@@ -234,6 +234,7 @@
     <Compile Include="Download\EpisodeGrabbedEvent.cs" />
     <Compile Include="MediaFiles\Events\SeriesRenamedEvent.cs" />
     <Compile Include="MediaFiles\RenameEpisodeFileService.cs" />
+    <Compile Include="MediaFiles\SameFilenameException.cs" />
     <Compile Include="MediaFiles\UpgradeMediaFileService.cs" />
     <Compile Include="MetadataSource\Trakt\TraktCommunicationException.cs" />
     <Compile Include="Notifications\Email\TestEmailCommand.cs" />