Fix crossplat pathing and other small issues in extension caching

3 files changed, 9 insertions, 13 deletions

diff --git a/src/WixToolset.Core.ExtensionCache/CachedExtension.cs b/src/WixToolset.Core.ExtensionCache/CachedExtension.cs
index 9ed874d9..5567541c 100644
--- a/src/WixToolset.Core.ExtensionCache/CachedExtension.cs
+++ b/src/WixToolset.Core.ExtensionCache/CachedExtension.cs
@@ -2,9 +2,9 @@
 
 namespace WixToolset.Core.ExtensionCache
 {
-    public class CachedExtension
+    internal class CachedExtension
     {
-        internal CachedExtension(string id, string version, bool damaged)
+        public CachedExtension(string id, string version, bool damaged)
         {
             this.Id = id;
             this.Version = version;
diff --git a/src/WixToolset.Core.ExtensionCache/ExtensionCacheManager.cs b/src/WixToolset.Core.ExtensionCache/ExtensionCacheManager.cs
index 3ec6451e..256eeb0b 100644
--- a/src/WixToolset.Core.ExtensionCache/ExtensionCacheManager.cs
+++ b/src/WixToolset.Core.ExtensionCache/ExtensionCacheManager.cs
@@ -19,16 +19,16 @@ namespace WixToolset.Core.ExtensionCache
     /// <summary>
     /// Extension cache manager.
     /// </summary>
-    public class ExtensionCacheManager
+    internal class ExtensionCacheManager
     {
         public string CacheFolder(bool global) => global ? this.GlobalCacheFolder() : this.LocalCacheFolder();
 
-        public string LocalCacheFolder() => Path.Combine(Environment.CurrentDirectory, @".wix\extensions\");
+        public string LocalCacheFolder() => Path.Combine(Environment.CurrentDirectory, ".wix", "extensions");
 
         public string GlobalCacheFolder()
         {
             var baseFolder = Environment.GetEnvironmentVariable("WIX_EXTENSIONS") ?? Environment.GetFolderPath(Environment.SpecialFolder.UserProfile);
-            return Path.Combine(baseFolder, @".wix\extensions\");
+            return Path.Combine(baseFolder, ".wix", "extensions");
         }
 
         public async Task<bool> AddAsync(bool global, string extension, CancellationToken cancellationToken)
@@ -84,12 +84,8 @@ namespace WixToolset.Core.ExtensionCache
             }
             else if (!String.IsNullOrEmpty(extensionVersion)) // looking for an explicit version of an extension.
             {
-                var extensionFolder = Path.Combine(cacheFolder, extensionId, extensionVersion);
-                if (Directory.Exists(extensionFolder))
-                {
-                    var present = ExtensionFileExists(cacheFolder, extensionId, extensionVersion);
-                    found.Add(new CachedExtension(extensionId, extensionVersion, !present));
-                }
+                var present = ExtensionFileExists(cacheFolder, extensionId, extensionVersion);
+                found.Add(new CachedExtension(extensionId, extensionVersion, !present));
             }
             else // looking for all versions of an extension or all versions of all extensions.
             {
diff --git a/src/WixToolset.Core.ExtensionCache/ExtensionCacheManagerCommand.cs b/src/WixToolset.Core.ExtensionCache/ExtensionCacheManagerCommand.cs
index 5016f430..24f17d06 100644
--- a/src/WixToolset.Core.ExtensionCache/ExtensionCacheManagerCommand.cs
+++ b/src/WixToolset.Core.ExtensionCache/ExtensionCacheManagerCommand.cs
@@ -116,7 +116,7 @@ namespace WixToolset.Core.ExtensionCache
 
         private async Task<bool> AddExtensions(ExtensionCacheManager cacheManager, CancellationToken cancellationToken)
         {
-            var success = true;
+            var success = false;
 
             foreach (var extensionRef in this.ExtensionReferences)
             {
@@ -129,7 +129,7 @@ namespace WixToolset.Core.ExtensionCache
 
         private async Task<bool> RemoveExtensions(ExtensionCacheManager cacheManager, CancellationToken cancellationToken)
         {
-            var success = true;
+            var success = false;
 
             foreach (var extensionRef in this.ExtensionReferences)
             {