oas-toolkit 0.5.3 → 0.6.0

package/merger.js

@@ -1,4 +1,5 @@
 const mergician = require("mergician");
+const traverse = require("traverse");
 const isEqual = require("lodash.isequal");
 const uniqWith = require("lodash.uniqwith");
 
@@ -28,6 +29,26 @@ function merge(objects, options) {
     }
   }
 
+  // Unique values that are at a non-deterministic path
+  combinedSpec = traverse(combinedSpec).map(function () {
+    if (["oneOf", "allOf", "anyOf"].includes(this.key)) {
+      this.node = uniqWith(this.node, isEqual);
+    }
+  });
+
+  // Validate that $ref is the only thing at a specific level
+  combinedSpec = traverse(combinedSpec).map(function () {
+    if (["oneOf", "allOf", "anyOf", "$ref"].includes(this.key)) {
+      if (Object.keys(this.parent.node).length > 1) {
+        throw new Error(
+          `Cannot have ${
+            this.key
+          } and other properties at the same level: ${this.path.join(".")}`
+        );
+      }
+    }
+  });
+
   return combinedSpec;
 }
 

package/merger.test.js

@@ -334,4 +334,190 @@ describe("returns unique items for:", () => {
       security: [{ appKey: [] }],
     });
   });
+
+  it("oneOf", () => {
+    expect(
+      merger([
+        {
+          components: {
+            schemas: {
+              DemoError: {
+                oneOf: [
+                  { $ref: "#/components/schemas/One" },
+                  { $ref: "#/components/schemas/Two" },
+                  { $ref: "#/components/schemas/Three" },
+                ],
+              },
+            },
+          },
+        },
+        {
+          components: {
+            schemas: {
+              DemoError: {
+                oneOf: [
+                  { $ref: "#/components/schemas/One" },
+                  { $ref: "#/components/schemas/Two" },
+                  { $ref: "#/components/schemas/Four" },
+                ],
+              },
+              AnotherError: {
+                $ref: "#/components/schemas/One",
+              },
+            },
+          },
+        },
+      ])
+    ).toEqual({
+      components: {
+        schemas: {
+          DemoError: {
+            oneOf: [
+              { $ref: "#/components/schemas/One" },
+              { $ref: "#/components/schemas/Two" },
+              { $ref: "#/components/schemas/Three" },
+              { $ref: "#/components/schemas/Four" },
+            ],
+          },
+          AnotherError: {
+            $ref: "#/components/schemas/One",
+          },
+        },
+      },
+    });
+  });
+
+  it("allOf", () => {
+    expect(
+      merger([
+        {
+          components: {
+            schemas: {
+              DemoError: {
+                allOf: [{ $ref: "#/components/schemas/One" }],
+              },
+            },
+          },
+        },
+        {
+          components: {
+            schemas: {
+              DemoError: {
+                allOf: [{ $ref: "#/components/schemas/One" }],
+              },
+            },
+          },
+        },
+      ])
+    ).toEqual({
+      components: {
+        schemas: {
+          DemoError: {
+            allOf: [{ $ref: "#/components/schemas/One" }],
+          },
+        },
+      },
+    });
+  });
+
+  it("anyOf", () => {
+    expect(
+      merger([
+        {
+          components: {
+            schemas: {
+              DemoError: {
+                allOf: [{ $ref: "#/components/schemas/One" }],
+              },
+            },
+          },
+        },
+        {
+          components: {
+            schemas: {
+              DemoError: {
+                allOf: [{ $ref: "#/components/schemas/One" }],
+              },
+            },
+          },
+        },
+      ])
+    ).toEqual({
+      components: {
+        schemas: {
+          DemoError: {
+            allOf: [{ $ref: "#/components/schemas/One" }],
+          },
+        },
+      },
+    });
+  });
+});
+
+describe("validates the final output", () => {
+  it("throws when $ref is detected alongside other values", () => {
+    expect(() => {
+      merger([
+        {
+          components: {
+            schemas: {
+              DemoError: {
+                type: "object",
+                properties: {
+                  code: {
+                    type: "string",
+                    example: "ERROR_ABC",
+                  },
+                },
+              },
+            },
+          },
+        },
+        {
+          components: {
+            schemas: {
+              DemoError: {
+                $ref: "#/components/schemas/AnotherError",
+              },
+            },
+          },
+        },
+      ]);
+    }).toThrow(
+      "Cannot have $ref and other properties at the same level: components.schemas.DemoError.$ref"
+    );
+  });
+
+  it("throws when $ref is detected alongside other values", () => {
+    expect(() => {
+      merger([
+        {
+          components: {
+            schemas: {
+              DemoError: {
+                type: "object",
+                properties: {
+                  code: {
+                    type: "string",
+                    example: "ERROR_ABC",
+                  },
+                },
+              },
+            },
+          },
+        },
+        {
+          components: {
+            schemas: {
+              DemoError: {
+                oneOf: ["#/components/schemas/AnotherError"],
+              },
+            },
+          },
+        },
+      ]);
+    }).toThrow(
+      "Cannot have oneOf and other properties at the same level: components.schemas.DemoError.oneOf"
+    );
+  });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oas-toolkit",
-  "version": "0.5.3",
+  "version": "0.6.0",
   "description": "",
   "main": "index.js",
   "bin": {