Async to Sync Revert for Java Generation

CHANGELOG.md

@@ -15,6 +15,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Fixed float to string conversion with invariant culture.
 - Fixed a bug where vendor mime types would be ignored. [#3666](https://github.com/microsoft/kiota/issues/3666)
 - Fixed a bug where additional parameter would be stripped from accept header. [#3667](https://github.com/microsoft/kiota/issues/3667), [#3667](https://github.com/microsoft/kiota/issues/3668)
+- Generation for Java no longer utilizes Async api components. [Kiota-Java #175](https://github.com/microsoft/kiota-java/issues/175)
 
 ## [1.8.1] - 2023-11-02
 

it/java/basic/pom.xml

@@ -15,7 +15,7 @@
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
 
-        <kiota-java.version>0.7.8</kiota-java.version>
+        <kiota-java.version>0.9.0</kiota-java.version>
     </properties>
 
     <dependencies>

it/java/pom.xml

@@ -15,7 +15,7 @@
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
 
-        <kiota-java.version>0.7.8</kiota-java.version>
+        <kiota-java.version>0.9.0</kiota-java.version>
     </properties>
 
     <dependencies>

src/Kiota.Builder/Writers/Java/CodeMethodWriter.cs

@@ -509,16 +509,16 @@ private string GetSendRequestMethodName(bool isCollection, string returnType, bo
     {
         if (conventions.PrimitiveTypes.Contains(returnType))
             if (isCollection)
-                return "sendPrimitiveCollectionAsync";
+                return "sendPrimitiveCollection";
             else
-                return "sendPrimitiveAsync";
+                return "sendPrimitive";
         else if (isEnum)
             if (isCollection)
-                return "sendEnumCollectionAsync";
+                return "sendEnumCollection";
             else
-                return "sendEnumAsync";
-        else if (isCollection) return "sendCollectionAsync";
-        return "sendAsync";
+                return "sendEnum";
+        else if (isCollection) return "sendCollection";
+        return "send";
     }
     private const string RequestInfoVarName = "requestInfo";
     private const string RequestConfigVarName = "requestConfig";
@@ -688,17 +688,14 @@ private void WriteSerializationMethodCall(CodeProperty otherProp, CodeMethod met
     private static readonly BaseCodeParameterOrderComparer parameterOrderComparer = new();
     private string GetFinalReturnType(CodeMethod code, string returnType)
     {
-        var voidType = code.IsAsync ? "Void" : "void";
         if (code.ReturnType is CodeType { TypeDefinition: CodeEnum { Flags: true }, IsCollection: false })
             returnType = $"EnumSet<{returnType}>";
-        var returnTypeAsyncPrefix = code.IsAsync ? "java.util.concurrent.CompletableFuture<" : string.Empty;
-        var returnTypeAsyncSuffix = code.IsAsync ? ">" : string.Empty;
-        var reType = returnType.Equals("void", StringComparison.OrdinalIgnoreCase) ? voidType : returnType;
+        var reType = returnType.Equals("void", StringComparison.OrdinalIgnoreCase) ? "void" : returnType;
         var collectionCorrectedReturnType = code.ReturnType.IsArray && code.IsOfKind(CodeMethodKind.RequestExecutor) ?
                                             $"Iterable<{returnType.StripArraySuffix()}>" :
                                             reType;
         var isConstructor = code.IsOfKind(CodeMethodKind.Constructor, CodeMethodKind.ClientConstructor, CodeMethodKind.RawUrlConstructor);
-        var finalReturnType = isConstructor ? string.Empty : $"{returnTypeAsyncPrefix}{collectionCorrectedReturnType}{returnTypeAsyncSuffix}";
+        var finalReturnType = isConstructor ? string.Empty : $"{collectionCorrectedReturnType}";
         return finalReturnType.Trim();
     }
     private void WriteMethodPrototype(CodeMethod code, LanguageWriter writer, string returnType)
@@ -729,20 +726,16 @@ private void WriteMethodDocumentation(CodeMethod code, LanguageWriter writer, st
     {
         var returnVoid = baseReturnType.Equals("void", StringComparison.OrdinalIgnoreCase);
         // Void returns, this includes constructors, should not have a return statement in the JavaDocs.
-        var returnRemark = returnVoid ? string.Empty : (code.IsAsync switch
-        {
-            true => $"@return a CompletableFuture of {baseReturnType}",
-            false => $"@return a {finalReturnType}",
-        });
+        var returnRemark = returnVoid ? string.Empty : $"@return a {finalReturnType}";
         conventions.WriteLongDescription(code,
                                         writer,
                                         code.Parameters
                                             .Where(static x => x.Documentation.DescriptionAvailable)
                                             .OrderBy(static x => x.Name, StringComparer.OrdinalIgnoreCase)
                                             .Select(x => $"@param {x.Name} {JavaConventionService.RemoveInvalidDescriptionCharacters(x.Documentation.Description)}")
                                             .Union(new[] { returnRemark }));
-        if (!returnVoid || code.IsAsync) //Nullable/Nonnull annotations for returns are a part of Method Documentation
-            writer.WriteLine(code.ReturnType.IsNullable && !code.IsAsync ? "@jakarta.annotation.Nullable" : "@jakarta.annotation.Nonnull");
+        if (!returnVoid) //Nullable/Nonnull annotations for returns are a part of Method Documentation
+            writer.WriteLine(code.ReturnType.IsNullable ? "@jakarta.annotation.Nullable" : "@jakarta.annotation.Nonnull");
     }
     private string GetDeserializationMethodName(CodeTypeBase propType, CodeMethod method)
     {

tests/Kiota.Builder.Tests/Writers/Java/CodeMethodWriterTests.cs

@@ -604,7 +604,8 @@ public void WritesNullableVoidTypeForExecutor()
         };
         writer.Write(method);
         var result = tw.ToString();
-        Assert.Contains("CompletableFuture<Void>", result);
+        Assert.Contains("void", result);
+        Assert.DoesNotContain("@jakarta.annotation", result);
         AssertExtensions.CurlyBracesAreClosed(result);
     }
     [Fact]
@@ -652,7 +653,7 @@ public void WritesRequestExecutorBody()
         Assert.Contains("put(\"4XX\", Error4XX::createFromDiscriminatorValue);", result);
         Assert.Contains("put(\"5XX\", Error5XX::createFromDiscriminatorValue);", result);
         Assert.Contains("put(\"401\", Error401::createFromDiscriminatorValue);", result);
-        Assert.Contains("sendAsync", result);
+        Assert.Contains("send", result);
         AssertExtensions.CurlyBracesAreClosed(result);
     }
     [Fact]
@@ -1174,7 +1175,7 @@ public void WritesRequestExecutorBodyForCollections()
         AddRequestBodyParameters();
         writer.Write(method);
         var result = tw.ToString();
-        Assert.Contains("sendCollectionAsync", result);
+        Assert.Contains("sendCollection", result);
         AssertExtensions.CurlyBracesAreClosed(result);
     }
     [Fact]
@@ -1507,35 +1508,6 @@ public void WritesSerializerBody()
         AssertExtensions.CurlyBracesAreClosed(result);
     }
     [Fact]
-    public void WritesMethodAsyncDescription()
-    {
-        setup();
-        method.Documentation.Description = MethodDescription;
-        var parameter = new CodeParameter
-        {
-            Documentation = new()
-            {
-                Description = ParamDescription,
-            },
-            Name = ParamName,
-            Type = new CodeType
-            {
-                Name = "String"
-            }
-        };
-        method.AddParameter(parameter);
-        writer.Write(method);
-        var result = tw.ToString();
-        Assert.Contains("/**", result);
-        Assert.Contains(MethodDescription, result);
-        Assert.Contains("@param ", result);
-        Assert.Contains(ParamName, result);
-        Assert.Contains(ParamDescription, result);
-        Assert.Contains("@return a CompletableFuture of", result);
-        Assert.Contains("*/", result);
-        AssertExtensions.CurlyBracesAreClosed(result);
-    }
-    [Fact]
     public void WritesMethodSyncDescription()
     {
         setup();
@@ -1604,24 +1576,13 @@ public void ThrowsIfParentIsNotClass()
         method.Parent = CodeNamespace.InitRootNamespace();
         Assert.Throws<InvalidOperationException>(() => writer.Write(method));
     }
-    private const string TaskPrefix = "CompletableFuture<";
     [Fact]
     public void WritesReturnType()
     {
         setup();
         writer.Write(method);
         var result = tw.ToString();
-        Assert.Contains($"{TaskPrefix}{ReturnTypeName}> {MethodName}", result);// async default
-        AssertExtensions.CurlyBracesAreClosed(result);
-    }
-    [Fact]
-    public void DoesNotAddAsyncInformationOnSyncMethods()
-    {
-        setup();
-        method.IsAsync = false;
-        writer.Write(method);
-        var result = tw.ToString();
-        Assert.DoesNotContain(TaskPrefix, result);
+        Assert.Contains($"{ReturnTypeName} {MethodName}", result);// async default
         AssertExtensions.CurlyBracesAreClosed(result);
     }
     [Fact]