Log errors that happen during file copy

Author: leotsarev

src/LibraryManager.Build/Contracts/HostInteraction.cs

@@ -83,7 +83,7 @@ public async Task<bool> CopyFileAsync(string sourcePath, string destinationPath,
                 throw new UnauthorizedAccessException();
             }
 
-            bool result = await FileHelpers.CopyFileAsync(sourcePath, absoluteDestinationPath, cancellationToken);
+            bool result = await FileHelpers.CopyFileAsync(sourcePath, absoluteDestinationPath, Logger, cancellationToken);
             if (result)
             {
                 Logger.Log(string.Format(Resources.Text.FileWrittenToDisk, destinationPath.Replace('\\', '/')), LogLevel.Operation);

src/LibraryManager.Contracts/FileHelpers.cs

@@ -136,21 +136,22 @@ public static Task<Stream> ReadFileAsStreamAsync(string fileName, CancellationTo
         /// </summary>
         /// <param name="sourceFile">Full path to the source file</param>
         /// <param name="destinationFile">Full path to the destination file</param>
+        /// <param name="logger">Pass logger to log errors.</param>
         /// <param name="cancellationToken">Cancellation token</param>
         /// <returns>A boolean indicating whether the file was copied successfully</returns>
-        public static async Task<bool> CopyFileAsync(string sourceFile, string destinationFile, CancellationToken cancellationToken)
+        public static async Task<bool> CopyFileAsync(string sourceFile, string destinationFile, ILogger logger, CancellationToken cancellationToken)
         {
             try
             {
-                using (FileStream sourceStream = File.Open(sourceFile, FileMode.Open, FileAccess.Read))
-                {
-                    await WriteToFileAsync(destinationFile, sourceStream, cancellationToken);
-                }
+                using FileStream sourceStream = File.Open(sourceFile, FileMode.Open, FileAccess.Read);
+
+                await WriteToFileAsync(destinationFile, sourceStream, cancellationToken);
 
                 return true;
             }
-            catch (Exception)
+            catch (Exception exception)
             {
+                logger.Log($"Error during copying file {exception}", LogLevel.Error);
                 return false;
             }
         }

src/LibraryManager.Vsix/Contracts/HostInteraction.cs

@@ -121,7 +121,7 @@ public async Task<bool> CopyFileAsync(string sourcePath, string destinationPath,
             }
 
             await VsHelpers.CheckFileOutOfSourceControlAsync(absoluteDestinationPath);
-            bool result = await FileHelpers.CopyFileAsync(sourcePath, absoluteDestinationPath, cancellationToken);
+            bool result = await FileHelpers.CopyFileAsync(sourcePath, absoluteDestinationPath, Logger, cancellationToken);
 
             if (result)
             {

src/libman/Contracts/HostInteraction.cs

@@ -108,7 +108,7 @@ public async Task<bool> CopyFileAsync(string sourcePath, string destinationPath,
                 throw new UnauthorizedAccessException();
             }
 
-            bool result = await FileHelpers.CopyFileAsync(sourcePath, absoluteDestinationPath, cancellationToken);
+            bool result = await FileHelpers.CopyFileAsync(sourcePath, absoluteDestinationPath, Logger, cancellationToken);
             if(result)
             {
                 Logger.Log(string.Format(Resources.Text.FileWrittenToDisk, destinationPath.Replace('\\', '/')), LogLevel.Operation);

test/LibraryManager.Mocks/HostInteraction.cs

@@ -152,7 +152,7 @@ public async Task<bool> CopyFileAsync(string sourcePath, string destinationPath,
                 throw new UnauthorizedAccessException();
             }
 
-            bool result = await FileHelpers.CopyFileAsync(sourcePath, absoluteDestinationPath, cancellationToken);
+            bool result = await FileHelpers.CopyFileAsync(sourcePath, absoluteDestinationPath, Logger, cancellationToken);
 
             return result;
         }