Fix parsing Shortcut/@Name and detect duplicate in GetMsiFilenameValue.

2 files changed, 2 insertions, 32 deletions

diff --git a/src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs b/src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs
index 83f53aca..81f780dc 100644
--- a/src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs
+++ b/src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs
@@ -384,7 +384,7 @@ namespace WixToolsetTest.CoreIntegration
             }
         }
 
-        [Fact(Skip = "Test demonstrates failure")]
+        [Fact]
         public void PopulatesDirectoryTableWithValidDefaultDir()
         {
             var folder = TestData.Get(@"TestData");
@@ -648,7 +648,7 @@ namespace WixToolsetTest.CoreIntegration
             }
         }
 
-        [Fact(Skip = "Test demonstrates failure")]
+        [Fact]
         public void PopulatesMsiShortcutPropertyTable()
         {
             var folder = TestData.Get(@"TestData");
diff --git a/src/test/WixToolsetTest.CoreIntegration/WixlibQueryFixture.cs b/src/test/WixToolsetTest.CoreIntegration/WixlibQueryFixture.cs
index 53bc5910..7f9b9686 100644
--- a/src/test/WixToolsetTest.CoreIntegration/WixlibQueryFixture.cs
+++ b/src/test/WixToolsetTest.CoreIntegration/WixlibQueryFixture.cs
@@ -12,36 +12,6 @@ namespace WixToolsetTest.CoreIntegration
 
     public class WixlibQueryFixture
     {
-        [Fact(Skip = "Test demonstrates failure")]
-        public void ShortcutNameWithPreprocessorVariableIsResolved()
-        {
-            var folder = TestData.Get(@"TestData\Shortcut");
-
-            using (var fs = new DisposableFileSystem())
-            {
-                var baseFolder = fs.GetFolder();
-                var intermediateFolder = Path.Combine(baseFolder, "obj");
-                var wixlibPath = Path.Combine(intermediateFolder, @"test.wixlib");
-
-                var result = WixRunner.Execute(new[]
-                {
-                    "build",
-                    Path.Combine(folder, "ShortcutProperty.wxs"),
-                    "-intermediateFolder", intermediateFolder,
-                    "-o", wixlibPath
-                });
-
-                result.AssertSuccess();
-
-                var intermediate = Intermediate.Load(wixlibPath);
-                var allTuples = intermediate.Sections.SelectMany(s => s.Tuples);
-                var shortcutTuple = allTuples.OfType<ShortcutTuple>()
-                                             .SingleOrDefault();
-                Assert.NotNull(shortcutTuple);
-                Assert.Equal("d", shortcutTuple.Name);
-            }
-        }
-
         [Fact]
         public void UpgradeProducesReferenceToRemoveExistingProducts()
         {