Merge pull request #154 from atc-net/feature/Fix-IsFileDataLengthEqual

fix: IsFileDataLengthEqual and rename it to AreFilesEqual

Author: davidkallesen

src/Atc.CodingRules.Updater.CLI/OptionsHelper.cs

@@ -25,7 +25,11 @@ public static async Task<OptionsFile> CreateDefault(
             return CreateDefaultOptions(projectPath);
         }
 
-        options.Mappings.ResolvePaths(new DirectoryInfo(optionsPath));
+        options.Mappings.ResolvePaths(
+            optionsPath.EndsWith(".json", StringComparison.CurrentCultureIgnoreCase)
+                ? new FileInfo(optionsPath).Directory!
+                : new DirectoryInfo(optionsPath));
+
         return options;
     }
 

src/Atc.CodingRules.Updater/DirectoryBuildPropsHelper.cs

@@ -44,7 +44,7 @@ public static void HandleFile(
 
         try
         {
-            if (!file.Directory!.Exists)
+            if (!Directory.Exists(file.Directory!.FullName))
             {
                 Directory.CreateDirectory(file.Directory.FullName);
             }
@@ -68,7 +68,7 @@ public static void HandleFile(
                 return;
             }
 
-            if (FileHelper.IsFileDataLengthEqual(contentGit, contentFile) &&
+            if (FileHelper.AreFilesEqual(contentGit, contentFile) &&
                 contentGit.Equals(contentFile, StringComparison.Ordinal))
             {
                 logger.LogInformation($"{EmojisConstants.FileNotUpdated}   {descriptionPart} nothing to update");
@@ -141,7 +141,7 @@ private static void UpdateFile(
         if (useLatestMinorNugetVersion)
         {
             var newFileContent = EnsureLatestPackageReferencesVersion(logger, fileContent, LogCategoryType.Debug);
-            if (!FileHelper.IsFileDataLengthEqual(fileContent, newFileContent) ||
+            if (!FileHelper.AreFilesEqual(fileContent, newFileContent) ||
                 !fileContent.Equals(newFileContent, StringComparison.Ordinal))
             {
                 fileContent = newFileContent;

src/Atc.CodingRules.Updater/EditorConfigHelper.cs

@@ -37,7 +37,7 @@ public static void HandleFile(
 
         try
         {
-            if (!file.Directory!.Exists)
+            if (!Directory.Exists(file.Directory!.FullName))
             {
                 Directory.CreateDirectory(file.Directory.FullName);
             }
@@ -67,7 +67,7 @@ public static void HandleFile(
 
         try
         {
-            if (FileHelper.IsFileDataLengthEqual(contentGit, contentFile))
+            if (FileHelper.AreFilesEqual(contentGit, contentFile))
             {
                 logger.LogInformation($"{EmojisConstants.FileNotUpdated}   {descriptionPart} nothing to update");
                 return;
@@ -82,7 +82,7 @@ public static void HandleFile(
             var contentGitBasePart = ExtractContentBasePart(contentGit);
             var contentFileBasePart = ExtractContentBasePart(contentFile);
 
-            if (FileHelper.IsFileDataLengthEqual(contentGitBasePart, contentFileBasePart))
+            if (FileHelper.AreFilesEqual(contentGitBasePart, contentFileBasePart))
             {
                 logger.LogInformation($"{EmojisConstants.FileNotUpdated}   {descriptionPart} nothing to update");
                 return;

src/Atc.CodingRules.Updater/FileHelper.cs

@@ -1,3 +1,4 @@
+// ReSharper disable ConvertIfStatementToReturnStatement
 namespace Atc.CodingRules.Updater;
 
 public static class FileHelper
@@ -51,7 +52,7 @@ public static void CreateFile(
         logger.LogInformation($"{EmojisConstants.FileCreated}   {descriptionPart} created");
     }
 
-    public static bool IsFileDataLengthEqual(
+    public static bool AreFilesEqual(
         string dataA,
         string dataB)
     {
@@ -61,7 +62,34 @@ public static bool IsFileDataLengthEqual(
         var l1 = dataA.EnsureEnvironmentNewLines().Length;
         var l2 = dataB.EnsureEnvironmentNewLines().Length;
 
-        return l1.Equals(l2);
+        var isSameFileLength = l1.Equals(l2);
+        if (!isSameFileLength)
+        {
+            return false;
+        }
+
+        var headerLinesA = dataA.ToLines().Take(10).ToList();
+        var headerLinesB = dataB.ToLines().Take(10).ToList();
+
+        if (headerLinesA.Find(x => x.StartsWith("# Version", StringComparison.CurrentCultureIgnoreCase)) !=
+            headerLinesB.Find(x => x.StartsWith("# Version", StringComparison.CurrentCultureIgnoreCase)))
+        {
+            return false;
+        }
+
+        if (headerLinesA.Find(x => x.StartsWith("# Updated", StringComparison.CurrentCultureIgnoreCase)) !=
+            headerLinesB.Find(x => x.StartsWith("# Updated", StringComparison.CurrentCultureIgnoreCase)))
+        {
+            return false;
+        }
+
+        if (headerLinesA.Find(x => x.StartsWith("# Distribution", StringComparison.CurrentCultureIgnoreCase)) !=
+            headerLinesB.Find(x => x.StartsWith("# Distribution", StringComparison.CurrentCultureIgnoreCase)))
+        {
+            return false;
+        }
+
+        return true;
     }
 
     public static bool ContainsEditorConfigFile(