Fix type/formats for primitive types (#56745)

Author: captainsafia

src/OpenApi/src/Extensions/JsonNodeSchemaExtensions.cs

@@ -27,7 +27,8 @@ internal static class JsonNodeSchemaExtensions
     private static readonly Dictionary<Type, OpenApiSchema> _simpleTypeToOpenApiSchema = new()
     {
         [typeof(bool)] = new() { Type = "boolean" },
-        [typeof(byte)] = new() { Type = "string", Format = "byte" },
+        [typeof(byte)] = new() { Type = "integer", Format = "uint8" },
+        [typeof(byte[])] = new() { Type = "string", Format = "byte" },
         [typeof(int)] = new() { Type = "integer", Format = "int32" },
         [typeof(uint)] = new() { Type = "integer", Format = "uint32" },
         [typeof(long)] = new() { Type = "integer", Format = "int64" },
@@ -40,7 +41,7 @@ internal static class JsonNodeSchemaExtensions
         [typeof(DateTime)] = new() { Type = "string", Format = "date-time" },
         [typeof(DateTimeOffset)] = new() { Type = "string", Format = "date-time" },
         [typeof(Guid)] = new() { Type = "string", Format = "uuid" },
-        [typeof(char)] = new() { Type = "string" },
+        [typeof(char)] = new() { Type = "string", Format = "char" },
         [typeof(Uri)] = new() { Type = "string", Format = "uri" },
         [typeof(string)] = new() { Type = "string" },
         [typeof(TimeOnly)] = new() { Type = "string", Format = "time" },

src/OpenApi/test/Services/OpenApiDocumentService/OpenApiDocumentServiceTests.RequestBody.cs

@@ -808,8 +808,8 @@ private void ActionWithDateTimeForm([FromForm] DateTime model) { }
         [([FromForm] decimal id) => {}, "number", "double"],
         [([FromForm] bool id) => {}, "boolean", null],
         [([FromForm] string id) => {}, "string", null],
-        [([FromForm] char id) => {}, "string", null],
-        [([FromForm] byte id) => {}, "string", "byte"],
+        [([FromForm] char id) => {}, "string", "char"],
+        [([FromForm] byte id) => {}, "integer", "uint8"],
         [([FromForm] short id) => {}, "integer", "int16"],
         [([FromForm] ushort id) => {}, "integer", "uint16"],
         [([FromForm] uint id) => {}, "integer", "uint32"],

src/OpenApi/test/Services/OpenApiSchemaService/OpenApiSchemaService.ParameterSchemas.cs

@@ -21,8 +21,9 @@ public partial class OpenApiSchemaServiceTests : OpenApiDocumentServiceTestBase
         [(decimal id) => {}, "number", "double", false],
         [(bool id) => {}, "boolean", null, false],
         [(string id) => {}, "string", null, false],
-        [(char id) => {}, "string", null, false],
-        [(byte id) => {}, "string", "byte", false],
+        [(char id) => {}, "string", "char", false],
+        [(byte id) => {}, "integer", "uint8", false],
+        [(byte[] id) => {}, "string", "byte", false],
         [(short id) => {}, "integer", "int16", false],
         [(ushort id) => {}, "integer", "uint16", false],
         [(uint id) => {}, "integer", "uint32", false],
@@ -37,8 +38,9 @@ public partial class OpenApiSchemaServiceTests : OpenApiDocumentServiceTestBase
         [(decimal? id) => {}, "number", "double", true],
         [(bool? id) => {}, "boolean", null, true],
         [(string? id) => {}, "string", null, true],
-        [(char? id) => {}, "string", null, true],
-        [(byte? id) => {}, "string", "byte", true],
+        [(char? id) => {}, "string", "char", true],
+        [(byte? id) => {}, "integer", "uint8", true],
+        [(byte[]? id) => {}, "string", "byte", true],
         [(short? id) => {}, "integer", "int16", true],
         [(ushort? id) => {}, "integer", "uint16", true],
         [(uint? id) => {}, "integer", "uint32", true],

src/OpenApi/test/Services/OpenApiSchemaService/OpenApiSchemaService.ResponseSchemas.cs

@@ -18,8 +18,9 @@ public partial class OpenApiSchemaServiceTests : OpenApiDocumentServiceTestBase
         [() => 12.0m, "application/json", "number", "double"],
         [() => false, "application/json", "boolean", null],
         [() => "test", "text/plain", "string", null],
-        [() => 't', "application/json", "string", null],
-        [() => byte.MaxValue, "application/json", "string", "byte"],
+        [() => 't', "application/json", "string", "char"],
+        [() => byte.MaxValue, "application/json", "integer", "uint8"],
+        [() => new byte[] { }, "application/json", "string", "byte"],
         [() => short.MaxValue, "application/json", "integer", "int16"],
         [() => ushort.MaxValue, "application/json", "integer", "uint16"],
         [() => uint.MaxValue, "application/json", "integer", "uint32"],