Fix review suggestions

ballerina-platform · Feb 28, 2024 · b42865e · b42865e
1 parent 9f0bf32
commit b42865e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/openapi-cli/src/main/java/io/ballerina/openapi/cmd/BallerinaCodeGenerator.java b/openapi-cli/src/main/java/io/ballerina/openapi/cmd/BallerinaCodeGenerator.java
@@ -111,8 +111,9 @@ public void generateClientAndService(String definitionPath, String serviceName,
         List<String> complexPaths = GeneratorUtils.getComplexPaths(openAPIDef);
         if (!complexPaths.isEmpty()) {
             isResource = false;
-            outStream.println("remote function(s) will be generated for client and the service generation can not be " +
-                    "proceed due to the given openapi definition contains following complex path(s):");
+            outStream.println("WARNING: remote function(s) will be generated for client and the service" +
+                    " generation can not be proceed due to the given openapi definition contains following" +
+                    " complex path(s):");
             for (String path: complexPaths) {
                 outStream.println(path);
             }
@@ -360,8 +361,8 @@ private List<GenSrcFile> generateClientFiles(Path openAPI, Filter filter, boolea
         // Validate the service generation
         List<String> complexPaths = GeneratorUtils.getComplexPaths(openAPIDef);
         if (!complexPaths.isEmpty()) {
-            outStream.println("remote function(s) will be generated for client due to the given openapi definition" +
-                    " contains following complex path(s):");
+            outStream.println("WARNING: remote function(s) will be generated for client due to the given openapi " +
+                    "definition contains following complex path(s):");
             for (String path: complexPaths) {
                 outStream.println(path);
             }

diff --git a/openapi-cli/src/test/java/io/ballerina/openapi/cmd/OpenAPICmdTest.java b/openapi-cli/src/test/java/io/ballerina/openapi/cmd/OpenAPICmdTest.java
@@ -704,7 +704,7 @@ public void testForComplexPathInClient() {
         OpenApiCmd cmd = new OpenApiCmd(standardOut, tmpDir, false);
         new CommandLine(cmd).parseArgs(args);
         cmd.execute();
-        Assert.assertTrue(outputStream.toString().contains("remote function(s) will be generated for client " +
+        Assert.assertTrue(outputStream.toString().contains("WARNING: remote function(s) will be generated for client " +
                 "due to the given openapi definition contains following complex path(s):" + System.lineSeparator() +
                 "/v4/spreadsheets/{spreadsheetId}/sheets/{sheetId}:copyTo" + System.lineSeparator() +
                 "/v4/spreadsheets/{spreadsheetId}/sheets/{sheetId}:copyFrom" + System.lineSeparator() +
@@ -724,7 +724,7 @@ public void testForComplexPathInBothClientAndService() {
         OpenApiCmd cmd = new OpenApiCmd(standardOut, tmpDir, false);
         new CommandLine(cmd).parseArgs(args);
         cmd.execute();
-        Assert.assertTrue(outputStream.toString().contains("remote function(s) will be generated for client" +
+        Assert.assertTrue(outputStream.toString().contains("WARNING: remote function(s) will be generated for client" +
                 " and the service generation can not be proceed due to the given openapi definition contains" +
                 " following complex path(s):" + System.lineSeparator() +
                 "/v4/spreadsheets/{spreadsheetId}/sheets/{sheetId}:copyTo" + System.lineSeparator() +