Use an options argument instead of various arguments, for childrenIterator()

Author: Joris-van-der-Wel

lib/SymbolTree.js

@@ -357,10 +357,12 @@ class SymbolTree {
          * @method childrenIterator
          * @memberOf module:symbol-tree#
          * @param {Object} parent
-         * @param {Boolean} [reverse=false]
+         * @param {Object} [options]
+         * @param {Boolean} [options.reverse=false]
          * @return {Object} An iterable iterator (ES6)
          */
-        childrenIterator(parent, reverse) {
+        childrenIterator(parent, options) {
+                const reverse = options && options.reverse;
                 const parentNode = this._node(parent);
 
                 return new TreeIterator(

test/SymbolTree.js

@@ -715,7 +715,7 @@ test('children iterator reverse', function(t) {
 
         const results = [];
 
-        for (const object of tree.childrenIterator(a, true)) {
+        for (const object of tree.childrenIterator(a, {reverse: true})) {
                 results.push(object);
         }
         t.deepEqual([ac, ab, aa], results);