- adds missing defensive programming for nonnull annotations

Author: baywet

src/main/java/com/microsoft/graph/concurrency/ChunkedUploadProvider.java

@@ -30,6 +30,7 @@
 import java.io.InputStream;
 import java.security.InvalidParameterException;
 import java.util.List;
+import java.util.Objects;
 import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.ExecutionException;
 
@@ -103,25 +104,15 @@ public ChunkedUploadProvider(@Nonnull final IUploadSession uploadSession,
                                  @Nonnull final InputStream inputStream,
                                  final long streamSize,
                                  @Nonnull final Class<UploadType> uploadTypeClass) {
-        if (uploadSession == null) {
-            throw new InvalidParameterException("Upload session is null.");
-        }
-
-        if (client == null) {
-            throw new InvalidParameterException("OneDrive client is null.");
-        }
-
-        if (inputStream == null) {
-            throw new InvalidParameterException("Input stream is null.");
-        }
+        Objects.requireNonNull(uploadSession, "Upload session is null.");
 
         if (streamSize <= 0) {
             throw new InvalidParameterException("Stream size should larger than 0.");
         }
 
-        this.client = client;
+        this.client = Objects.requireNonNull(client, "Graph client is null.");
         this.readSoFar = 0;
-        this.inputStream = inputStream;
+        this.inputStream = Objects.requireNonNull(inputStream, "Input stream is null.");
         this.streamSize = streamSize;
         this.uploadUrl = uploadSession.getUploadUrl();
         this.responseHandler = new ChunkedUploadResponseHandler<UploadType>(uploadTypeClass, uploadSession.getClass());

src/main/java/com/microsoft/graph/concurrency/ChunkedUploadRequest.java

@@ -6,6 +6,7 @@
 
 import java.util.List;
 import java.util.Locale;
+import java.util.Objects;
 
 import javax.annotation.Nullable;
 import javax.annotation.Nonnull;
@@ -61,6 +62,9 @@ protected ChunkedUploadRequest(@Nonnull final String requestUrl,
                                 final int chunkSize,
                                 final long beginIndex,
                                 final long totalLength) {
+        Objects.requireNonNull(requestUrl, "parameter requestUrl cannot be null");
+        Objects.requireNonNull(client, "parameter client cannot be null");
+        Objects.requireNonNull(chunk, "parameter chunk cannot be null");
         this.data = new byte[chunkSize];
         System.arraycopy(chunk, 0, this.data, 0, chunkSize);
         this.baseRequest = new BaseRequest<ChunkedUploadResult<UploadType>>(requestUrl, client, options, (Class<? extends ChunkedUploadResult<UploadType>>)(new ChunkedUploadResult<>((UploadType)null)).getClass()) {
@@ -84,6 +88,7 @@ protected ChunkedUploadRequest(@Nonnull final String requestUrl,
     @Nonnull
     public ChunkedUploadResult<UploadType> upload(
             @Nonnull final ChunkedUploadResponseHandler<UploadType> responseHandler) {
+        Objects.requireNonNull(responseHandler, "parameter responseHandler cannot be null");
         ChunkedUploadResult<UploadType> result = null;
 
         try {

src/main/java/com/microsoft/graph/concurrency/ChunkedUploadResponseHandler.java

@@ -35,6 +35,7 @@
 import java.io.ByteArrayInputStream;
 import java.io.IOException;
 import java.io.InputStream;
+import java.util.Objects;
 
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
@@ -65,8 +66,8 @@ public class ChunkedUploadResponseHandler<UploadType>
      * @param uploadSessionType the type of the upload session
 	 */
 	protected ChunkedUploadResponseHandler(@Nonnull final Class<UploadType> uploadType, @Nonnull final Class<? extends IUploadSession>  uploadSessionType) {
-        this.deserializeTypeClass = uploadType;
-        this.uploadSessionClass = uploadSessionType;
+        this.deserializeTypeClass = Objects.requireNonNull(uploadType, "parameter uploadType cannot be null");
+        this.uploadSessionClass = Objects.requireNonNull(uploadSessionType, "parameter uploadSessionType cannot be null");
 	}
 
 	/**
@@ -96,6 +97,10 @@ public ChunkedUploadResult<UploadType> generateResult(
 			@Nonnull final Response response,
 			@Nonnull final ISerializer serializer,
 			@Nonnull final ILogger logger) throws Exception {
+        Objects.requireNonNull(request, "parameter request cannot be null");
+        Objects.requireNonNull(response, "parameter response cannot be null");
+        Objects.requireNonNull(serializer, "parameter serializer cannot be null");
+        Objects.requireNonNull(logger, "parameter logger cannot be null");
 		if (response.code() >= HttpResponseCode.HTTP_CLIENT_ERROR) {
 			logger.logDebug("Receiving error during upload, see detail on result error");
 

src/main/java/com/microsoft/graph/concurrency/ChunkedUploadResult.java

@@ -8,6 +8,9 @@
 import com.microsoft.graph.http.GraphServiceException;
 
 import javax.annotation.Nullable;
+
+import java.util.Objects;
+
 import javax.annotation.Nonnull;
 
 /**
@@ -68,7 +71,10 @@ protected ChunkedUploadResult(@Nullable final ClientException error) {
      * @param exception The exception received from server.
      */
     protected ChunkedUploadResult(@Nonnull final GraphServiceException exception) {
-        this(new ClientException(exception.getMessage(/* verbose */ true), exception));
+        this(new ClientException(Objects
+                                .requireNonNull(exception, "parameter exception cannot be null")
+                                .getMessage(/* verbose */ true),
+                                exception));
     }
 
     /**

src/main/java/com/microsoft/graph/content/BatchRequest.java

@@ -43,7 +43,7 @@ public class BatchRequest extends BaseRequest<BatchResponseContent> {
      * @param client the client to use to execute the request
      * @param options the options to apply to the request
      */
-    public BatchRequest(@Nonnull final String requestUrl, @Nonnull final IBaseClient client, @Nonnull final List<? extends Option> options) {
+    public BatchRequest(@Nonnull final String requestUrl, @Nonnull final IBaseClient<?> client, @Nonnull final List<? extends Option> options) {
         super(requestUrl, client, options, BatchResponseContent.class);
     }
 
@@ -55,7 +55,7 @@ public BatchRequest(@Nonnull final String requestUrl, @Nonnull final IBaseClient
      * @throws ClientException an exception occurs if there was an error while the request was sent
      */
     @Nullable
-    public BatchResponseContent post(@Nonnull final BatchRequestContent content) throws ClientException {
+    public BatchResponseContent post(@Nullable final BatchRequestContent content) throws ClientException {
         this.setHttpMethod(HttpMethod.POST);
         final BatchResponseContent response = this.getClient().getHttpProvider().send(this, BatchResponseContent.class, content);
         setSerializerOnSteps(response);
@@ -69,7 +69,7 @@ public BatchResponseContent post(@Nonnull final BatchRequestContent content) thr
      * @throws ClientException an exception occurs if there was an error while the request was sent
      */
     @Nullable
-    public java.util.concurrent.CompletableFuture<BatchResponseContent> postAsync(@Nonnull final BatchRequestContent content) throws ClientException {
+    public java.util.concurrent.CompletableFuture<BatchResponseContent> postAsync(@Nullable final BatchRequestContent content) throws ClientException {
         this.setHttpMethod(HttpMethod.POST);
         return this.getClient().getHttpProvider().sendAsync(this, BatchResponseContent.class, content).thenApply(response -> {
             setSerializerOnSteps(response);

src/main/java/com/microsoft/graph/core/BaseClient.java

@@ -75,7 +75,7 @@ public String getServiceRoot() {
 
     @Override
     public void setServiceRoot(@Nonnull final String value) {
-        endpoint = value;
+        endpoint = Objects.requireNonNull(value, "value parameter cannot be null");
     }
 
 	/**
@@ -89,7 +89,9 @@ public void setServiceRoot(@Nonnull final String value) {
 	 */
 	@Nonnull
 	public <T> CustomRequestBuilder<T> customRequest(@Nonnull final String url, @Nonnull final Class<T> responseType) {
-		return new CustomRequestBuilder<>(getServiceRoot() + url, this, null, responseType);
+		Objects.requireNonNull(url, "url parameter cannot be null");
+        Objects.requireNonNull(responseType, "responseType parameter cannot be null");
+        return new CustomRequestBuilder<>(getServiceRoot() + url, this, null, responseType);
 	}
 
 	/**
@@ -101,8 +103,7 @@ public <T> CustomRequestBuilder<T> customRequest(@Nonnull final String url, @Non
 	 */
 	@Nonnull
 	public CustomRequestBuilder<JsonElement> customRequest(@Nonnull final String url) {
-		return new CustomRequestBuilder<>(getServiceRoot() + url, this, null,
-				JsonElement.class);
+		return this.customRequest(url, JsonElement.class);
     }
 
     /**
@@ -197,7 +198,7 @@ private IHttpProvider<nativeRequestType> getHttpProvider() {
 		 */
 		@Nonnull
 		public Builder<httpClientType, nativeRequestType> serializer(@Nonnull final ISerializer serializer) {
-			checkNotNull(serializer, "serializer");
+			Objects.requireNonNull(serializer, "parameter serializer cannot be null");
 			this.serializer = serializer;
 			return this;
 		}
@@ -211,7 +212,7 @@ public Builder<httpClientType, nativeRequestType> serializer(@Nonnull final ISer
 		 */
 		@Nonnull
 		public Builder<httpClientType, nativeRequestType> httpProvider(@Nonnull final IHttpProvider<nativeRequestType> httpProvider) {
-			checkNotNull(httpProvider, "httpProvider");
+			Objects.requireNonNull(httpProvider, "parameter httpProvider cannot be null");
 			this.httpProvider = httpProvider;
 			return this;
 		}
@@ -225,7 +226,7 @@ public Builder<httpClientType, nativeRequestType> httpProvider(@Nonnull final IH
 		 */
 		@Nonnull
 		public Builder<httpClientType, nativeRequestType> logger(@Nonnull final ILogger logger) {
-			checkNotNull(logger, "logger");
+			Objects.requireNonNull(logger, "parameter logger cannot be null");
 			this.logger = logger;
 			return this;
 		}
@@ -239,7 +240,7 @@ public Builder<httpClientType, nativeRequestType> logger(@Nonnull final ILogger
 		 */
 		@Nonnull
 		public Builder<httpClientType, nativeRequestType> httpClient(@Nonnull final httpClientType client) {
-			checkNotNull(client, "client");
+			Objects.requireNonNull(client, "parameter client cannot be null");
 			this.httpClient = client;
 			return this;
 		}
@@ -252,7 +253,7 @@ public Builder<httpClientType, nativeRequestType> httpClient(@Nonnull final http
 		 */
 		@Nonnull
 		public Builder<httpClientType, nativeRequestType> authenticationProvider(@Nonnull final IAuthenticationProvider auth) {
-			checkNotNull(auth, "auth");
+			Objects.requireNonNull(auth, "parameter auth cannot be null");
 			this.auth = auth;
 			return this;
 		}
@@ -288,18 +289,6 @@ public IBaseClient<nativeRequestType> buildClient() throws ClientException {
 		}
 	}
 
-	/**
-	 * Checks whether the provided object is null or not and throws an exception if it is
-	 *
-	 * @param o object to check
-	 * @param name name to use in the exception message
-	 */
-	protected static void checkNotNull(@Nullable final Object o, @Nonnull final String name) {
-		if (o==null) {
-			throw new NullPointerException(name + " cannot be null");
-		}
-    }
-
     /**
      * The HTTP provider instance
      */
@@ -353,7 +342,7 @@ public ISerializer getSerializer() {
      * @param logger The logger
      */
     protected void setLogger(@Nonnull final ILogger logger) {
-        checkNotNull(logger, "logger");
+        Objects.requireNonNull(logger, "parameter logger cannot be null");
         this.logger = logger;
     }
 
@@ -363,7 +352,7 @@ protected void setLogger(@Nonnull final ILogger logger) {
      * @param httpProvider The HTTP provider
      */
     protected void setHttpProvider(@Nonnull final IHttpProvider<nativeRequestType> httpProvider) {
-        checkNotNull(httpProvider, "httpProvider");
+        Objects.requireNonNull(httpProvider, "parameter httpProvider cannot be null");
         this.httpProvider = httpProvider;
     }
 
@@ -373,7 +362,7 @@ protected void setHttpProvider(@Nonnull final IHttpProvider<nativeRequestType> h
      * @param serializer The serializer
      */
     public void setSerializer(@Nonnull final ISerializer serializer) {
-        checkNotNull(serializer, "serializer");
+        Objects.requireNonNull(serializer, "parameter serializer cannot be null");
         this.serializer = serializer;
     }
 

src/main/java/com/microsoft/graph/core/CustomRequestBuilder.java

@@ -1,6 +1,7 @@
 package com.microsoft.graph.core;
 
 import java.util.List;
+import java.util.Objects;
 
 import javax.annotation.Nullable;
 import javax.annotation.Nonnull;
@@ -28,7 +29,7 @@ public class CustomRequestBuilder<T> extends BaseRequestBuilder<T> {
 	 */
 	public CustomRequestBuilder(@Nonnull final String requestUrl, @Nonnull final IBaseClient<?> client, @Nullable final List<? extends Option> requestOptions, @Nonnull final Class<T> responseType) {
 		super(requestUrl, client, requestOptions);
-		this.responseType = responseType;
+		this.responseType = Objects.requireNonNull(responseType, "parameter responseType cannot be null");
 	}
 
 	/**

src/main/java/com/microsoft/graph/core/DateOnly.java

@@ -3,6 +3,7 @@
 
 import java.text.ParseException;
 import java.util.Locale;
+import java.util.Objects;
 
 import javax.annotation.Nullable;
 import javax.annotation.Nonnull;
@@ -36,6 +37,7 @@ public class DateOnly {
      */
     @Nullable
     public static DateOnly parse(@Nonnull final String dateStr) throws ParseException {
+        Objects.requireNonNull(dateStr, "parameter dateStr cannot be null");
         // break the date up into its constituent parts
         final String[] dateInfo = dateStr.split("-");
 

src/main/java/com/microsoft/graph/core/Multipart.java

@@ -7,6 +7,7 @@
 import java.math.BigInteger;
 import java.security.SecureRandom;
 import java.util.Map;
+import java.util.Objects;
 
 import javax.annotation.Nullable;
 import javax.annotation.Nonnull;
@@ -51,7 +52,7 @@ public String getBoundary() {
 	 * @param boundary The multipart boundary
 	 */
 	public void setBoundary(@Nonnull final String boundary) {
-		this.boundary = boundary;
+		this.boundary = Objects.requireNonNull(boundary, "parameter boundary cannot be null");
 	}
 
 	/**
@@ -68,7 +69,7 @@ public String getContentType() {
 	 * @param contentType The multipart Content-Type
 	 */
 	public void setContentType(@Nonnull final String contentType) {
-		this.contentType = contentType;
+		this.contentType = Objects.requireNonNull(contentType, "parameter contentType cannot be null");
 	}
 
 	/**

src/main/java/com/microsoft/graph/core/TimeOfDay.java

@@ -7,6 +7,7 @@
 import java.math.BigDecimal;
 import java.text.ParseException;
 import java.util.Locale;
+import java.util.Objects;
 
 /**
  * The time of day
@@ -37,6 +38,7 @@ public class TimeOfDay {
      */
     @Nullable
     public static TimeOfDay parse(@Nonnull final String timeStr) throws ParseException {
+        Objects.requireNonNull(timeStr, "parameter timeStr cannot be null");
         // break the date up into its constituent parts
         final String[] timeInfo = timeStr.split(":");