fix: openapischema schema property is now a Uri

Signed-off-by: Vincent Biret <[email protected]>

Author: baywet

src/Microsoft.OpenApi/Models/Interfaces/IOpenApiSchema.cs

@@ -1,4 +1,5 @@
-using System.Collections.Generic;
+using System;
+using System.Collections.Generic;
 using System.Text.Json.Nodes;
 using Microsoft.OpenApi.Interfaces;
 
@@ -19,7 +20,7 @@ public interface IOpenApiSchema : IOpenApiDescribedElement, IOpenApiSerializable
     /// <summary>
     /// $schema, a JSON Schema dialect identifier. Value must be a URI
     /// </summary>
-    public string Schema { get; }
+    public Uri Schema { get; }
 
     /// <summary>
     /// $id - Identifies a schema resource with its canonical URI.

src/Microsoft.OpenApi/Models/OpenApiSchema.cs

@@ -24,7 +24,7 @@ public class OpenApiSchema : IOpenApiReferenceable, IOpenApiExtensible, IOpenApi
         public string Title { get; set; }
 
         /// <inheritdoc />
-        public string Schema { get; set; }
+        public Uri Schema { get; set; }
 
         /// <inheritdoc />
         public string Id { get; set; }
@@ -400,7 +400,7 @@ public void SerializeAsV2(IOpenApiWriter writer)
         internal void WriteJsonSchemaKeywords(IOpenApiWriter writer)
         {
             writer.WriteProperty(OpenApiConstants.Id, Id);
-            writer.WriteProperty(OpenApiConstants.DollarSchema, Schema);
+            writer.WriteProperty(OpenApiConstants.DollarSchema, Schema?.ToString());
             writer.WriteProperty(OpenApiConstants.Comment, Comment);
             writer.WriteProperty(OpenApiConstants.Const, Const);
             writer.WriteOptionalMap(OpenApiConstants.Vocabulary, Vocabulary, (w, s) => w.WriteValue(s));

src/Microsoft.OpenApi/Models/References/OpenApiSchemaReference.cs

@@ -52,7 +52,7 @@ public string Description
         /// <inheritdoc/>
         public string Title { get => Target?.Title; }
         /// <inheritdoc/>
-        public string Schema { get => Target?.Schema; }
+        public Uri Schema { get => Target?.Schema; }
         /// <inheritdoc/>
         public string Id { get => Target?.Id; }
         /// <inheritdoc/>

src/Microsoft.OpenApi/Reader/V31/OpenApiSchemaDeserializer.cs

@@ -23,7 +23,7 @@ internal static partial class OpenApiV31Deserializer
             },
             {
                 "$schema",
-                (o, n, _) => o.Schema = n.GetScalarValue()
+                (o, n, _) => { if (n.GetScalarValue() is string {} sSchema && Uri.TryCreate(sSchema, UriKind.Absolute, out var schema)) {o.Schema = schema;}}
             },
             {
                 "$id",

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

@@ -38,7 +38,7 @@ public async Task ParseBasicV31SchemaShouldSucceed()
             var expectedObject = new OpenApiSchema()
             {
                 Id = "https://example.com/arrays.schema.json",
-                Schema = "https://json-schema.org/draft/2020-12/schema",
+                Schema = new Uri("https://json-schema.org/draft/2020-12/schema"),
                 Description = "A representation of a person, company, organization, or place",
                 Type = JsonSchemaType.Object,
                 Properties = new Dictionary<string, IOpenApiSchema>
@@ -124,7 +124,7 @@ public void ParseSchemaWithTypeArrayWorks()
             var expected = new OpenApiSchema()
             {
                 Id = "https://example.com/arrays.schema.json",
-                Schema = "https://json-schema.org/draft/2020-12/schema",
+                Schema = new Uri("https://json-schema.org/draft/2020-12/schema"),
                 Description = "A representation of a person, company, organization, or place",
                 Type = JsonSchemaType.Object | JsonSchemaType.Null
             };

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

@@ -451,7 +451,7 @@ namespace Microsoft.OpenApi.Models.Interfaces
         System.Collections.Generic.IDictionary<string, Microsoft.OpenApi.Models.Interfaces.IOpenApiSchema> Properties { get; }
         bool ReadOnly { get; }
         System.Collections.Generic.ISet<string> Required { get; }
-        string Schema { get; }
+        System.Uri Schema { get; }
         string Title { get; }
         Microsoft.OpenApi.Models.JsonSchemaType? Type { get; }
         bool UnEvaluatedProperties { get; }
@@ -1056,7 +1056,7 @@ namespace Microsoft.OpenApi.Models
         public System.Collections.Generic.IDictionary<string, Microsoft.OpenApi.Models.Interfaces.IOpenApiSchema> Properties { get; set; }
         public bool ReadOnly { get; set; }
         public System.Collections.Generic.ISet<string> Required { get; set; }
-        public string Schema { get; set; }
+        public System.Uri Schema { get; set; }
         public string Title { get; set; }
         public Microsoft.OpenApi.Models.JsonSchemaType? Type { get; set; }
         public bool UnEvaluatedProperties { get; set; }
@@ -1409,7 +1409,7 @@ namespace Microsoft.OpenApi.Models.References
         public System.Collections.Generic.IDictionary<string, Microsoft.OpenApi.Models.Interfaces.IOpenApiSchema> Properties { get; }
         public bool ReadOnly { get; }
         public System.Collections.Generic.ISet<string> Required { get; }
-        public string Schema { get; }
+        public System.Uri Schema { get; }
         public string Title { get; }
         public Microsoft.OpenApi.Models.JsonSchemaType? Type { get; }
         public bool UnEvaluatedProperties { get; }