add check for null for setConsume

Author: yvonnep165

src/main/java/com/google/firebase/appcheck/VerifyAppCheckTokenOptions.java

@@ -16,6 +16,8 @@
 
 package com.google.firebase.appcheck;
 
+import static com.google.common.base.Preconditions.checkNotNull;
+
 import java.util.Optional;
 
 /**
@@ -60,11 +62,11 @@ public Builder setConsume(boolean consume) {
     /**
      * Sets whether to consume the token during verification.
      *
-     * @param consume Optional boolean value.
+     * @param consume Optional boolean value. Must not be null.
      * @return This builder.
      */
     public Builder setConsume(Optional<Boolean> consume) {
-      this.consume = consume != null ? consume : Optional.<Boolean>empty();
+      this.consume = checkNotNull(consume, "consume must not be null");
       return this;
     }
 

src/test/java/com/google/firebase/appcheck/VerifyAppCheckTokenOptionsTest.java

@@ -16,10 +16,13 @@
 
 package com.google.firebase.appcheck;
 
+import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertThrows;
 import static org.junit.Assert.assertTrue;
 
+import java.util.Optional;
 import org.junit.Test;
 
 public class VerifyAppCheckTokenOptionsTest {
@@ -48,4 +51,30 @@ public void testBuilder_SetConsumeFalse() {
     assertTrue(options.getConsume().isPresent());
     assertFalse(options.getConsume().get());
   }
+
+  @Test
+  public void testBuilder_SetConsumeOptional_Present() {
+    VerifyAppCheckTokenOptions options =
+        VerifyAppCheckTokenOptions.builder().setConsume(Optional.of(true)).build();
+    assertNotNull(options);
+    assertTrue(options.getConsume().isPresent());
+    assertTrue(options.getConsume().get());
+  }
+
+  @Test
+  public void testBuilder_SetConsumeOptional_Empty() {
+    VerifyAppCheckTokenOptions options =
+        VerifyAppCheckTokenOptions.builder().setConsume(Optional.empty()).build();
+    assertNotNull(options);
+    assertFalse(options.getConsume().isPresent());
+  }
+
+  @Test
+  public void testBuilder_SetConsumeOptionalNull_ThrowsException() {
+    NullPointerException e =
+        assertThrows(
+            NullPointerException.class,
+            () -> VerifyAppCheckTokenOptions.builder().setConsume((Optional<Boolean>) null));
+    assertEquals("consume must not be null", e.getMessage());
+  }
 }