java 8 code migration style

Valentin Popov · Valentin Popov · commit a20229844f3e · 2021-09-09T11:26:34.000+03:00
diff --git a/src/main/java/com/microsoft/graph/authentication/TokenCredentialAuthProvider.java b/src/main/java/com/microsoft/graph/authentication/TokenCredentialAuthProvider.java
@@ -1,17 +1,16 @@
 package com.microsoft.graph.authentication;
 
+import com.azure.core.credential.AccessToken;
 import com.azure.core.credential.TokenCredential;
 import com.azure.core.credential.TokenRequestContext;
 
+import javax.annotation.Nonnull;
 import java.net.URL;
-import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;
 import java.util.Objects;
 import java.util.concurrent.CompletableFuture;
 
-import javax.annotation.Nonnull;
-
 /**
  * An implementation of the Authentication Provider with Azure-identity
  */
@@ -57,7 +56,7 @@ public CompletableFuture<String> getAuthorizationTokenAsync(@Nonnull final URL r
             return this.tokenCredential
                         .getToken(this.context)
                         .toFuture()
-                        .thenApply(resp -> resp.getToken());
+                        .thenApply(AccessToken::getToken);
         else
             return CompletableFuture.completedFuture((String)null);
     }
diff --git a/src/main/java/com/microsoft/graph/content/BatchRequestContent.java b/src/main/java/com/microsoft/graph/content/BatchRequestContent.java
@@ -110,7 +110,7 @@ public <T> String addBatchRequestStep(@Nonnull final IHttpRequest request, @Nonn
             url = protocolAndHostReplacementMatcher.replaceAll("");
             body = serializableBody;
             method = httpMethod.toString().toUpperCase(Locale.getDefault());
-            dependsOn = dependsOnRequestsIds != null && dependsOnRequestsIds.length > 0 ? new HashSet<String>(Arrays.asList(dependsOnRequestsIds)) : null;
+            dependsOn = dependsOnRequestsIds != null && dependsOnRequestsIds.length > 0 ? new HashSet<>(Arrays.asList(dependsOnRequestsIds)) : null;
             id = getNextRequestId();
         }};
 
@@ -141,7 +141,7 @@ public void removeBatchRequestStepWithId(@Nonnull final String ...stepIds) {
             requests.removeIf(x -> stepId.equals(x.id));
             for(final BatchRequestStep<?> step : requests) {
                 if(step.dependsOn != null) {
-                    step.dependsOn.removeIf(x -> stepId.equals(x));
+                    step.dependsOn.removeIf(stepId::equals);
                     if(step.dependsOn.isEmpty())
                         step.dependsOn = null; // so we don't send dependsOn: [] over the wire
                 }
diff --git a/src/main/java/com/microsoft/graph/core/CustomRequestBuilder.java b/src/main/java/com/microsoft/graph/core/CustomRequestBuilder.java
@@ -51,6 +51,6 @@ public CustomRequest<T> buildRequest(@Nullable final com.microsoft.graph.options
 	 */
 	@Nonnull
 	public CustomRequest<T> buildRequest(@Nullable final List<? extends Option> requestOptions) {
-		return new CustomRequest<T>(getRequestUrl(), getClient(), requestOptions, responseType);
+		return new CustomRequest<>(getRequestUrl(), getClient(), requestOptions, responseType);
 	}
 }
diff --git a/src/main/java/com/microsoft/graph/core/IBaseClient.java b/src/main/java/com/microsoft/graph/core/IBaseClient.java
@@ -100,7 +100,7 @@ public interface IBaseClient<nativeRequestType> {
      * @return a request builder to execute a batch.
      */
     @Nonnull
-    public BatchRequestBuilder batch();
+    BatchRequestBuilder batch();
 
     /**
      * Gets the service SDK version if the service SDK is in use, null otherwise
diff --git a/src/main/java/com/microsoft/graph/core/Multipart.java b/src/main/java/com/microsoft/graph/core/Multipart.java
@@ -35,7 +35,7 @@ public class Multipart {
 	 */
 	public Multipart() {
 		out = new ByteArrayOutputStream();
-		boundary = "part_" + new BigInteger(130, new SecureRandom()).toString();
+		boundary = "part_" + new BigInteger(130, new SecureRandom());
 	}
 
 	/**
diff --git a/src/main/java/com/microsoft/graph/http/BaseActionCollectionRequest.java b/src/main/java/com/microsoft/graph/http/BaseActionCollectionRequest.java
@@ -81,7 +81,7 @@ public java.util.concurrent.CompletableFuture<T3> postAsync() {
                             .sendAsync(this,
                                 responseCollectionClass,
                                 bodyToSend)
-                            .thenApply(r -> buildFromResponse(r));
+                            .thenApply(this::buildFromResponse);
     }
     /**
      * Invokes the method and returns the resulting collection of objects
diff --git a/src/main/java/com/microsoft/graph/http/BaseCollectionPage.java b/src/main/java/com/microsoft/graph/http/BaseCollectionPage.java
@@ -72,7 +72,7 @@ public BaseCollectionPage(@Nonnull final ICollectionResponse<T> response, @Nulla
     public BaseCollectionPage(@Nonnull final List<T> pageContents, @Nullable final T2 nextRequestBuilder) {
         // CollectionPages are never directly modifiable, either 'update'/'delete' the specific child or 'add' the new
         // object to the 'children' of the collection.
-        this.pageContents = Collections.unmodifiableList(pageContents == null ? new ArrayList<T>() : pageContents);
+        this.pageContents = Collections.unmodifiableList(pageContents == null ? new ArrayList<>() : pageContents);
         requestBuilder = nextRequestBuilder;
     }
 
diff --git a/src/main/java/com/microsoft/graph/http/BaseEntityCollectionRequest.java b/src/main/java/com/microsoft/graph/http/BaseEntityCollectionRequest.java
@@ -66,7 +66,7 @@ public BaseEntityCollectionRequest(@Nonnull final String requestUrl,
      */
     @Nonnull
     public java.util.concurrent.CompletableFuture<T3> getAsync() {
-        return sendAsync().thenApply(r -> buildFromResponse(r));
+        return sendAsync().thenApply(this::buildFromResponse);
     }
     /**
      * Gets the collection of items
diff --git a/src/main/java/com/microsoft/graph/http/BaseFunctionCollectionRequest.java b/src/main/java/com/microsoft/graph/http/BaseFunctionCollectionRequest.java
@@ -66,7 +66,7 @@ public BaseFunctionCollectionRequest(@Nonnull final String requestUrl,
      */
     @Nonnull
     public java.util.concurrent.CompletableFuture<T3> getAsync() {
-        return sendAsync().thenApply(r -> buildFromResponse(r));
+        return sendAsync().thenApply(this::buildFromResponse);
     }
 
     /**
diff --git a/src/main/java/com/microsoft/graph/http/BaseRequest.java b/src/main/java/com/microsoft/graph/http/BaseRequest.java
@@ -193,9 +193,9 @@ public URL getRequestUrl() {
             return new URL(uriBuilder.build().toString());
         } catch (final MalformedURLException e) {
         	if (this instanceof CustomRequest) {
-        		this.getClient().getLogger().logError("Invalid custom URL: " + uriBuilder.toString(), e);
+        		this.getClient().getLogger().logError("Invalid custom URL: " + uriBuilder, e);
         	} else {
-        		throw new ClientException("Invalid URL: " + uriBuilder.toString(), e);
+        		throw new ClientException("Invalid URL: " + uriBuilder, e);
         	}
         }
 		return null;
diff --git a/src/main/java/com/microsoft/graph/http/CustomRequest.java b/src/main/java/com/microsoft/graph/http/CustomRequest.java
@@ -64,7 +64,7 @@ public CustomRequest(@Nonnull final String requestUrl, @Nonnull final IBaseClien
      */
     @Nonnull
     public static CustomRequest<JsonObject> create(@Nonnull final String requestUrl, @Nonnull final IBaseClient<?> client, @Nullable final java.util.List<? extends Option> requestOptions) {
-        return new CustomRequest<JsonObject>(requestUrl, client, requestOptions, JsonObject.class);
+        return new CustomRequest<>(requestUrl, client, requestOptions, JsonObject.class);
     }
 
     /**
diff --git a/src/main/java/com/microsoft/graph/http/GraphInnerError.java b/src/main/java/com/microsoft/graph/http/GraphInnerError.java
@@ -1,16 +1,16 @@
 // ------------------------------------------------------------------------------
 // Copyright (c) 2017 Microsoft Corporation
-// 
+//
 // Permission is hereby granted, free of charge, to any person obtaining a copy
 // of this software and associated documentation files (the "Software"), to deal
 // in the Software without restriction, including without limitation the rights
 // to use, copy, modify, merge, publish, distribute, sub-license, and/or sell
 // copies of the Software, and to permit persons to whom the Software is
 // furnished to do so, subject to the following conditions:
-// 
+//
 // The above copyright notice and this permission notice shall be included in
 // all copies or substantial portions of the Software.
-// 
+//
 // THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 // IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 // FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
@@ -25,7 +25,6 @@
 import com.google.gson.annotations.SerializedName;
 
 import javax.annotation.Nullable;
-import javax.annotation.Nonnull;
 
 /**
  * Represents an inner error returned by the service
diff --git a/src/main/java/com/microsoft/graph/httpcore/ChaosHttpHandler.java b/src/main/java/com/microsoft/graph/httpcore/ChaosHttpHandler.java
@@ -3,7 +3,6 @@
 import java.io.IOException;
 import java.util.concurrent.ThreadLocalRandom;
 
-import javax.annotation.Nullable;
 import javax.annotation.Nonnull;
 
 import com.microsoft.graph.httpcore.middlewareoption.MiddlewareType;
diff --git a/src/main/java/com/microsoft/graph/httpcore/middlewareoption/RedirectOptions.java b/src/main/java/com/microsoft/graph/httpcore/middlewareoption/RedirectOptions.java
@@ -23,12 +23,7 @@ public class RedirectOptions implements IMiddlewareControl{
     /**
      * Default redirect evaluation, always follow redirect information.
      */
-    public static final IShouldRedirect DEFAULT_SHOULD_REDIRECT = new IShouldRedirect() {
-        @Override
-        public boolean shouldRedirect(Response response) {
-            return true;
-        }
-    };
+    public static final IShouldRedirect DEFAULT_SHOULD_REDIRECT = response -> true;
 
     /**
      * Create default instance of redirect options, with default values of max redirects and should redirect
diff --git a/src/main/java/com/microsoft/graph/httpcore/middlewareoption/RetryOptions.java b/src/main/java/com/microsoft/graph/httpcore/middlewareoption/RetryOptions.java
@@ -14,12 +14,7 @@ public class RetryOptions implements IMiddlewareControl {
     /**
      * Default retry evaluation, always retry.
      */
-    public static final IShouldRetry DEFAULT_SHOULD_RETRY = new IShouldRetry() {
-        @Override
-        public boolean shouldRetry(long delay, int executionCount, Request request, Response response) {
-            return true;
-        }
-    };
+    public static final IShouldRetry DEFAULT_SHOULD_RETRY = (delay, executionCount, request, response) -> true;
 
     private int mMaxRetries;
     /**
diff --git a/src/main/java/com/microsoft/graph/serializer/DefaultSerializer.java b/src/main/java/com/microsoft/graph/serializer/DefaultSerializer.java
@@ -32,6 +32,7 @@
 import java.io.InputStream;
 import java.io.InputStreamReader;
 import java.lang.reflect.Field;
+import java.nio.charset.StandardCharsets;
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.List;
@@ -98,7 +99,7 @@ public <T> T deserializeObject(@Nonnull final String inputString, @Nonnull final
 	public <T> T deserializeObject(@Nonnull final InputStream inputStream, @Nonnull final Class<T> clazz, @Nullable final Map<String, List<String>> responseHeaders) {
 		Objects.requireNonNull(inputStream, "parameter inputStream cannot be null");
         T result = null;
-        try (final InputStreamReader streamReader =  new InputStreamReader(inputStream, "UTF-8")) {
+        try (final InputStreamReader streamReader =  new InputStreamReader(inputStream, StandardCharsets.UTF_8)) {
 			final JsonElement rawElement = gson.fromJson(streamReader, JsonElement.class);
 			result = deserializeObject(rawElement, clazz, responseHeaders);
         } catch (IOException ex) {
@@ -155,24 +156,22 @@ private void setChildAdditionalData(final IJsonBackedObject serializedObject, fi
 							// If the object is a HashMap, iterate through its children
 							@SuppressWarnings("unchecked")
 							final HashMap<String, Object> serializableChildren = (HashMap<String, Object>) fieldObject;
-							final Iterator<Entry<String, Object>> it = serializableChildren.entrySet().iterator();
 
-							while (it.hasNext()) {
-								final Map.Entry<String, Object> pair = (Map.Entry<String, Object>)it.next();
-								final Object child = pair.getValue();
+                            for (Entry<String, Object> pair : serializableChildren.entrySet()) {
+                                final Object child = pair.getValue();
 
-								// If the item is a valid Graph object, set its additional data
-								if (child instanceof IJsonBackedObject) {
-									final AdditionalDataManager childAdditionalDataManager = ((IJsonBackedObject) child).additionalDataManager();
-									final JsonElement fieldElement = rawJson.get(field.getName());
-									if(fieldElement != null && fieldElement.isJsonObject()
-											&& fieldElement.getAsJsonObject().get(pair.getKey()) != null
-											&& fieldElement.getAsJsonObject().get(pair.getKey()).isJsonObject()) {
-										childAdditionalDataManager.setAdditionalData(fieldElement.getAsJsonObject().get(pair.getKey()).getAsJsonObject());
-										setChildAdditionalData((IJsonBackedObject) child,fieldElement.getAsJsonObject().get(pair.getKey()).getAsJsonObject());
-									}
-								}
-							}
+                                // If the item is a valid Graph object, set its additional data
+                                if (child instanceof IJsonBackedObject) {
+                                    final AdditionalDataManager childAdditionalDataManager = ((IJsonBackedObject) child).additionalDataManager();
+                                    final JsonElement fieldElement = rawJson.get(field.getName());
+                                    if (fieldElement != null && fieldElement.isJsonObject()
+                                        && fieldElement.getAsJsonObject().get(pair.getKey()) != null
+                                        && fieldElement.getAsJsonObject().get(pair.getKey()).isJsonObject()) {
+                                        childAdditionalDataManager.setAdditionalData(fieldElement.getAsJsonObject().get(pair.getKey()).getAsJsonObject());
+                                        setChildAdditionalData((IJsonBackedObject) child, fieldElement.getAsJsonObject().get(pair.getKey()).getAsJsonObject());
+                                    }
+                                }
+                            }
 						}
 						// If the object is a list of Graph objects, iterate through elements
 						else if (fieldObject instanceof List) {
diff --git a/src/main/java/com/microsoft/graph/serializer/EdmNativeTypeSerializer.java b/src/main/java/com/microsoft/graph/serializer/EdmNativeTypeSerializer.java
@@ -1,11 +1,9 @@
 package com.microsoft.graph.serializer;
 
-import java.lang.reflect.Type;
 import java.math.BigDecimal;
 import java.util.UUID;
 
 import com.google.gson.JsonElement;
-import com.google.gson.JsonParseException;
 import com.microsoft.graph.logger.ILogger;
 
 import javax.annotation.Nonnull;
diff --git a/src/main/java/com/microsoft/graph/serializer/GsonFactory.java b/src/main/java/com/microsoft/graph/serializer/GsonFactory.java
diff --git a/src/main/java/com/microsoft/graph/serializer/OffsetDateTimeSerializer.java b/src/main/java/com/microsoft/graph/serializer/OffsetDateTimeSerializer.java
diff --git a/src/main/java/com/microsoft/graph/tasks/LargeFileUploadRequest.java b/src/main/java/com/microsoft/graph/tasks/LargeFileUploadRequest.java
diff --git a/src/main/java/com/microsoft/graph/tasks/LargeFileUploadTask.java b/src/main/java/com/microsoft/graph/tasks/LargeFileUploadTask.java

Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,6 @@ public CustomRequest<T> buildRequest(@Nullable final com.microsoft.graph.options`
`51`	`51`	`*/`
`52`	`52`	`@Nonnull`
`53`	`53`	`public CustomRequest<T> buildRequest(@Nullable final List<? extends Option> requestOptions) {`
`54`		`- return new CustomRequest<T>(getRequestUrl(), getClient(), requestOptions, responseType);`
	`54`	`+ return new CustomRequest<>(getRequestUrl(), getClient(), requestOptions, responseType);`
`55`	`55`	`}`
`56`	`56`	`}`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ public class Multipart {`
`35`	`35`	`*/`
`36`	`36`	`public Multipart() {`
`37`	`37`	`out = new ByteArrayOutputStream();`
`38`		`- boundary = "part_" + new BigInteger(130, new SecureRandom()).toString();`
	`38`	`+ boundary = "part_" + new BigInteger(130, new SecureRandom());`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,7 @@ public java.util.concurrent.CompletableFuture<T3> postAsync() {`
`81`	`81`	`.sendAsync(this,`
`82`	`82`	`responseCollectionClass,`
`83`	`83`	`bodyToSend)`
`84`		`- .thenApply(r -> buildFromResponse(r));`
	`84`	`+ .thenApply(this::buildFromResponse);`
`85`	`85`	`}`
`86`	`86`	`/**`
`87`	`87`	`* Invokes the method and returns the resulting collection of objects`
Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ public BaseCollectionPage(@Nonnull final ICollectionResponse<T> response, @Nulla`
`72`	`72`	`public BaseCollectionPage(@Nonnull final List<T> pageContents, @Nullable final T2 nextRequestBuilder) {`
`73`	`73`	`// CollectionPages are never directly modifiable, either 'update'/'delete' the specific child or 'add' the new`
`74`	`74`	`// object to the 'children' of the collection.`
`75`		`- this.pageContents = Collections.unmodifiableList(pageContents == null ? new ArrayList<T>() : pageContents);`
	`75`	`+ this.pageContents = Collections.unmodifiableList(pageContents == null ? new ArrayList<>() : pageContents);`
`76`	`76`	`requestBuilder = nextRequestBuilder;`
`77`	`77`	`}`
`78`	`78`
Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ public BaseEntityCollectionRequest(@Nonnull final String requestUrl,`
`66`	`66`	`*/`
`67`	`67`	`@Nonnull`
`68`	`68`	`public java.util.concurrent.CompletableFuture<T3> getAsync() {`
`69`		`- return sendAsync().thenApply(r -> buildFromResponse(r));`
	`69`	`+ return sendAsync().thenApply(this::buildFromResponse);`
`70`	`70`	`}`
`71`	`71`	`/**`
`72`	`72`	`* Gets the collection of items`
Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ public BaseFunctionCollectionRequest(@Nonnull final String requestUrl,`
`66`	`66`	`*/`
`67`	`67`	`@Nonnull`
`68`	`68`	`public java.util.concurrent.CompletableFuture<T3> getAsync() {`
`69`		`- return sendAsync().thenApply(r -> buildFromResponse(r));`
	`69`	`+ return sendAsync().thenApply(this::buildFromResponse);`
`70`	`70`	`}`
`71`	`71`
`72`	`72`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -193,9 +193,9 @@ public URL getRequestUrl() {`
`193`	`193`	`return new URL(uriBuilder.build().toString());`
`194`	`194`	`} catch (final MalformedURLException e) {`
`195`	`195`	`if (this instanceof CustomRequest) {`
`196`		`- this.getClient().getLogger().logError("Invalid custom URL: " + uriBuilder.toString(), e);`
	`196`	`+ this.getClient().getLogger().logError("Invalid custom URL: " + uriBuilder, e);`
`197`	`197`	`} else {`
`198`		`- throw new ClientException("Invalid URL: " + uriBuilder.toString(), e);`
	`198`	`+ throw new ClientException("Invalid URL: " + uriBuilder, e);`
`199`	`199`	`}`
`200`	`200`	`}`
`201`	`201`	`return null;`