fix: refactor ToIdentifier() to normalize flaggable enums

src/Microsoft.OpenApi/Extensions/OpenApiTypeMapper.cs

@@ -3,6 +3,7 @@
 
 using System;
 using System.Collections.Generic;
+using System.Linq;
 using Microsoft.OpenApi.Exceptions;
 using Microsoft.OpenApi.Models;
 
@@ -19,34 +20,60 @@
         /// </summary>
         /// <param name="schemaType"></param>
         /// <returns></returns>
-        public static string? ToIdentifier(this JsonSchemaType? schemaType)
+        public static string[]? ToIdentifiers(this JsonSchemaType? schemaType)
         {
             if (schemaType is null)
             {
                 return null;
             }
-            return schemaType.Value.ToIdentifier();
+            return schemaType.Value.ToIdentifiers();
         }
 
         /// <summary>
         /// Maps a JsonSchema data type to an identifier.
         /// </summary>
         /// <param name="schemaType"></param>
         /// <returns></returns>
-        public static string? ToIdentifier(this JsonSchemaType schemaType)
+        public static string[] ToIdentifiers(this JsonSchemaType schemaType)
         {
-            return schemaType switch
+            var types = new List<string>();
+
+            if (schemaType.HasFlag(JsonSchemaType.Boolean)) types.Add("boolean");
+            if (schemaType.HasFlag(JsonSchemaType.Integer)) types.Add("integer");
+            if (schemaType.HasFlag(JsonSchemaType.Number)) types.Add("number");
+            if (schemaType.HasFlag(JsonSchemaType.String)) types.Add("string");
+            if (schemaType.HasFlag(JsonSchemaType.Object)) types.Add("object");
+            if (schemaType.HasFlag(JsonSchemaType.Array)) types.Add("array");
+            if (schemaType.HasFlag(JsonSchemaType.Null)) types.Add("null");
+
+            return types.ToArray();
+        }
+
+        /// <summary>
+        /// Returns the first identifier from a string array.
+        /// </summary>
+        /// <param name="schemaType"></param>
+        /// <returns></returns>
+        internal static string ToFirstIdentifier(this JsonSchemaType schemaType)
+        {
+            var identifier = schemaType.ToIdentifiers();
+            return identifier[0];
+        }
+
+        /// <summary>
+        /// Returns a single identifier from an array with only one item.
+        /// </summary>
+        /// <param name="schemaType"></param>
+        /// <returns></returns>
+        public static string ToSingleIdentifier(this JsonSchemaType schemaType)
+        {
+            if (schemaType.ToIdentifiers().Length != 1)
             {
-                JsonSchemaType.Null => "null",
-                JsonSchemaType.Boolean => "boolean",
-                JsonSchemaType.Integer => "integer",
-                JsonSchemaType.Number => "number",
-                JsonSchemaType.String => "string",
-                JsonSchemaType.Array => "array",
-                JsonSchemaType.Object => "object",
-                _ => null,
-            };
+                throw new OpenApiException($"Schema type {schemaType} must have exactly one identifier.");
+            }
+            return schemaType.ToFirstIdentifier();
         }
+
 #nullable restore
 
         /// <summary>
@@ -61,7 +88,7 @@
                 "null" => JsonSchemaType.Null,
                 "boolean" => JsonSchemaType.Boolean,
                 "integer" or "int" => JsonSchemaType.Integer,
                 "number" or "double" or "float" or "decimal"=> JsonSchemaType.Number,
                 "string" => JsonSchemaType.String,
                 "array" => JsonSchemaType.Array,
                 "object" => JsonSchemaType.Object,
@@ -70,18 +97,38 @@
             };
         }
 
+        /// <summary>
+        /// Converts a schema type's identifier into the enum equivalent
+        /// </summary>
+        /// <param name="identifier"></param>
+        /// <returns></returns>
+        public static JsonSchemaType? ToJsonSchemaType(this string[] identifier)
+        {
+            if (identifier == null)
+            {
+                return null;
+            }
+
+            JsonSchemaType type = 0;
+            foreach (var id in identifier)
+            {
+                type |= id.ToJsonSchemaType();
+            }
+            return type;
+        }
+
         private static readonly Dictionary<Type, Func<OpenApiSchema>> _simpleTypeToOpenApiSchema = new()
         {
             [typeof(bool)] = () => new() { Type = JsonSchemaType.Boolean },
             [typeof(byte)] = () => new() { Type = JsonSchemaType.String, Format = "byte" },
             [typeof(int)] = () => new() { Type = JsonSchemaType.Integer, Format = "int32" },
             [typeof(uint)] = () => new() { Type = JsonSchemaType.Integer, Format = "int32" },
             [typeof(long)] = () => new() { Type = JsonSchemaType.Integer, Format = "int64" },
             [typeof(ulong)] = () => new() { Type = JsonSchemaType.Integer, Format = "int64" },
             [typeof(float)] = () => new() { Type = JsonSchemaType.Number, Format = "float" },
             [typeof(double)] = () => new() { Type = JsonSchemaType.Number, Format = "double" },
             [typeof(decimal)] = () => new() { Type = JsonSchemaType.Number, Format = "double" },
             [typeof(DateTime)] = () => new() { Type = JsonSchemaType.String, Format = "date-time" },
             [typeof(DateTimeOffset)] = () => new() { Type = JsonSchemaType.String, Format = "date-time" },
             [typeof(Guid)] = () => new() { Type = JsonSchemaType.String, Format = "uuid" },
             [typeof(char)] = () => new() { Type = JsonSchemaType.String },
@@ -141,7 +188,7 @@
         }
 
         /// <summary>
-        /// Maps an JsonSchema data type and format to a simple type.
+        /// Maps a JsonSchema data type and format to a simple type.
         /// </summary>
         /// <param name="schema">The OpenApi data type</param>
         /// <returns>The simple type</returns>
@@ -153,37 +200,36 @@
                 throw new ArgumentNullException(nameof(schema));
             }
 
-            var type = ((schema.Type & ~JsonSchemaType.Null).ToIdentifier(), schema.Format?.ToLowerInvariant(), schema.Type & JsonSchemaType.Null) switch
+            var type = (schema.Type, schema.Format?.ToLowerInvariant()) switch
             {
-                ("integer" or "number", "int32", JsonSchemaType.Null) => typeof(int?),
-                ("integer" or "number", "int64", JsonSchemaType.Null) => typeof(long?),
-                ("integer", null, JsonSchemaType.Null) => typeof(long?),
-                ("number", "float", JsonSchemaType.Null) => typeof(float?),
-                ("number", "double", JsonSchemaType.Null) => typeof(double?),
-                ("number", null, JsonSchemaType.Null) => typeof(double?),
-                ("number", "decimal", JsonSchemaType.Null) => typeof(decimal?),
-                ("string", "byte", JsonSchemaType.Null) => typeof(byte?),
-                ("string", "date-time", JsonSchemaType.Null) => typeof(DateTimeOffset?),
-                ("string", "uuid", JsonSchemaType.Null) => typeof(Guid?),
-                ("string", "char", JsonSchemaType.Null) => typeof(char?),
-                ("boolean", null, JsonSchemaType.Null) => typeof(bool?),
-                ("boolean", null, _) => typeof(bool),
+                (JsonSchemaType.Integer | JsonSchemaType.Null or JsonSchemaType.Number | JsonSchemaType.Null, "int32") => typeof(int?),
+                (JsonSchemaType.Integer | JsonSchemaType.Null or JsonSchemaType.Number | JsonSchemaType.Null, "int64") => typeof(long?),
+                (JsonSchemaType.Integer | JsonSchemaType.Null, null) => typeof(long?),
+                (JsonSchemaType.Number | JsonSchemaType.Null, "float") => typeof(float?),
+                (JsonSchemaType.Number | JsonSchemaType.Null, "double") => typeof(double?),
+                (JsonSchemaType.Number | JsonSchemaType.Null, null) => typeof(double?),
+                (JsonSchemaType.Number | JsonSchemaType.Null, "decimal") => typeof(decimal?),
+                (JsonSchemaType.String | JsonSchemaType.Null, "byte") => typeof(byte?),
+                (JsonSchemaType.String | JsonSchemaType.Null, "date-time") => typeof(DateTimeOffset?),
+                (JsonSchemaType.String | JsonSchemaType.Null, "uuid") => typeof(Guid?),
+                (JsonSchemaType.String | JsonSchemaType.Null, "char") => typeof(char?),
+                (JsonSchemaType.Boolean | JsonSchemaType.Null, null) => typeof(bool?),
+                (JsonSchemaType.Boolean, null) => typeof(bool),
                 // integer is technically not valid with format, but we must provide some compatibility
-                ("integer" or "number", "int32", _) => typeof(int),
-                ("integer" or "number", "int64", _) => typeof(long),
-                ("integer", null, _) => typeof(long),
-                ("number", "float", _) => typeof(float),
-                ("number", "double", _) => typeof(double),
-                ("number", "decimal", _) => typeof(decimal),
-                ("number", null, _) => typeof(double),
-                ("string", "byte", _) => typeof(byte),
-                ("string", "date-time", _) => typeof(DateTimeOffset),
-                ("string", "uuid", _) => typeof(Guid),
-                ("string", "duration", _) => typeof(TimeSpan),
-                ("string", "char", _) => typeof(char),
-                ("string", null, _) => typeof(string),
-                ("object", null, _) => typeof(object),
-                ("string", "uri", _) => typeof(Uri),
+                (JsonSchemaType.Integer or JsonSchemaType.Number, "int32") => typeof(int),
+                (JsonSchemaType.Integer or JsonSchemaType.Number, "int64") => typeof(long),
+                (JsonSchemaType.Integer, null) => typeof(long),
+                (JsonSchemaType.Number, "float") => typeof(float),
+                (JsonSchemaType.Number, "double") => typeof(double),
+                (JsonSchemaType.Number, "decimal") => typeof(decimal),
+                (JsonSchemaType.Number, null) => typeof(double),
+                (JsonSchemaType.String, "byte") => typeof(byte),
+                (JsonSchemaType.String, "date-time") => typeof(DateTimeOffset),
+                (JsonSchemaType.String, "uuid") => typeof(Guid),
+                (JsonSchemaType.String, "char") => typeof(char),
+                (JsonSchemaType.String, null) => typeof(string),
+                (JsonSchemaType.Object, null) => typeof(object),
+                (JsonSchemaType.String, "uri") => typeof(Uri),
                 _ => typeof(string),
             };
 

src/Microsoft.OpenApi/Models/OpenApiSchema.cs

@@ -185,7 +185,7 @@
         /// Initializes a copy of <see cref="IOpenApiSchema"/> object
         /// </summary>
         /// <param name="schema">The schema object to copy from.</param>
         internal OpenApiSchema(IOpenApiSchema schema)
         {
             Utils.CheckArgumentNull(schema);
             Title = schema.Title ?? Title;
@@ -413,7 +413,7 @@
         internal void WriteAsItemsProperties(IOpenApiWriter writer)
         {
             // type
-            writer.WriteProperty(OpenApiConstants.Type, (Type & ~JsonSchemaType.Null).ToIdentifier());
+            writer.WriteProperty(OpenApiConstants.Type, (Type & ~JsonSchemaType.Null)?.ToFirstIdentifier());
 
             // format
             WriteFormatProperty(writer);
@@ -629,10 +629,10 @@
         private void SerializeTypeProperty(JsonSchemaType? type, IOpenApiWriter writer, OpenApiSpecVersion version)
         {
             // check whether nullable is true for upcasting purposes
-            var isNullable = (Type.HasValue && Type.Value.HasFlag(JsonSchemaType.Null))  ||
+            var isNullable = (Type.HasValue && Type.Value.HasFlag(JsonSchemaType.Null)) ||
                                 Extensions is not null &&
                                 Extensions.TryGetValue(OpenApiConstants.NullableExtension, out var nullExtRawValue) && 
-                                nullExtRawValue is OpenApiAny { Node: JsonNode jsonNode} &&
+                                nullExtRawValue is OpenApiAny { Node: JsonNode jsonNode } &&
                                 jsonNode.GetValueKind() is JsonValueKind.True;
             if (type is null)
             {
@@ -651,14 +651,14 @@
                         break;
                     case OpenApiSpecVersion.OpenApi3_0 when isNullable && type.Value == JsonSchemaType.Null:
                         writer.WriteProperty(OpenApiConstants.Nullable, true);
-                        writer.WriteProperty(OpenApiConstants.Type, JsonSchemaType.Object.ToIdentifier());
+                        writer.WriteProperty(OpenApiConstants.Type, JsonSchemaType.Object.ToFirstIdentifier());
                         break;
                     case OpenApiSpecVersion.OpenApi3_0 when isNullable && type.Value != JsonSchemaType.Null:
                         writer.WriteProperty(OpenApiConstants.Nullable, true);
-                        writer.WriteProperty(OpenApiConstants.Type, type.Value.ToIdentifier());
+                        writer.WriteProperty(OpenApiConstants.Type, type.Value.ToFirstIdentifier());
                         break;
                     default:
-                        writer.WriteProperty(OpenApiConstants.Type, type.Value.ToIdentifier());
+                        writer.WriteProperty(OpenApiConstants.Type, type.Value.ToFirstIdentifier());
                         break;
                 }
             }
@@ -674,7 +674,13 @@
                     var list = (from JsonSchemaType flag in jsonSchemaTypeValues
                                 where type.Value.HasFlag(flag)
                                 select flag).ToList();
-                    writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s.ToIdentifier()));
+                    writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) =>
+                    {
+                        foreach(var item in s.ToIdentifiers())
+                        {
+                            w.WriteValue(item);
+                        }
+                    });
                 }
             } 
         }
@@ -697,7 +703,7 @@
             var temporaryType = type | JsonSchemaType.Null;
             var list = (from JsonSchemaType flag in jsonSchemaTypeValues// Check if the flag is set in 'type' using a bitwise AND operation
                         where temporaryType.HasFlag(flag)
-                        select flag.ToIdentifier()).ToList();
+                        select flag.ToFirstIdentifier()).ToList();
             if (list.Count > 1)
             {
                 writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s));
@@ -734,7 +740,7 @@
                     if (schemaType.HasFlag(flag) && flag != JsonSchemaType.Null)
                     {
                         // Write the non-null flag value to the writer
-                        writer.WriteProperty(OpenApiConstants.Type, flag.ToIdentifier());
+                        writer.WriteProperty(OpenApiConstants.Type, flag.ToFirstIdentifier());
                     }
                 }
                 writer.WriteProperty(nullableProp, true);
@@ -747,7 +753,7 @@
                 }
                 else
                 {
-                    writer.WriteProperty(OpenApiConstants.Type, schemaType.ToIdentifier());
+                    writer.WriteProperty(OpenApiConstants.Type, schemaType.ToFirstIdentifier());
                 }
             }
         }

src/Microsoft.OpenApi/Validations/Rules/RuleHelpers.cs

@@ -1,6 +1,7 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT license.
 
+using System.Linq;
 using System.Text.Json;
 using System.Text.Json.Nodes;
 using Microsoft.OpenApi.Extensions;
@@ -41,7 +42,7 @@
             return true;
         }
 
         public static void ValidateDataTypeMismatch(
             IValidationContext context,
             string ruleName,
             JsonNode value,
@@ -55,7 +56,7 @@
             // convert value to JsonElement and access the ValueKind property to determine the type.
             var valueKind = value.GetValueKind();
 
-            var type = schema.Type.ToIdentifier();
+            var type = (schema.Type & ~JsonSchemaType.Null)?.ToFirstIdentifier();
             var format = schema.Format;
 
             // Before checking the type, check first if the schema allows null.
@@ -84,7 +85,7 @@
                     return;
                 }
 
                 foreach (var kvp in anyObject)
                 {
                     var key = kvp.Key;
                     context.Enter(key);
@@ -136,7 +137,7 @@
                 return;
             }
 
             if (type is "integer" or "number" && format is "int32")
             {
                 if (valueKind is not JsonValueKind.Number)
                 {
@@ -208,7 +209,7 @@
                 return;
             }
 
             if (type is "string" && format is "byte")
             {
                 if (valueKind is not JsonValueKind.String)
                 {

test/Microsoft.OpenApi.Readers.Tests/V31Tests/OpenApiSchemaTests.cs

@@ -9,11 +9,13 @@
 using FluentAssertions;
 using FluentAssertions.Equivalency;
 using Microsoft.OpenApi.Models;
+using Microsoft.OpenApi.Extensions;
 using Microsoft.OpenApi.Models.Interfaces;
 using Microsoft.OpenApi.Reader;
 using Microsoft.OpenApi.Tests;
 using Microsoft.OpenApi.Writers;
 using Xunit;
+using Microsoft.OpenApi.Exceptions;
 
 namespace Microsoft.OpenApi.Readers.Tests.V31Tests
 {
@@ -31,7 +33,7 @@ public static MemoryStream GetMemoryStream(string fileName)
 
         public OpenApiSchemaTests()
         {
-           OpenApiReaderRegistry.RegisterReader("yaml", new OpenApiYamlReader());
+            OpenApiReaderRegistry.RegisterReader("yaml", new OpenApiYamlReader());
         }
 
         [Fact]
@@ -298,8 +300,8 @@ public void CloningSchemaWithExamplesAndEnumsShouldSucceed()
             clone.Default = 6;
 
             // Assert
-            Assert.Equivalent(new int[] {1, 2, 3, 4}, clone.Enum.Select(static x => x.GetValue<int>()).ToArray());
-            Assert.Equivalent(new int[] {2, 3, 4}, clone.Examples.Select(static x => x.GetValue<int>()).ToArray());
+            Assert.Equivalent(new int[] { 1, 2, 3, 4 }, clone.Enum.Select(static x => x.GetValue<int>()).ToArray());
+            Assert.Equivalent(new int[] { 2, 3, 4 }, clone.Examples.Select(static x => x.GetValue<int>()).ToArray());
             Assert.Equivalent(6, clone.Default.GetValue<int>());
         }
 
@@ -398,7 +400,7 @@ public void SerializeSchemaWithTypeArrayAndNullableDoesntEmitType()
             schema.SerializeAsV2(new OpenApiYamlWriter(writer));
             var schemaString = writer.ToString();
 
-            Assert.Equal(expected.MakeLineBreaksEnvironmentNeutral(), schemaString.MakeLineBreaksEnvironmentNeutral()); 
+            Assert.Equal(expected.MakeLineBreaksEnvironmentNeutral(), schemaString.MakeLineBreaksEnvironmentNeutral());
         }
 
         [Theory]
@@ -506,7 +508,7 @@ public async Task ParseSchemaWithConstWorks()
         }
 
         [Fact]
-        public void ParseSchemaWithUnrecognizedKeywordsWorks() 
+        public void ParseSchemaWithUnrecognizedKeywordsWorks()
         {
             var input = @"{
     ""type"": ""string"",
@@ -520,5 +522,46 @@ public void ParseSchemaWithUnrecognizedKeywordsWorks()
             Assert.Equal(2, schema.UnrecognizedKeywords.Count);
         }
 
+        [Theory]
+        [InlineData(JsonSchemaType.Integer | JsonSchemaType.String, new[] { "integer", "string" })]
+        [InlineData(JsonSchemaType.Integer | JsonSchemaType.Null, new[] { "integer", "null" })]
+        [InlineData(JsonSchemaType.Integer, new[] { "integer" })]
+        public void NormalizeFlaggableJsonSchemaTypeEnumWorks(JsonSchemaType type, string[] expected)
+        {
+            var schema = new OpenApiSchema
+            {
+                Type = type
+            };
+
+            var actual = schema.Type.ToIdentifiers();
+            Assert.Equal(expected, actual);
+        }
+
+        [Theory]
+        [InlineData(new[] { "integer", "string" }, JsonSchemaType.Integer | JsonSchemaType.String)]
+        [InlineData(new[] { "integer", "null" }, JsonSchemaType.Integer | JsonSchemaType.Null)]
+        [InlineData(new[] { "integer" }, JsonSchemaType.Integer)]
+        public void ArrayIdentifierToEnumConversionWorks(string[] type, JsonSchemaType expected)
+        {
+            var actual = type.ToJsonSchemaType();
+            Assert.Equal(expected, actual);
+        }
+
+        [Fact]
+        public void StringIdentifierToEnumConversionWorks()
+        {
+            var actual = "integer".ToJsonSchemaType();
+            Assert.Equal(JsonSchemaType.Integer, actual);
+        }
+
+        [Fact]
+        public void ReturnSingleIdentifierWorks()
+        {
+            var type = JsonSchemaType.Integer;
+            var types = JsonSchemaType.Integer | JsonSchemaType.Null;
+
+            Assert.Equal("integer", type.ToSingleIdentifier());
+            Assert.Throws<OpenApiException>(() => types.ToSingleIdentifier());
+        }
     }
 }

test/Microsoft.OpenApi.Tests/PublicApi/PublicApi.approved.txt

@@ -190,9 +190,11 @@ namespace Microsoft.OpenApi.Extensions
     {
         public static System.Type MapOpenApiPrimitiveTypeToSimpleType(this Microsoft.OpenApi.Models.OpenApiSchema schema) { }
         public static Microsoft.OpenApi.Models.OpenApiSchema MapTypeToOpenApiPrimitiveType(this System.Type type) { }
-        public static string? ToIdentifier(this Microsoft.OpenApi.Models.JsonSchemaType schemaType) { }
-        public static string? ToIdentifier(this Microsoft.OpenApi.Models.JsonSchemaType? schemaType) { }
+        public static string[] ToIdentifiers(this Microsoft.OpenApi.Models.JsonSchemaType schemaType) { }
+        public static string[]? ToIdentifiers(this Microsoft.OpenApi.Models.JsonSchemaType? schemaType) { }
         public static Microsoft.OpenApi.Models.JsonSchemaType ToJsonSchemaType(this string identifier) { }
+        public static Microsoft.OpenApi.Models.JsonSchemaType? ToJsonSchemaType(this string[] identifier) { }
+        public static string ToSingleIdentifier(this Microsoft.OpenApi.Models.JsonSchemaType schemaType) { }
     }
 }
 namespace Microsoft.OpenApi.Interfaces