improve tests and fix javadoc

Author: c24Felix

src/main/java/com/google/firebase/FirebaseOptions.java

@@ -214,7 +214,7 @@ public int getReadTimeout() {
    * Returns the write timeout in milliseconds, which is applied to outgoing REST calls
    * made by the SDK.
    *
-   * @return Read timeout in milliseconds. 0 indicates an infinite timeout.
+   * @return Write timeout in milliseconds. 0 indicates an infinite timeout.
    */
   public int getWriteTimeout() {
     return writeTimeout;
@@ -511,11 +511,11 @@ public Builder setReadTimeout(int readTimeout) {
     }
 
     /**
-     * Sets the read timeout for outgoing HTTP (REST) calls made by the SDK. This does not affect
+     * Sets the write timeout for outgoing HTTP (REST) calls made by the SDK. This does not affect
      * the {@link com.google.firebase.database.FirebaseDatabase} and
      * {@link com.google.firebase.cloud.FirestoreClient} APIs.
      *
-     * @param writeTimeout Read timeout in milliseconds. Must not be negative.
+     * @param writeTimeout Write timeout in milliseconds. Must not be negative.
      * @return This <code>Builder</code> instance is returned so subsequent calls can be chained.
      */
     public Builder setWriteTimeout(int writeTimeout) {

src/test/java/com/google/firebase/FirebaseOptionsTest.java

@@ -87,7 +87,7 @@ public void createOptionsWithAllValuesSet() throws IOException {
             .setThreadManager(MOCK_THREAD_MANAGER)
             .setConnectTimeout(30000)
             .setReadTimeout(60000)
-            .setWriteTimeout(60000)
+            .setWriteTimeout(90000)
             .setFirestoreOptions(firestoreOptions)
             .build();
     assertEquals(FIREBASE_DB_URL, firebaseOptions.getDatabaseUrl());
@@ -98,7 +98,7 @@ public void createOptionsWithAllValuesSet() throws IOException {
     assertSame(MOCK_THREAD_MANAGER, firebaseOptions.getThreadManager());
     assertEquals(30000, firebaseOptions.getConnectTimeout());
     assertEquals(60000, firebaseOptions.getReadTimeout());
-    assertEquals(60000, firebaseOptions.getWriteTimeout());
+    assertEquals(90000, firebaseOptions.getWriteTimeout());
     assertSame(firestoreOptions, firebaseOptions.getFirestoreOptions());
 
     GoogleCredentials credentials = firebaseOptions.getCredentials();

src/test/java/com/google/firebase/auth/FirebaseUserManagerTest.java

@@ -979,7 +979,7 @@ public void testTimeout() throws Exception {
         .setHttpTransport(transport)
         .setConnectTimeout(30000)
         .setReadTimeout(60000)
-        .setWriteTimeout(60000)
+        .setWriteTimeout(90000)
         .build());
     FirebaseAuth auth = FirebaseAuth.getInstance();
     FirebaseUserManager userManager = auth.getUserManager();
@@ -990,7 +990,7 @@ public void testTimeout() throws Exception {
     HttpRequest request = interceptor.getResponse().getRequest();
     assertEquals(30000, request.getConnectTimeout());
     assertEquals(60000, request.getReadTimeout());
-    assertEquals(60000, request.getWriteTimeout());
+    assertEquals(90000, request.getWriteTimeout());
   }
 
   @Test

src/test/java/com/google/firebase/internal/FirebaseRequestInitializerTest.java

@@ -38,7 +38,7 @@ public class FirebaseRequestInitializerTest {
   private static final int MAX_RETRIES = 5;
   private static final int CONNECT_TIMEOUT_MILLIS = 30000;
   private static final int READ_TIMEOUT_MILLIS = 60000;
-  private static final int WRITE_TIMEOUT_MILLIS = 60000;
+  private static final int WRITE_TIMEOUT_MILLIS = 90000;
 
   @After
   public void tearDown() {