Add Name to Payloads created during directory harvesting.

7161

4 files changed, 43 insertions, 2 deletions

diff --git a/src/tools/heat/DirectoryHarvester.cs b/src/tools/heat/DirectoryHarvester.cs
index c153fef5..e69857ab 100644
--- a/src/tools/heat/DirectoryHarvester.cs
+++ b/src/tools/heat/DirectoryHarvester.cs
@@ -177,7 +177,7 @@ namespace WixToolset.Harvesters
             {
                 try
                 {
-                    int fileCount = this.HarvestDirectory(path, "SourceDir\\", harvestParent, generateType);
+                    int fileCount = this.HarvestDirectory(path, "", harvestParent, generateType);
 
                     if (generateType != GenerateType.PayloadGroup)
                     {
@@ -256,7 +256,7 @@ namespace WixToolset.Harvesters
                 foreach (string filePath in Directory.GetFiles(path))
                 {
                     string fileName = Path.GetFileName(filePath);
-                    string source = String.Concat(relativePath, fileName);
+                    string source = String.Concat("SourceDir\\", relativePath, fileName);
 
                     Wix.ISchemaElement newChild;
                     if (generateType == GenerateType.PayloadGroup)
@@ -265,6 +265,11 @@ namespace WixToolset.Harvesters
                         newChild = payload;
 
                         payload.SourceFile = source;
+
+                        if (!String.IsNullOrEmpty(relativePath))
+                        {
+                            payload.Name = String.Concat(relativePath, fileName);
+                        }
                     }
                     else
                     {
diff --git a/src/tools/test/WixToolsetTest.Heat/DirectoryToPayloadGroupFixture.cs b/src/tools/test/WixToolsetTest.Heat/DirectoryToPayloadGroupFixture.cs
index c9b8ee30..9ca84936 100644
--- a/src/tools/test/WixToolsetTest.Heat/DirectoryToPayloadGroupFixture.cs
+++ b/src/tools/test/WixToolsetTest.Heat/DirectoryToPayloadGroupFixture.cs
@@ -76,5 +76,39 @@ namespace WixToolsetTest.Heat
                 }, wxs);
             }
         }
+
+        [Fact]
+        public void CanHarvestNestedFiles()
+        {
+            var folder = TestData.Get("TestData", "NestedFiles");
+
+            using (var fs = new DisposableFileSystem())
+            {
+                var outputPath = Path.Combine(fs.GetFolder(), "out.wxs");
+
+                var args = new[]
+                {
+                    "dir", folder,
+                    "-generate", "payloadgroup",
+                    "-o", outputPath
+                };
+
+                var result = HeatRunner.Execute(args);
+                result.AssertSuccess();
+
+                var wxs = File.ReadAllLines(outputPath).Select(s => s.Replace("\"", "'")).ToArray();
+                WixAssert.CompareLineByLine(new[]
+                {
+                    "<Wix xmlns='http://wixtoolset.org/schemas/v4/wxs'>",
+                    "    <Fragment>",
+                    "        <PayloadGroup Id='TARGETDIR'>",
+                    "            <Payload SourceFile='SourceDir\\Nested\\c.txt' Name='Nested\\c.txt' />",
+                    "            <Payload SourceFile='SourceDir\\b.txt' />",
+                    "        </PayloadGroup>",
+                    "    </Fragment>",
+                    "</Wix>",
+                }, wxs);
+            }
+        }
     }
 }
diff --git a/src/tools/test/WixToolsetTest.Heat/TestData/NestedFiles/Nested/c.txt b/src/tools/test/WixToolsetTest.Heat/TestData/NestedFiles/Nested/c.txt
new file mode 100644
index 00000000..17c43215
--- /dev/null
+++ b/src/tools/test/WixToolsetTest.Heat/TestData/NestedFiles/Nested/c.txt
@@ -0,0 +1 @@
+This be c.txt.
\ No newline at end of file
diff --git a/src/tools/test/WixToolsetTest.Heat/TestData/NestedFiles/b.txt b/src/tools/test/WixToolsetTest.Heat/TestData/NestedFiles/b.txt
new file mode 100644
index 00000000..704b3d88
--- /dev/null
+++ b/src/tools/test/WixToolsetTest.Heat/TestData/NestedFiles/b.txt
@@ -0,0 +1 @@
+This is b.txt