Make settings work

shana · shana · commit b61570fdbe20 · 2017-05-08T17:29:20.000+02:00
Add paths properties to environment for common places where we store things.
Fix json serialization
Remove unneeded code
diff --git a/src/GitHub.Api/AppConfiguration.cs b/src/GitHub.Api/AppConfiguration.cs
@@ -8,27 +8,11 @@ class AppConfiguration : IAppConfiguration
     {
         public AppConfiguration()
         {
-            applicationName = ApplicationInfo.ApplicationName;
-            applicationDescription = ApplicationInfo.ApplicationDescription;
-
             var executingAssembly = typeof(AppConfiguration).Assembly;
             AssemblyName = executingAssembly.GetName();
             ProductHeader = new ProductHeaderValue(ApplicationInfo.ApplicationSafeName, AssemblyName.Version.ToString());
         }
 
-        readonly string applicationName;
-        readonly string applicationDescription;
-
-        /// <summary>
-        /// Name of this application
-        /// </summary>
-        public string ApplicationName { get { return applicationName; } }
-
-        /// <summary>
-        /// Name of this application
-        /// </summary>
-        public string ApplicationDescription { get { return applicationDescription; } }
-
         /// <summary>
         /// The currently executing assembly.
         /// </summary>
diff --git a/src/GitHub.Api/ApplicationManagerBase.cs b/src/GitHub.Api/ApplicationManagerBase.cs
@@ -26,16 +26,14 @@ protected void Initialize(IUIDispatcher uiDispatcher)
         {
             // accessing Environment triggers environment initialization if it hasn't happened yet
             LocalSettings = new LocalSettings(Environment);
-            UserSettings = new UserSettings(Environment, ApplicationInfo.ApplicationName);
-            SystemSettings = new SystemSettings(Environment, ApplicationInfo.ApplicationName);
+            UserSettings = new UserSettings(Environment);
+            SystemSettings = new SystemSettings(Environment);
 
             LocalSettings.Initialize();
 
             UserSettings.Initialize();
 
-#if !DEVELOPER_BUILD
             Logging.TracingEnabled = UserSettings.Get("EnableTraceLogging", false);
-#endif
 
             SystemSettings.Initialize();
 
diff --git a/src/GitHub.Api/Authentication/Keychain.cs b/src/GitHub.Api/Authentication/Keychain.cs
@@ -28,6 +28,8 @@ class ConnectionCacheItem
     /// </summary>
     class Keychain : IKeychain
     {
+        const string ConnectionFile = "connections.json";
+
         private readonly ILogging logger = Logging.GetLogger<Keychain>();
 
         private readonly ICredentialManager credentialManager;
@@ -42,7 +44,7 @@ class Keychain : IKeychain
 
         public Keychain(IEnvironment environment, ICredentialManager credentialManager)
         {
-            cachePath = environment.ConnectionCachePath;
+            cachePath = environment.UserCachePath.Combine(ConnectionFile);
             this.credentialManager = credentialManager;
         }
 
diff --git a/src/GitHub.Api/DefaultEnvironment.cs b/src/GitHub.Api/DefaultEnvironment.cs
@@ -26,7 +26,8 @@ public string GetEnvironmentVariable(string variable)
         public string UnityAssetsPath { get; set; }
         public string UnityProjectPath { get; set; }
         public string ExtensionInstallPath { get; set; }
-        public NPath ConnectionCachePath { get; set; }
+        public NPath UserCachePath { get; set; }
+        public NPath SystemCachePath { get; set; }
         public string UserProfilePath { get { return Environment.GetEnvironmentVariable("USERPROFILE"); } }
         public string Path { get { return Environment.GetEnvironmentVariable("PATH"); } }
         public string NewLine { get { return Environment.NewLine; } }
diff --git a/src/GitHub.Api/IAppConfiguration.cs b/src/GitHub.Api/IAppConfiguration.cs
@@ -6,8 +6,6 @@ namespace GitHub.Unity
     // Represents the currently executing program.
     interface IAppConfiguration
     {
-        string ApplicationName { get; }
-        string ApplicationDescription { get; }
         AssemblyName AssemblyName { get; }
         ProductHeaderValue ProductHeader { get; }
     }
diff --git a/src/GitHub.Api/IEnvironment.cs b/src/GitHub.Api/IEnvironment.cs
@@ -19,9 +19,10 @@ interface IEnvironment
         string UnityAssetsPath { get; set; }
         string UnityProjectPath { get; set; }
         string ExtensionInstallPath { get; set; }
-        NPath ConnectionCachePath { get; set; }
+        NPath UserCachePath { get; set; }
         string RepositoryPath { get; }
         string GitInstallPath { get; }
         IRepository Repository { get; set; }
+        NPath SystemCachePath { get; set; }
     }
 }
diff --git a/src/GitHub.Api/LocalSettings.cs b/src/GitHub.Api/LocalSettings.cs
@@ -16,7 +16,7 @@ abstract class BaseSettings : ISettings
         public abstract void Unset(string key);
 
         protected virtual string SettingsFileName { get; set; }
-        protected virtual string SettingsPath { get; set; }
+        protected virtual NPath SettingsPath { get; set; }
     }
 
     class JsonBackedSettings : BaseSettings
@@ -44,7 +44,7 @@ public JsonBackedSettings()
 
         public override void Initialize()
         {
-            cachePath = Path.Combine(SettingsPath, SettingsFileName);
+            cachePath = SettingsPath.Combine(SettingsFileName);
             LoadFromCache(cachePath);
         }
 
@@ -157,7 +157,7 @@ private void EnsureCachePath(string cachePath)
 
         private class CacheData
         {
-            public Dictionary<string, object> GitHubUnity = new Dictionary<string, object>();
+            public Dictionary<string, object> GitHubUnity { get; set; } = new Dictionary<string, object>();
         }
 
     }
@@ -179,9 +179,9 @@ class UserSettings : JsonBackedSettings
     {
         private const string settingsFileName = "settings.json";
 
-        public UserSettings(IEnvironment environment, string path)
+        public UserSettings(IEnvironment environment)
         {
-            SettingsPath = environment.GetSpecialFolder(Environment.SpecialFolder.LocalApplicationData).ToNPath().Combine(path);
+            SettingsPath = environment.UserCachePath;
         }
 
         protected override string SettingsFileName { get { return settingsFileName; } }
@@ -191,9 +191,9 @@ class SystemSettings : JsonBackedSettings
     {
         private const string settingsFileName = "settings.json";
 
-        public SystemSettings(IEnvironment environment, string path)
+        public SystemSettings(IEnvironment environment)
         {
-            SettingsPath = environment.GetSpecialFolder(Environment.SpecialFolder.ApplicationData).ToNPath().Combine(path);
+            SettingsPath = environment.SystemCachePath;
         }
 
         protected override string SettingsFileName { get { return settingsFileName; } }
diff --git a/src/GitHub.Logging/Logging.cs b/src/GitHub.Logging/Logging.cs
@@ -4,13 +4,6 @@ namespace GitHub.Unity
 {
     public static class Logging
     {
-        static Logging()
-        {
-#if DEVELOPER_BUILD
-            tracingEnabled = true;
-#endif
-        }
-
         private static bool tracingEnabled;
 
         public static bool TracingEnabled
diff --git a/src/IntegrationTests/Git/IntegrationTestEnvironment.cs b/src/IntegrationTests/Git/IntegrationTestEnvironment.cs
@@ -108,11 +108,20 @@ public string ExtensionInstallPath
             set { throw new NotImplementedException(); }
         }
 
-        public NPath ConnectionCachePath
+        public NPath UserCachePath
         {
             get
             {
-                return GetSpecialFolder(Environment.SpecialFolder.LocalApplicationData).ToNPath().Combine("connections.json");
+                return GetSpecialFolder(Environment.SpecialFolder.LocalApplicationData).ToNPath().Combine(ApplicationInfo.ApplicationName);
+            }
+            set { throw new NotImplementedException(); }
+        }
+
+        public NPath SystemCachePath
+        {
+            get
+            {
+                return GetSpecialFolder(Environment.SpecialFolder.CommonApplicationData).ToNPath().Combine(ApplicationInfo.ApplicationName);
             }
             set { throw new NotImplementedException(); }
         }
diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/ApplicationManager.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/ApplicationManager.cs
@@ -84,9 +84,12 @@ protected override void InitializeEnvironment()
             Environment.UnityAssetsPath = assetsPath.ToString(SlashMode.Forward);
             Environment.UnityProjectPath = projectPath.ToString(SlashMode.Forward);
 
-            Environment.ConnectionCachePath = Environment.GetSpecialFolder(System.Environment.SpecialFolder.LocalApplicationData)
+            Environment.UserCachePath = Environment.GetSpecialFolder(System.Environment.SpecialFolder.LocalApplicationData)
                                              .ToNPath()
-                                             .Combine(AppConfiguration.ApplicationName, "connections.json");
+                                             .Combine(ApplicationInfo.ApplicationName);
+            Environment.SystemCachePath = Environment.GetSpecialFolder(System.Environment.SpecialFolder.CommonApplicationData)
+                                             .ToNPath()
+                                             .Combine(ApplicationInfo.ApplicationName);
 
             base.InitializeEnvironment();
         }

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,8 @@ class ConnectionCacheItem`
`28`	`28`	`/// </summary>`
`29`	`29`	`class Keychain : IKeychain`
`30`	`30`	`{`
	`31`	`+ const string ConnectionFile = "connections.json";`
	`32`	`+`
`31`	`33`	`private readonly ILogging logger = Logging.GetLogger<Keychain>();`
`32`	`34`
`33`	`35`	`private readonly ICredentialManager credentialManager;`
`@@ -42,7 +44,7 @@ class Keychain : IKeychain`
`42`	`44`
`43`	`45`	`public Keychain(IEnvironment environment, ICredentialManager credentialManager)`
`44`	`46`	`{`
`45`		`- cachePath = environment.ConnectionCachePath;`
	`47`	`+ cachePath = environment.UserCachePath.Combine(ConnectionFile);`
`46`	`48`	`this.credentialManager = credentialManager;`
`47`	`49`	`}`
`48`	`50`
Original file line number	Diff line number	Diff line change
`@@ -6,8 +6,6 @@ namespace GitHub.Unity`
`6`	`6`	`// Represents the currently executing program.`
`7`	`7`	`interface IAppConfiguration`
`8`	`8`	`{`
`9`		`- string ApplicationName { get; }`
`10`		`- string ApplicationDescription { get; }`
`11`	`9`	`AssemblyName AssemblyName { get; }`
`12`	`10`	`ProductHeaderValue ProductHeader { get; }`
`13`	`11`	`}`
Original file line number	Diff line number	Diff line change
`@@ -19,9 +19,10 @@ interface IEnvironment`
`19`	`19`	`string UnityAssetsPath { get; set; }`
`20`	`20`	`string UnityProjectPath { get; set; }`
`21`	`21`	`string ExtensionInstallPath { get; set; }`
`22`		`- NPath ConnectionCachePath { get; set; }`
	`22`	`+ NPath UserCachePath { get; set; }`
`23`	`23`	`string RepositoryPath { get; }`
`24`	`24`	`string GitInstallPath { get; }`
`25`	`25`	`IRepository Repository { get; set; }`
	`26`	`+ NPath SystemCachePath { get; set; }`
`26`	`27`	`}`
`27`	`28`	`}`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ abstract class BaseSettings : ISettings`
`16`	`16`	`public abstract void Unset(string key);`
`17`	`17`
`18`	`18`	`protected virtual string SettingsFileName { get; set; }`
`19`		`- protected virtual string SettingsPath { get; set; }`
	`19`	`+ protected virtual NPath SettingsPath { get; set; }`
`20`	`20`	`}`
`21`	`21`
`22`	`22`	`class JsonBackedSettings : BaseSettings`
`@@ -44,7 +44,7 @@ public JsonBackedSettings()`
`44`	`44`
`45`	`45`	`public override void Initialize()`
`46`	`46`	`{`
`47`		`- cachePath = Path.Combine(SettingsPath, SettingsFileName);`
	`47`	`+ cachePath = SettingsPath.Combine(SettingsFileName);`
`48`	`48`	`LoadFromCache(cachePath);`
`49`	`49`	`}`
`50`	`50`
`@@ -157,7 +157,7 @@ private void EnsureCachePath(string cachePath)`
`157`	`157`
`158`	`158`	`private class CacheData`
`159`	`159`	`{`
`160`		`- public Dictionary<string, object> GitHubUnity = new Dictionary<string, object>();`
	`160`	`+ public Dictionary<string, object> GitHubUnity { get; set; } = new Dictionary<string, object>();`
`161`	`161`	`}`
`162`	`162`
`163`	`163`	`}`
`@@ -179,9 +179,9 @@ class UserSettings : JsonBackedSettings`
`179`	`179`	`{`
`180`	`180`	`private const string settingsFileName = "settings.json";`
`181`	`181`
`182`		`- public UserSettings(IEnvironment environment, string path)`
	`182`	`+ public UserSettings(IEnvironment environment)`
`183`	`183`	`{`
`184`		`- SettingsPath = environment.GetSpecialFolder(Environment.SpecialFolder.LocalApplicationData).ToNPath().Combine(path);`
	`184`	`+ SettingsPath = environment.UserCachePath;`
`185`	`185`	`}`
`186`	`186`
`187`	`187`	`protected override string SettingsFileName { get { return settingsFileName; } }`
`@@ -191,9 +191,9 @@ class SystemSettings : JsonBackedSettings`
`191`	`191`	`{`
`192`	`192`	`private const string settingsFileName = "settings.json";`
`193`	`193`
`194`		`- public SystemSettings(IEnvironment environment, string path)`
	`194`	`+ public SystemSettings(IEnvironment environment)`
`195`	`195`	`{`
`196`		`- SettingsPath = environment.GetSpecialFolder(Environment.SpecialFolder.ApplicationData).ToNPath().Combine(path);`
	`196`	`+ SettingsPath = environment.SystemCachePath;`
`197`	`197`	`}`
`198`	`198`
`199`	`199`	`protected override string SettingsFileName { get { return settingsFileName; } }`
Original file line number	Diff line number	Diff line change
`@@ -4,13 +4,6 @@ namespace GitHub.Unity`
`4`	`4`	`{`
`5`	`5`	`public static class Logging`
`6`	`6`	`{`
`7`		`- static Logging()`
`8`		`- {`
`9`		`-#if DEVELOPER_BUILD`
`10`		`- tracingEnabled = true;`
`11`		`-#endif`
`12`		`- }`
`13`		`-`
`14`	`7`	`private static bool tracingEnabled;`
`15`	`8`
`16`	`9`	`public static bool TracingEnabled`
Original file line number	Diff line number	Diff line change
`@@ -108,11 +108,20 @@ public string ExtensionInstallPath`
`108`	`108`	`set { throw new NotImplementedException(); }`
`109`	`109`	`}`
`110`	`110`
`111`		`- public NPath ConnectionCachePath`
	`111`	`+ public NPath UserCachePath`
`112`	`112`	`{`
`113`	`113`	`get`
`114`	`114`	`{`
`115`		`- return GetSpecialFolder(Environment.SpecialFolder.LocalApplicationData).ToNPath().Combine("connections.json");`
	`115`	`+ return GetSpecialFolder(Environment.SpecialFolder.LocalApplicationData).ToNPath().Combine(ApplicationInfo.ApplicationName);`
	`116`	`+ }`
	`117`	`+ set { throw new NotImplementedException(); }`
	`118`	`+ }`
	`119`	`+`
	`120`	`+ public NPath SystemCachePath`
	`121`	`+ {`
	`122`	`+ get`
	`123`	`+ {`
	`124`	`+ return GetSpecialFolder(Environment.SpecialFolder.CommonApplicationData).ToNPath().Combine(ApplicationInfo.ApplicationName);`
`116`	`125`	`}`
`117`	`126`	`set { throw new NotImplementedException(); }`
`118`	`127`	`}`