commented out skipped code, might need it later.

Author: daveshanley

document_test.go

@@ -1471,3 +1471,19 @@ func TestDocument_Issue418(t *testing.T) {
 	assert.Len(t, errs, 0)
 	assert.Len(t, m.Model.Index.GetResolver().GetResolvingErrors(), 0)
 }
+
+func TestDocument_Issue418_NoFile(t *testing.T) {
+
+	spec, _ := os.ReadFile("test_specs/nested_files/openapi-issue-418.yaml")
+
+	doc, err := NewDocumentWithConfiguration(spec, &datamodel.DocumentConfiguration{
+		AllowFileReferences: true,
+		BasePath:            "test_specs/nested_files",
+	})
+	if err != nil {
+		panic(err)
+	}
+	m, errs := doc.BuildV3Model()
+	assert.Len(t, errs, 0)
+	assert.Len(t, m.Model.Index.GetResolver().GetResolvingErrors(), 0)
+}

index/find_component.go

@@ -44,16 +44,20 @@ func (index *SpecIndex) FindComponent(ctx context.Context, componentId string) *
 
 			// check if the context has a root index set, if so this is a deep search that has moved through multiple
 			// indexes and we need to adjust the URI to reflect the location of the root index.
-			if ctx.Value(RootIndexKey) != nil {
-				rootIndex := ctx.Value(RootIndexKey).(*SpecIndex)
-				if rootIndex != nil && rootIndex.specAbsolutePath != "" {
-					dir := filepath.Dir(rootIndex.specAbsolutePath)
-					// create an absolute path to the file.
-					absoluteFilePath := filepath.Join(dir, componentId)
-					// split into a URI.
-					uri = []string{absoluteFilePath}
-				}
-			}
+			//
+			// the below code has been commended out due to being handled in the index. Keeping it for legacy and for
+			// future bugs.
+			//
+			//if ctx.Value(RootIndexKey) != nil {
+			//	rootIndex := ctx.Value(RootIndexKey).(*SpecIndex)
+			//	if rootIndex != nil && rootIndex.specAbsolutePath != "" {
+			//		dir := filepath.Dir(rootIndex.specAbsolutePath)
+			//		// create an absolute path to the file.
+			//		absoluteFilePath := filepath.Join(dir, componentId)
+			//		// split into a URI.
+			//		uri = []string{absoluteFilePath}
+			//	}
+			//}
 
 			return index.lookupRolodex(uri)
 		}