better error message for empty operationId

wing328 · wing328 · commit dd34d75f0038 · 2015-10-23T12:46:44.000+08:00
diff --git a/modules/swagger-codegen/src/main/java/io/swagger/codegen/DefaultGenerator.java b/modules/swagger-codegen/src/main/java/io/swagger/codegen/DefaultGenerator.java
@@ -531,7 +531,8 @@ public void processOperation(String resourcePath, String httpMethod, Operation o
                         + "  Tag: " + tag + "\n"//
                         + "  Operation: " + operation.getOperationId() + "\n" //
                         + "  Resource: " + httpMethod + " " + resourcePath + "\n"//
-                        + "  Definitions: " + swagger.getDefinitions();
+                        + "  Definitions: " + swagger.getDefinitions() + "\n"  //
+                        + "  Exception: " + ex.getMessage();
                     throw new RuntimeException(msg, ex);
                 }
             }
diff --git a/modules/swagger-codegen/src/main/java/io/swagger/codegen/languages/JavaClientCodegen.java b/modules/swagger-codegen/src/main/java/io/swagger/codegen/languages/JavaClientCodegen.java
@@ -378,7 +378,7 @@ public String getSwaggerType(Property p) {
     public String toOperationId(String operationId) {
         // throw exception if method name is empty
         if (StringUtils.isEmpty(operationId)) {
-            throw new RuntimeException("Empty method name (operationId) not allowed");
+            throw new RuntimeException("Empty method/operation name (operationId) not allowed");
         }
 
         // method name cannot use reserved keyword, e.g. return

Original file line number	Diff line number	Diff line change
`@@ -531,7 +531,8 @@ public void processOperation(String resourcePath, String httpMethod, Operation o`
`531`	`531`	`+ " Tag: " + tag + "\n"//`
`532`	`532`	`+ " Operation: " + operation.getOperationId() + "\n" //`
`533`	`533`	`+ " Resource: " + httpMethod + " " + resourcePath + "\n"//`
`534`		`- + " Definitions: " + swagger.getDefinitions();`
	`534`	`+ + " Definitions: " + swagger.getDefinitions() + "\n" //`
	`535`	`+ + " Exception: " + ex.getMessage();`
`535`	`536`	`throw new RuntimeException(msg, ex);`
`536`	`537`	`}`
`537`	`538`	`}`
Original file line number	Diff line number	Diff line change
`@@ -378,7 +378,7 @@ public String getSwaggerType(Property p) {`
`378`	`378`	`public String toOperationId(String operationId) {`
`379`	`379`	`// throw exception if method name is empty`
`380`	`380`	`if (StringUtils.isEmpty(operationId)) {`
`381`		`- throw new RuntimeException("Empty method name (operationId) not allowed");`
	`381`	`+ throw new RuntimeException("Empty method/operation name (operationId) not allowed");`
`382`	`382`	`}`
`383`	`383`
`384`	`384`	`// method name cannot use reserved keyword, e.g. return`