fix(specmap): fix resolver being affected by the order of schemas (#3857)

Refs swagger-api/swagger-ui#10096

Author: glowcloud

src/resolver/specmap/index.js

@@ -136,7 +136,11 @@ class SpecMap {
               if (!traversed) {
                 if (isObj) {
                   // Only store the ref if it exists
-                  if (specmap.allowMetaPatches && objRef) {
+                  if (
+                    specmap.allowMetaPatches &&
+                    objRef &&
+                    isSubPath(pathDiscriminator, updatedPath)
+                  ) {
                     refCache[objRef] = true;
                   }
                   yield* traverse(val, updatedPath, patch);

test/subtree-resolver/strategies/openapi-2--3-0.js

@@ -626,6 +626,33 @@ describe('subtree $ref resolver', () => {
     });
   });
 
+  test('should resolve nested allOf with $ref with no influence of order of schemas', async () => {
+    const input = {
+      components: {
+        schemas: {
+          Second: { allOf: [{ $ref: '#/components/schemas/Third' }] },
+          First: { allOf: [{ $ref: '#/components/schemas/Second' }] },
+          Third: { allOf: [{ $ref: '#/components/schemas/Fourth' }] },
+          Fourth: { allOf: [{ $ref: '#/components/schemas/Fifth' }] },
+          Fifth: { properties: { a: { type: 'string' } } },
+        },
+      },
+    };
+
+    const res = await resolve(input, ['components', 'schemas', 'First']);
+
+    expect(res).toEqual({
+      errors: [],
+      spec: {
+        properties: {
+          a: {
+            type: 'string',
+          },
+        },
+      },
+    });
+  });
+
   test('should resolve complex allOf correctly', async () => {
     const input = {
       definitions: {
@@ -679,6 +706,7 @@ describe('subtree $ref resolver', () => {
       },
     });
   });
+
   test('should fully resolve across remote documents correctly', async () => {
     const mockPool = mockAgent.get('http://example.com');
     mockPool.intercept({ path: '/remote.json' }).reply(