show more available method signatures in embedded API viewer

Author: daspilker

docs/Home.md

@@ -27,6 +27,8 @@ Browse the Jenkins issue tracker to see any [open issues](https://issues.jenkins
 
 ## Release Notes
 * 1.57 (unreleased)
+ * Show more available method signatures in embedded API viewer
+   ([#982](https://github.com/jenkinsci/job-dsl-plugin/pull/982))
 * 1.56 (January 06 2016)
  * Fixed support for
    [Config File Provider Plugin](https://wiki.jenkins-ci.org/display/JENKINS/Config+File+Provider+Plugin)

job-dsl-plugin/src/main/groovy/javaposse/jobdsl/plugin/EmbeddedApiDocGenerator.groovy

@@ -91,13 +91,43 @@ class EmbeddedApiDocGenerator {
             }
             knownMethods.addAll(extensions*.name)
 
+            methods.each { JSONObject method ->
+                if (!hasOptionalClosureSignature(method)) {
+                    knownMethods.remove(method.getString('name'))
+                }
+            }
+
             Map<String, DescribableModel> symbols = findDescribableModels(extensibleContextClass, knownMethods)
             symbols.sort().each { String symbol, DescribableModel model ->
-                methods << generateMethod(symbol, model)
+                JSONObject method = methods.find { it.getString('name') == symbol } as JSONObject
+                if (method) {
+                    method.getJSONArray('signatures').add(generateSignature(model))
+                } else {
+                    methods << generateMethod(symbol, model)
+                }
             }
         }
     }
 
+    private static boolean hasOptionalClosureSignature(JSONObject method) {
+        method.getJSONArray('signatures').any { JSONObject signature ->
+            isOptionalClosureSignature(signature)
+        }
+    }
+
+    private static boolean isOptionalClosureSignature(JSONObject signature) {
+        if (!signature.has('parameters')) {
+            return true
+        }
+        JSONArray parameters = signature.getJSONArray('parameters')
+        if (parameters.size() > 1) {
+            return false
+        } else if (parameters.empty) {
+            return true
+        }
+        parameters[0].getString('type') == 'Closure'
+    }
+
     /**
      * Generates an extension method for an {@link ContextExtensionPoint} of a built-in context.
      */

job-dsl-plugin/src/test/resources/expected-dsl.json

@@ -121,6 +121,10 @@
                 }
               ],
               "extension": true
+            },
+            {
+              "parameters": [],
+              "generated": true
             }
           ]
         },