Add allOf required spec and fix items branch (PaloAltoNetworks#1190)

* Add allOf required spec and fix items branch

* Place allOfRequired example in allOf category

* move test to allOf

* Remove duplicate allOfRequired spec

Author: sserrata

demo/examples/tests/allOf.yaml

@@ -367,6 +367,51 @@ paths:
                       otherOuterProp:
                         type: string
 
+  /allof-parent-required:
+    get:
+      tags:
+        - allOf
+      summary: allOf inherits parent required
+      description: |
+        Parent object marks the "pet" property as required while its schema is composed via allOf.
+
+        Schema:
+        ```yaml
+        type: object
+        properties:
+          pet:
+            allOf:
+              - type: object
+                properties:
+                  name:
+                    type: string
+              - type: object
+                properties:
+                  age:
+                    type: integer
+        required: [pet]
+        ```
+      responses:
+        "200":
+          description: Successful response
+          content:
+            application/json:
+              schema:
+                type: object
+                properties:
+                  pet:
+                    allOf:
+                      - type: object
+                        properties:
+                          name:
+                            type: string
+                      - type: object
+                        properties:
+                          age:
+                            type: integer
+                required:
+                  - pet
+
 components:
   schemas:
     # Your existing schemas are integrated here.

packages/docusaurus-theme-openapi-docs/src/theme/Schema/index.tsx

@@ -725,9 +725,7 @@ const SchemaEdge: React.FC<SchemaEdgeProps> = ({
           name={name}
           schemaName={mergedSchemaName}
           required={
-            Array.isArray(mergedSchemas.required)
-              ? mergedSchemas.required.includes(name)
-              : mergedSchemas.required
+            Array.isArray(required) ? required.includes(name) : required
           }
           nullable={mergedSchemas.nullable}
           schema={mergedSchemas}
@@ -742,9 +740,7 @@ const SchemaEdge: React.FC<SchemaEdgeProps> = ({
           name={name}
           schemaName={mergedSchemaName}
           required={
-            Array.isArray(mergedSchemas.required)
-              ? mergedSchemas.required.includes(name)
-              : mergedSchemas.required
+            Array.isArray(required) ? required.includes(name) : required
           }
           nullable={mergedSchemas.nullable}
           schema={mergedSchemas}
@@ -754,18 +750,18 @@ const SchemaEdge: React.FC<SchemaEdgeProps> = ({
     }
 
     if (mergedSchemas.items?.properties) {
-      <SchemaNodeDetails
-        name={name}
-        schemaName={mergedSchemaName}
-        required={
-          Array.isArray(mergedSchemas.required)
-            ? mergedSchemas.required.includes(name)
-            : mergedSchemas.required
-        }
-        nullable={mergedSchemas.nullable}
-        schema={mergedSchemas}
-        schemaType={schemaType}
-      />;
+      return (
+        <SchemaNodeDetails
+          name={name}
+          schemaName={mergedSchemaName}
+          required={
+            Array.isArray(required) ? required.includes(name) : required
+          }
+          nullable={mergedSchemas.nullable}
+          schema={mergedSchemas}
+          schemaType={schemaType}
+        />
+      );
     }
 
     return (