From 92bd26dab11023a0c4c787945fd87b23aa0a78c8 Mon Sep 17 00:00:00 2001 From: Sean Hall Date: Wed, 13 Nov 2019 11:04:59 +1000 Subject: Fix parsing Shortcut/@Name and detect duplicate in GetMsiFilenameValue. --- .../MsiQueryFixture.cs | 4 +-- .../WixlibQueryFixture.cs | 30 ---------------------- 2 files changed, 2 insertions(+), 32 deletions(-) (limited to 'src/test') 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() - .SingleOrDefault(); - Assert.NotNull(shortcutTuple); - Assert.Equal("d", shortcutTuple.Name); - } - } - [Fact] public void UpgradeProducesReferenceToRemoveExistingProducts() { -- cgit v1.2.3-55-g6feb