Remove WixFileNotFoundException, report checked paths and improve bind path command-line parsing

3 files changed, 29 insertions, 14 deletions

diff --git a/src/WixToolset.Core/Bind/FileResolver.cs b/src/WixToolset.Core/Bind/FileResolver.cs
index 86075e46..01dfe36c 100644
--- a/src/WixToolset.Core/Bind/FileResolver.cs
+++ b/src/WixToolset.Core/Bind/FileResolver.cs
@@ -43,17 +43,24 @@ namespace WixToolset.Core.Bind
 
         public string Resolve(SourceLineNumber sourceLineNumbers, IntermediateTupleDefinition tupleDefinition, string source)
         {
+            var checkedPaths = new List<string>();
+
             foreach (var extension in this.LibrarianExtensions)
             {
-                var resolved = extension.Resolve(sourceLineNumbers, tupleDefinition, source);
+                var resolved = extension.ResolveFile(sourceLineNumbers, tupleDefinition, source);
+
+                if (resolved?.CheckedPaths != null)
+                {
+                    checkedPaths.AddRange(resolved.CheckedPaths);
+                }
 
-                if (null != resolved)
+                if (!String.IsNullOrEmpty(resolved?.Path))
                 {
-                    return resolved;
+                    return resolved?.Path;
                 }
             }
 
-            return this.ResolveUsingBindPaths(source, tupleDefinition, sourceLineNumbers, BindStage.Normal);
+            return this.MustResolveUsingBindPaths(source, tupleDefinition, sourceLineNumbers, BindStage.Normal, checkedPaths);
         }
 
         /// <summary>
@@ -66,24 +73,32 @@ namespace WixToolset.Core.Bind
         /// <returns>Should return a valid path for the stream to be imported.</returns>
         public string ResolveFile(string source, IntermediateTupleDefinition tupleDefinition, SourceLineNumber sourceLineNumbers, BindStage bindStage)
         {
+            var checkedPaths = new List<string>();
+
             foreach (var extension in this.ResolverExtensions)
             {
                 var resolved = extension.ResolveFile(source, tupleDefinition, sourceLineNumbers, bindStage);
 
-                if (null != resolved)
+                if (resolved?.CheckedPaths != null)
+                {
+                    checkedPaths.AddRange(resolved.CheckedPaths);
+                }
+
+                if (!String.IsNullOrEmpty(resolved?.Path))
                 {
-                    return resolved;
+                    return resolved?.Path;
                 }
             }
 
-            return this.ResolveUsingBindPaths(source, tupleDefinition, sourceLineNumbers, bindStage);
+            return this.MustResolveUsingBindPaths(source, tupleDefinition, sourceLineNumbers, bindStage, checkedPaths);
         }
 
-        private string ResolveUsingBindPaths(string source, IntermediateTupleDefinition tupleDefinition, SourceLineNumber sourceLineNumbers, BindStage bindStage)
+        private string MustResolveUsingBindPaths(string source, IntermediateTupleDefinition tupleDefinition, SourceLineNumber sourceLineNumbers, BindStage bindStage, List<string> checkedPaths)
         {
             string resolved = null;
 
             // If the file exists, we're good to go.
+            checkedPaths.Add(source);
             if (CheckFileExists(source))
             {
                 resolved = source;
@@ -121,6 +136,7 @@ namespace WixToolset.Core.Bind
                     {
                         var filePath = Path.Combine(bindPath.Path, pathWithoutSourceDir);
 
+                        checkedPaths.Add(filePath);
                         if (CheckFileExists(filePath))
                         {
                             resolved = filePath;
@@ -131,6 +147,7 @@ namespace WixToolset.Core.Bind
                     {
                         var filePath = Path.Combine(bindPath.Path, path);
 
+                        checkedPaths.Add(filePath);
                         if (CheckFileExists(filePath))
                         {
                             resolved = filePath;
@@ -141,10 +158,9 @@ namespace WixToolset.Core.Bind
 
             if (null == resolved)
             {
-                throw new WixFileNotFoundException(sourceLineNumbers, source, tupleDefinition.Name);
+                throw new WixException(ErrorMessages.FileNotFound(sourceLineNumbers, source, tupleDefinition.Name, checkedPaths));
             }
 
-            // Didn't find the file.
             return resolved;
         }
 
diff --git a/src/WixToolset.Core/Bind/ResolveFieldsCommand.cs b/src/WixToolset.Core/Bind/ResolveFieldsCommand.cs
index 0d5c3142..b7ed8a18 100644
--- a/src/WixToolset.Core/Bind/ResolveFieldsCommand.cs
+++ b/src/WixToolset.Core/Bind/ResolveFieldsCommand.cs
@@ -152,10 +152,9 @@ namespace WixToolset.Core.Bind
                                     }
 #endif
                                 }
-                                catch (WixFileNotFoundException)
+                                catch (WixException e)
                                 {
-                                    // display the error with source line information
-                                    this.Messaging.Write(ErrorMessages.FileNotFound(row.SourceLineNumbers, objectField.Path));
+                                    this.Messaging.Write(e.Error);
                                 }
                             }
 
diff --git a/src/WixToolset.Core/Bind/TransferFilesCommand.cs b/src/WixToolset.Core/Bind/TransferFilesCommand.cs
index b89d3d03..b9c54a14 100644
--- a/src/WixToolset.Core/Bind/TransferFilesCommand.cs
+++ b/src/WixToolset.Core/Bind/TransferFilesCommand.cs
@@ -63,7 +63,7 @@ namespace WixToolset.Core.Bind
                     }
                     catch (FileNotFoundException e)
                     {
-                        throw new WixFileNotFoundException(fileTransfer.SourceLineNumbers, e.FileName);
+                        throw new WixException(ErrorMessages.FileNotFound(fileTransfer.SourceLineNumbers, e.FileName));
                     }
                     catch (DirectoryNotFoundException)
                     {