Make label matching case-insensitive

Author: SteveDesmond-ca

src/App.cs

@@ -33,10 +33,12 @@ public async Task Run(Settings settings)
 				if (repo.Archived)
 				{
 					_log($"(skipping archived {repo.Name})");
-					continue;
+				}
+				else
+				{
+					await _sync.SyncRepo(repo, settings, labels);
 				}
 
-				await _sync.SyncRepo(repo, settings, labels);
 				_log(string.Empty);
 			}
 

src/Synchronizer.cs

@@ -124,7 +124,10 @@ private void ShowSynchronizedLabels(Repository repo, IEnumerable<Label> accountL
 		private IEnumerable<Label> GetLabelsToAdd(Repository repo, IEnumerable<Label> accountLabels, IEnumerable<Label> repoLabels)
 		{
 			_setStatus($"Finding labels to add to {repo.Name}...");
-			var newLabels = accountLabels.Where(al => repoLabels.All(rl => rl.Name != al.Name && rl.Description != al.Description)).ToArray();
+			var newLabels = accountLabels.Where(al => repoLabels
+					.All(rl => !string.Equals(rl.Name, al.Name, StringComparison.CurrentCultureIgnoreCase)
+					           && !string.Equals(rl.Description, al.Description, StringComparison.CurrentCultureIgnoreCase)))
+				.ToArray();
 			_log($"{newLabels.Length,3} {"to add",-9} : {LabelNames(newLabels)}");
 			return newLabels;
 		}
@@ -140,7 +143,10 @@ private IEnumerable<Label> GetLabelsToEdit(Repository repo, IEnumerable<Label> a
 		private IEnumerable<Label> GetLabelsToDelete(Repository repo, IEnumerable<Label> accountLabels, IEnumerable<Label> repoLabels)
 		{
 			_setStatus($"Finding labels to delete from {repo.Name}...");
-			var oldLabels = repoLabels.Where(rl => accountLabels.All(al => al.Name != rl.Name && al.Description != rl.Description)).ToArray();
+			var oldLabels = repoLabels.Where(rl => accountLabels
+					.All(al => !string.Equals(al.Name, rl.Name, StringComparison.InvariantCultureIgnoreCase)
+					           && !string.Equals(al.Description, rl.Description, StringComparison.InvariantCultureIgnoreCase)))
+				.ToArray();
 			_log($"{oldLabels.Length,3} {"to delete",-9} : {LabelNames(oldLabels)}");
 			return oldLabels;
 		}
@@ -170,12 +176,13 @@ private async Task DeleteLabels(Repository repo, IEnumerable<Label> oldLabels)
 		}
 
 		private static bool Matching(Label accountLabel, Label repoLabel)
-			=> repoLabel.Name == accountLabel.Name
-			   && repoLabel.Description == accountLabel.Description
-			   && repoLabel.Color == accountLabel.Color;
+			=> string.Equals(repoLabel.Name, accountLabel.Name, StringComparison.InvariantCultureIgnoreCase)
+			   && string.Equals(repoLabel.Description, accountLabel.Description, StringComparison.InvariantCultureIgnoreCase)
+			   && string.Equals(repoLabel.Color, accountLabel.Color, StringComparison.InvariantCultureIgnoreCase);
 
 		private static bool NeedsUpdating(Label accountLabel, Label repoLabel)
-			=> accountLabel.Name == repoLabel.Name &&
-			   (accountLabel.Description != repoLabel.Description || accountLabel.Color != repoLabel.Color);
+			=> string.Equals(accountLabel.Name, repoLabel.Name, StringComparison.InvariantCultureIgnoreCase) &&
+			   (!string.Equals(accountLabel.Description, repoLabel.Description, StringComparison.InvariantCultureIgnoreCase)
+			    || !string.Equals(accountLabel.Color, repoLabel.Color, StringComparison.InvariantCultureIgnoreCase));
 	}
 }

test/SynchronizerTests.cs

@@ -11,24 +11,24 @@ public class SynchronizerTests
 	{
 		private readonly Stubs.Label[] _accountLabels =
 		{
-			new("t1", "test1", "000000"),
-			new("t2", "test2", "000000"),
-			new("t3", "test3", "000000"),
-			new("t4", "test4", "000000"),
-			new("t5", "test5", "000000"),
-			new("t6", "test6", "000000")
+			new("t1", "test1", "aaaaaa"),
+			new("t2", "test2", "bbbbbb"),
+			new("t3", "test3", "cccccc"),
+			new("t4", "test4", "dddddd"),
+			new("t5", "test5", "eeeeee"),
+			new("t6", "test6", "ffffff")
 		};
 
 		private readonly Stubs.Label[] _repoLabels =
 		{
-			new("t3", "", "000000"),
-			new("t4", "test4", "FFFFFF"),
-			new("t5", "", "FFFFFF"),
-			new("t6", "test6", "000000"),
+			new("T3", "", "cccccc"),
+			new("t4", "Test4", "eeeeee"),
+			new("T5", "", "dddddd"),
+			new("t6", "Test6", "ffffff"),
 			new("t7", "test7", "000000"),
-			new("t8", "test8", "000000"),
-			new("t9", "test9", "000000"),
-			new("t0", "test0", "000000")
+			new("t8", "test8", "111111"),
+			new("t9", "test9", "222222"),
+			new("t0", "test0", "333333")
 		};
 
 		private readonly Action<string> _noOp = _ => { };