commit 4bc78be2713d20b1013b1afe0f867ae6962af221
author Dan Stahr <danstahr@google.com> Wed Mar 15 09:05:17 2023 +0000
committer Dan Stahr <danstahr@google.com> Wed Mar 15 09:08:59 2023 +0000
tree bc7039067afc7c0dced83869299f7346576dd1ea
parent e3e791bf81ffb6bbf4405966bbce4aa41bb8934d

Enable tests for idle connection timeout getter / setter.

Test: atest CtsNetHttpTestCases
Bug: 273626798, 273626794, 251012627
Change-Id: Ie6f4ee3f17fa8e2dc82a059bc13ad305771ed329

Cronet/tests/cts/src/android/net/http/cts/QuicOptionsTest.kt

diff --git a/Cronet/tests/cts/src/android/net/http/cts/QuicOptionsTest.kt b/Cronet/tests/cts/src/android/net/http/cts/QuicOptionsTest.kt
index 5f9b7d4..a05aecd 100644
--- a/Cronet/tests/cts/src/android/net/http/cts/QuicOptionsTest.kt
+++ b/Cronet/tests/cts/src/android/net/http/cts/QuicOptionsTest.kt
@@ -18,6 +18,7 @@
 import android.net.http.QuicOptions
 import androidx.test.ext.junit.runners.AndroidJUnit4
 import com.google.common.truth.Truth.assertThat
+import java.time.Duration
 import org.junit.Test
 import org.junit.runner.RunWith
 
@@ -28,8 +29,7 @@
         val quicOptions = QuicOptions.Builder().build()
         assertThat(quicOptions.allowedQuicHosts).isEmpty()
         assertThat(quicOptions.handshakeUserAgent).isNull()
-        // TODO(danstahr): idleConnectionTimeout getter should be public
-        // assertThat(quicOptions.idleConnectionTimeout).isNull()
+        assertThat(quicOptions.idleConnectionTimeout).isNull()
         assertThat(quicOptions.inMemoryServerConfigsCacheSize).isNull()
     }
 
@@ -46,8 +46,6 @@
                 .inOrder()
     }
 
-    // TODO(danstahr): idleConnectionTimeout getter should be public
-    /*
     @Test
     fun testQuicOptions_idleConnectionTimeout_returnsSetValue() {
         val timeout = Duration.ofMinutes(10)
@@ -57,7 +55,6 @@
         assertThat(quicOptions.idleConnectionTimeout)
                 .isEqualTo(timeout)
     }
-    */
 
     @Test
     fun testQuicOptions_inMemoryServerConfigsCacheSize_returnsSetValue() {