From 7d699fdbbe8a15c72b69f633ac9799fb757e2acc Mon Sep 17 00:00:00 2001
From: Rob Mensching <rob@firegiant.com>
Date: Wed, 16 Oct 2019 17:04:28 -0700
Subject: Several fixes

---
 .../WixToolsetTest.CoreIntegration/LinkerFixture.cs    |  2 +-
 src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs  | 18 ++++++++++++------
 .../WixToolsetTest.CoreIntegration/MsiQueryFixture.cs  |  8 ++++----
 .../WixlibQueryFixture.cs                              |  4 ++--
 4 files changed, 19 insertions(+), 13 deletions(-)

(limited to 'src/test')

diff --git a/src/test/WixToolsetTest.CoreIntegration/LinkerFixture.cs b/src/test/WixToolsetTest.CoreIntegration/LinkerFixture.cs
index dc34c2cc..80a6d1dd 100644
--- a/src/test/WixToolsetTest.CoreIntegration/LinkerFixture.cs
+++ b/src/test/WixToolsetTest.CoreIntegration/LinkerFixture.cs
@@ -13,7 +13,7 @@ namespace WixToolsetTest.CoreIntegration
 
     public class LinkerFixture
     {
-        [Fact(Skip = "Test demonstrates failure")]
+        [Fact]
         public void CanBuildWithOverridableActions()
         {
             var folder = TestData.Get(@"TestData\OverridableActions");
diff --git a/src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs b/src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs
index e30441bf..2d6feb4e 100644
--- a/src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs
+++ b/src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs
@@ -608,7 +608,7 @@ namespace WixToolsetTest.CoreIntegration
             }
         }
 
-        [Fact(Skip = "Test demonstrates failure")]
+        [Fact]
         public void CanBuildVersionIndependentProgId()
         {
             var folder = TestData.Get(@"TestData\ProgId");
@@ -639,11 +639,17 @@ namespace WixToolsetTest.CoreIntegration
                 var section = intermediate.Sections.Single();
 
                 var progids = section.Tuples.OfType<ProgIdTuple>().OrderBy(tuple => tuple.ProgId).ToList();
-                Assert.Equal(2, progids.Count);
-                Assert.Equal("Foo.File.hol", progids[0].ProgId);
-                Assert.Equal("Foo.File.hol.15", progids[0].ParentProgIdRef);
-                Assert.Equal("Foo.File.hol.15", progids[1].ProgId);
-                Assert.Null(progids[1].ParentProgIdRef);
+                Assert.Equal(new[]
+                {
+                    "Foo.File.hol",
+                    "Foo.File.hol.15"
+                }, progids.Select(p => p.ProgId).ToArray());
+
+                Assert.Equal(new[]
+                {
+                    "Foo.File.hol.15",
+                    null
+                }, progids.Select(p => p.ParentProgIdRef).ToArray());
             }
         }
 
diff --git a/src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs b/src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs
index e2a672b7..d31c9ee8 100644
--- a/src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs
+++ b/src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs
@@ -181,7 +181,7 @@ namespace WixToolsetTest.CoreIntegration
             }
         }
 
-        [Fact(Skip = "Test demonstrates failure")]
+        [Fact]
         public void PopulatesClassTablesWhenProgIdIsNestedUnderAdvertisedClass()
         {
             var folder = TestData.Get(@"TestData");
@@ -396,7 +396,7 @@ namespace WixToolsetTest.CoreIntegration
             }
         }
 
-        [Fact(Skip = "Test demonstrates failure")]
+        [Fact]
         public void PopulatesInstallExecuteSequenceTable()
         {
             var folder = TestData.Get(@"TestData");
@@ -516,7 +516,7 @@ namespace WixToolsetTest.CoreIntegration
             }
         }
 
-        [Fact(Skip = "Test demonstrates failure")]
+        [Fact]
         public void PopulatesMsiShortcutPropertyTable()
         {
             var folder = TestData.Get(@"TestData");
@@ -549,7 +549,7 @@ namespace WixToolsetTest.CoreIntegration
             }
         }
 
-        [Fact(Skip = "Test demonstrates failure")]
+        [Fact]
         public void PopulatesRegistryTableFromRegistryValue()
         {
             var folder = TestData.Get(@"TestData");
diff --git a/src/test/WixToolsetTest.CoreIntegration/WixlibQueryFixture.cs b/src/test/WixToolsetTest.CoreIntegration/WixlibQueryFixture.cs
index 061eae07..517bddbf 100644
--- a/src/test/WixToolsetTest.CoreIntegration/WixlibQueryFixture.cs
+++ b/src/test/WixToolsetTest.CoreIntegration/WixlibQueryFixture.cs
@@ -12,8 +12,8 @@ namespace WixToolsetTest.CoreIntegration
 
     public class WixlibQueryFixture
     {
-        [Fact(Skip = "Test demonstrates failure")]
-        public void DetectOnlyUpgradeProducesReferenceToRemoveExistingProducts()
+        [Fact]
+        public void UpgradeProducesReferenceToRemoveExistingProducts()
         {
             var folder = TestData.Get(@"TestData\Upgrade");
 
-- 
cgit v1.2.3-55-g6feb