Merge pull request #821 from DeagleGross/dmkorolev/resolve-local-file-references

[bugfix] Resolve local file reference with Schema type properly

Author: darrelmiller

src/Microsoft.OpenApi.Readers/V3/OpenApiV3VersionService.cs

@@ -75,18 +75,6 @@ public OpenApiReference ConvertToOpenApiReference(
                 var segments = reference.Split('#');
                 if (segments.Length == 1)
                 {
-                    // Either this is an external reference as an entire file
-                    // or a simple string-style reference for tag and security scheme.
-                    if (type == null)
-                    {
-                        // "$ref": "Pet.json"
-                        return new OpenApiReference
-                        {
-                            Type = type,
-                            ExternalResource = segments[0]
-                        };
-                    }
-
                     if (type == ReferenceType.Tag || type == ReferenceType.SecurityScheme)
                     {
                         return new OpenApiReference
@@ -95,6 +83,14 @@ public OpenApiReference ConvertToOpenApiReference(
                             Id = reference
                         };
                     }
+
+                    // Either this is an external reference as an entire file
+                    // or a simple string-style reference for tag and security scheme.
+                    return new OpenApiReference
+                    {
+                        Type = type,
+                        ExternalResource = segments[0]
+                    };
                 }
                 else if (segments.Length == 2)
                 {

test/Microsoft.OpenApi.Readers.Tests/ReferenceService/ConvertToOpenApiReferenceV3Tests.cs

@@ -108,5 +108,21 @@ public void ParseSecuritySchemeReference()
             reference.ExternalResource.Should().BeNull();
             reference.Id.Should().Be(id);
         }
+
+        [Fact]
+        public void ParseLocalFileReference()
+        {
+            // Arrange
+            var versionService = new OpenApiV3VersionService(Diagnostic);
+            var referenceType = ReferenceType.Schema;
+            var input = $"../schemas/collection.json";
+
+            // Act
+            var reference = versionService.ConvertToOpenApiReference(input, referenceType);
+
+            // Assert
+            reference.Type.Should().Be(referenceType);
+            reference.ExternalResource.Should().Be(input);
+        }
     }
 }