DOC-13343 call allOf recursively (#852)

Author: torcolvin

lib/helpers/lib/each-openapi-module.js

@@ -137,8 +137,10 @@ function child_entries(node) {
                 format_additionalProperties(node.additionalProperties) ]]
             : []
 
-    const via_allOf = node.allOf?.flatMap(
-        a => Object.entries(a.properties || {})) || []
+    // allOf can be nested, so call child_entries recursively
+    const via_allOf = (node.allOf || []).flatMap(subNode =>
+      child_entries(subNode)
+    )
 
     return [...via_properties, ...via_additionalProperties, ...via_allOf]
 }

test/fixtures/toc-bootstrap.html

@@ -73,7 +73,19 @@
          }
       },
       console: {
-
+         <a href="#logging-console-collation_buffer_size">collation_buffer_size</a>: 10,
+         <a href="#logging-console-color_enabled">color_enabled</a>: false,
+         <a href="#logging-console-enabled">enabled</a>: false,
+         <a href="#logging-console-file_output">file_output</a>: "string",
+         <a href="#logging-console-log_keys">log_keys</a>: ["CRUD,HTTP,Query"...],
+         <a href="#logging-console-log_level">log_level</a>: "info",
+         <a href="#logging-console-rotation">rotation</a>: {
+            <a href="#logging-console-rotation-localtime">localtime</a>: false,
+            <a href="#logging-console-rotation-max_age">max_age</a>: 0,
+            <a href="#logging-console-rotation-max_size">max_size</a>: 100,
+            <a href="#logging-console-rotation-rotated_logs_size_limit">rotated_logs_size_limit</a>: 1024,
+            <a href="#logging-console-rotation-rotation_interval">rotation_interval</a>: "0"
+         }
       },
       <a href="#logging-debug">debug</a>: {
          <a href="#logging-debug-collation_buffer_size">collation_buffer_size</a>: 1000,

test/test-each-openapi.js

@@ -310,6 +310,21 @@ describe('child_entries', function () {
         ['bar', {type: 'number'}]
       ])
   })
+
+  ok('via allOf recursive', function () {
+    assert.deepEqual(
+      child_entries({
+        allOf: [
+          {properties: {'outer': {type: 'string'}}},
+          {allOf: [{properties: {'inner': {type: 'number'}}}]}
+        ]
+      }),
+      [
+        ['outer', {type: 'string'}],
+        ['inner', {type: 'number'}]
+      ])
+  })
+
 })
 
 // Helper function to clean up whitespace in strings for comparison
@@ -318,4 +333,4 @@ function assert_equal_no_whitespace (actual, expected) {
   assert.equal(
     cleanup_whitespace(actual),
     cleanup_whitespace(expected))
-}
+}