chore: add tests to validate

Author: MaggieKimani1

test/Microsoft.OpenApi.Tests/Extensions/DictionaryExtensionsTests.cs renamed to test/Microsoft.OpenApi.Tests/Extensions/CollectionExtensionsTests.cs

@@ -14,7 +14,7 @@
 
 namespace Microsoft.OpenApi.Tests.Extensions
 {
-    public class DictionaryExtensionsTests
+    public class CollectionExtensionsTests
     {
         public static readonly OpenApiDocument Document = new OpenApiDocument
         {
@@ -137,72 +137,73 @@ public class DictionaryExtensionsTests
             }
         };
 
-        [Fact]
-        public void SortEmptyDictionaryReturnsEmptyDictionary()
+        [Theory]
+        [MemberData(nameof(OpenApiSpecVersions))]
+        public async Task SortOpenApiDocumentUsingCustomComparerSucceeds(OpenApiSpecVersion version)
         {
             // Arrange
-            Document.Components.Headers = new Dictionary<string, IOpenApiHeader>();
+            var outputStringWriter = new StringWriter(CultureInfo.InvariantCulture);
+            var settings = new OpenApiWriterSettings
+            {
+                Comparer = StringComparer.OrdinalIgnoreCase
+            };
+            var writer = new OpenApiYamlWriter(outputStringWriter, settings);
 
             // Act
-            var sortedDictionary = Document.Components.Headers.Sort();
+            Document.SerializeAs(version, writer);
+            await writer.FlushAsync();
 
             // Assert
-            Assert.Empty(sortedDictionary);
+            await Verifier.Verify(outputStringWriter).UseParameters(version);
         }
 
         [Fact]
-        public async Task SortOpenApiDocumentLexicographicallySucceeds()
+        public async Task SortHashSetsWorks()
         {
             // Arrange
             var expected = @"required:
   - id
   - name
-properties:
-  id:
-    type: integer
-    format: int64
-  name:
-    type: string
-  tag:
-    type: string";
+  - tag";
+            var schema = new OpenApiSchema
+            {
+                Required = new HashSet<string> { "tag", "id", "name" },
+            };
 
             var outputStringWriter = new StringWriter(CultureInfo.InvariantCulture);
             var settings = new OpenApiWriterSettings
             {
-                EnableSorting = true
+                Comparer = StringComparer.OrdinalIgnoreCase
             };
             var writer = new OpenApiYamlWriter(outputStringWriter, settings);
 
             // Act
-            var schema = Document.Components.Schemas["pet"];
-
             schema.SerializeAsV3(writer);
             await writer.FlushAsync();
-            var actual = outputStringWriter.ToString();
+            var sortedString = outputStringWriter.ToString();
 
             // Assert
-            Assert.Equal(expected.MakeLineBreaksEnvironmentNeutral(), actual.MakeLineBreaksEnvironmentNeutral());
+            Assert.Equal(expected.MakeLineBreaksEnvironmentNeutral(), sortedString.MakeLineBreaksEnvironmentNeutral());
         }
 
-        [Theory]
-        [MemberData(nameof(OpenApiSpecVersions))]
-        public async Task SortOpenApiDocumentUsingCustomComparerSucceeds(OpenApiSpecVersion version)
+        [Fact]
+        public void SortTagsByNameUsingComparerWorks()
         {
             // Arrange
-            var outputStringWriter = new StringWriter(CultureInfo.InvariantCulture);
-            var settings = new OpenApiWriterSettings
+            var tags = new HashSet<OpenApiTag>
             {
-                KeyComparer = StringComparer.OrdinalIgnoreCase
+                new() { Name = "three" },
+                new() { Name = "two" },
+                new() { Name = "one" }
             };
-            var writer = new OpenApiYamlWriter(outputStringWriter, settings);
 
             // Act
-            Document.SerializeAs(version, writer);
-            await writer.FlushAsync();
+            var sortedTags = tags.Sort(new OpenApiTagNameComparer());
 
             // Assert
-            await Verifier.Verify(outputStringWriter).UseParameters(version);
-        }        
+            Assert.Equal(3, sortedTags.Count);
+            Assert.True(sortedTags[0].Name == "one");
+        }
 
         public static TheoryData<OpenApiSpecVersion> OpenApiSpecVersions()
         {
@@ -214,4 +215,12 @@ public static TheoryData<OpenApiSpecVersion> OpenApiSpecVersions()
             return values;
         }
     }
+
+    public class OpenApiTagNameComparer : IComparer<OpenApiTag>
+    {
+        public int Compare(OpenApiTag tag1, OpenApiTag tag2)
+        {
+            return string.Compare(tag1.Name, tag2.Name, StringComparison.OrdinalIgnoreCase);
+        }
+    }
 }