fix upgrade appsettings

Altinn · Sep 26, 2023 · e829a24 · e829a24
1 parent 806d405
commit e829a24
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 14 deletions.
diff --git a/cli-tools/altinn-app-cli/Program.cs b/cli-tools/altinn-app-cli/Program.cs
@@ -1,4 +1,5 @@
 using System.CommandLine;
+using System.CommandLine.Invocation;
 using System.Reflection;
 using altinn_app_cli.v7Tov8.AppSettingsRewriter;
 using altinn_app_cli.v7Tov8.CodeRewriters;
@@ -43,8 +44,18 @@ static async Task<int> Main(string[] args)
         rootCommand.AddCommand(versionCommand);
 
         upgradeCommand.SetHandler(
-            async (projectFolder, projectFile, processFile, appSettingsFolder, targetVersion, skipCodeUpgrade, skipProcessUpgrade, skipCsprojUpgrade, skipAppSettingsUpgrade) =>
+            async (InvocationContext context) =>
             {
+                var projectFolder = context.ParseResult.GetValueForOption(projectFolderOption)!;
+                var projectFile = context.ParseResult.GetValueForOption(projectFileOption)!;
+                var processFile = context.ParseResult.GetValueForOption(processFileOption)!;
+                var appSettingsFolder = context.ParseResult.GetValueForOption(appSettingsFolderOption)!;
+                var targetVersion = context.ParseResult.GetValueForOption(targetVersionOption)!;
+                var skipCodeUpgrade = context.ParseResult.GetValueForOption(skipCodeUpgradeOption)!;
+                var skipProcessUpgrade = context.ParseResult.GetValueForOption(skipProcessUpgradeOption)!;
+                var skipCsprojUpgrade = context.ParseResult.GetValueForOption(skipCsprojUpgradeOption)!;
+                var skipAppSettingsUpgrade = context.ParseResult.GetValueForOption(skipAppSettingsUpgradeOption)!;
+
                 if (projectFolder == "CurrentDirectory")
                 {
                     projectFolder = Directory.GetCurrentDirectory();
@@ -114,16 +125,7 @@ static async Task<int> Main(string[] args)
                 {
                     Console.WriteLine("Upgrade completed with errors. Please check for errors in the log above.");
                 }
-            },
-            projectFolderOption,
-            projectFileOption,
-            processFileOption,
-            appSettingsFolderOption,
-            targetVersionOption,
-            skipCodeUpgradeOption,
-            skipProcessUpgradeOption,
-            skipCsprojUpgradeOption,
-            skipAppSettingsUpgradeOption
+            }
         );
 
         versionCommand.SetHandler(() =>
@@ -221,13 +223,13 @@ static async Task<int> UpgradeAppSettings(string appSettingsFolder)
             return 1;
         }
 
-        if (Directory.GetFiles(appSettingsFolder, "appsettings.*.json").Count() == 0)
+        if (Directory.GetFiles(appSettingsFolder, AppSettingsRewriter.APP_SETTINGS_FILE_PATTERN).Count() == 0)
         {
             Console.WriteLine($"No appsettings.*.json files found in {appSettingsFolder}");
             return 1;
         }
 
-        Console.WriteLine("Trying to upgrade appsettings.*.json files");
+        Console.WriteLine("Trying to upgrade appsettings*.json files");
         AppSettingsRewriter rewriter = new(appSettingsFolder);
         rewriter.Upgrade();
         await rewriter.Write();

diff --git a/cli-tools/altinn-app-cli/v7Tov8/AppSettingsRewriter/AppSettingsRewriter.cs b/cli-tools/altinn-app-cli/v7Tov8/AppSettingsRewriter/AppSettingsRewriter.cs
@@ -4,12 +4,19 @@
 
 namespace altinn_app_cli.v7Tov8.AppSettingsRewriter;
 
+
 /// <summary>
 /// Rewrites the appsettings.*.json files
 /// </summary>
 public class AppSettingsRewriter
 {
+    /// <summary>
+    /// The pattern used to search for appsettings.*.json files
+    /// </summary>
+    public static readonly string APP_SETTINGS_FILE_PATTERN = "appsettings*.json";
+
     private Dictionary<string, JsonObject> appSettingsJsonCollection;
+
     private readonly IList<string> warnings = new List<string>();
 
     /// <summary>
@@ -18,7 +25,7 @@ public class AppSettingsRewriter
     public AppSettingsRewriter(string appSettingsFolder)
     {
         appSettingsJsonCollection = new Dictionary<string, JsonObject>();
-        foreach (var file in Directory.GetFiles(appSettingsFolder, "appsettings.*.json"))
+        foreach (var file in Directory.GetFiles(appSettingsFolder, APP_SETTINGS_FILE_PATTERN))
         {
             var json = File.ReadAllText(file);
             var appSettingsJson = JsonNode.Parse(json);