Apply code review suggestions

- Update comments.
- Update Copilot style suggestion.

Author: martincostello

src/OpenApi/src/Extensions/ApiDescriptionExtensions.cs

@@ -20,6 +20,7 @@ internal static class ApiDescriptionExtensions
     public static HttpMethod? GetHttpMethod(this ApiDescription apiDescription) =>
         apiDescription.HttpMethod?.ToUpperInvariant() switch
         {
+            // Only add methods documented in the OpenAPI spec: https://spec.openapis.org/oas/v3.1.1.html#path-item-object
             "GET" => HttpMethod.Get,
             "POST" => HttpMethod.Post,
             "PUT" => HttpMethod.Put,
@@ -28,7 +29,6 @@ internal static class ApiDescriptionExtensions
             "HEAD" => HttpMethod.Head,
             "OPTIONS" => HttpMethod.Options,
             "TRACE" => HttpMethod.Trace,
-            "QUERY" => null, // OpenAPI as of 3.1 does not yet support HTTP QUERY
             _ => null,
         };
 

src/OpenApi/test/Microsoft.AspNetCore.OpenApi.Tests/Integration/OpenApiDocumentIntegrationTests.cs

@@ -68,8 +68,6 @@ public async Task OpenApiDocumentIsValid(string documentName, OpenApiSpecVersion
         Assert.Empty(errors);
     }
 
-    // The test below can be removed when https://github.com/microsoft/OpenAPI.NET/issues/2453 is implemented
-
     [Theory] // See https://github.com/dotnet/aspnetcore/issues/63090
     [MemberData(nameof(OpenApiDocuments))]
     public async Task OpenApiDocumentReferencesAreValid(string documentName, OpenApiSpecVersion version)
@@ -103,6 +101,7 @@ private async Task<string> GetOpenApiDocument(string documentName, OpenApiSpecVe
         var documentService = fixture.Services.GetRequiredKeyedService<OpenApiDocumentService>(documentName);
         var scopedServiceProvider = fixture.Services.CreateScope();
         var document = await documentService.GetOpenApiDocumentAsync(scopedServiceProvider.ServiceProvider);
+
         return await document.SerializeAsJsonAsync(version);
     }