diff --git a/Cronet/tests/cts/src/android/net/http/cts/BidirectionalStreamTest.kt b/Cronet/tests/cts/src/android/net/http/cts/BidirectionalStreamTest.kt index 3a77097780..464862d8a6 100644 --- a/Cronet/tests/cts/src/android/net/http/cts/BidirectionalStreamTest.kt +++ b/Cronet/tests/cts/src/android/net/http/cts/BidirectionalStreamTest.kt @@ -27,6 +27,7 @@ import android.os.Build import androidx.test.core.app.ApplicationProvider import com.android.testutils.DevSdkIgnoreRule import com.android.testutils.DevSdkIgnoreRunner +import com.android.testutils.SkipPresubmit import com.google.common.truth.Truth.assertThat import kotlin.test.Test import kotlin.test.assertEquals @@ -35,7 +36,6 @@ import org.hamcrest.Matchers import org.junit.After import org.junit.AssumptionViolatedException import org.junit.Before -import org.junit.Ignore import org.junit.runner.RunWith private const val URL = "https://source.android.com" @@ -69,7 +69,7 @@ class BidirectionalStreamTest { @Test @Throws(Exception::class) - @Ignore("b/292298108 Re-enable and confirm non-flaky after SLO") + @SkipPresubmit(reason = "b/293141085 Confirm non-flaky and move to presubmit after SLO") fun testBidirectionalStream_GetStream_CompletesSuccessfully() { stream = createBidirectionalStreamBuilder(URL).setHttpMethod("GET").build() stream!!.start() diff --git a/Cronet/tests/mts/jarjar_excludes.txt b/Cronet/tests/mts/jarjar_excludes.txt index a3e86de3b7..e8fd39b3d4 100644 --- a/Cronet/tests/mts/jarjar_excludes.txt +++ b/Cronet/tests/mts/jarjar_excludes.txt @@ -1,5 +1,8 @@ -# It's prohibited to jarjar androidx packages +# jarjar-gen can't handle some kotlin object expression, exclude packages that include them androidx\..+ +kotlin\.test\..+ +kotlin\.reflect\..+ +org\.mockito\..+ # Do not jarjar the api classes android\.net\..+ # cronet_tests.so is not jarjared and uses base classes. We can remove this when there's a diff --git a/TEST_MAPPING b/TEST_MAPPING index 0326bf25af..83d435ac3e 100644 --- a/TEST_MAPPING +++ b/TEST_MAPPING @@ -99,6 +99,9 @@ { "name": "NetHttpCoverageTests", "options": [ + { + "exclude-annotation": "com.android.testutils.SkipPresubmit" + }, { // These sometimes take longer than 1 min which is the presubmit timeout "exclude-annotation": "androidx.test.filters.LargeTest" @@ -219,6 +222,9 @@ { "name": "NetHttpCoverageTests[CaptivePortalLoginGoogle.apk+NetworkStackGoogle.apk+com.google.android.resolv.apex+com.google.android.tethering.apex]", "options": [ + { + "exclude-annotation": "com.android.testutils.SkipPresubmit" + }, { // These sometimes take longer than 1 min which is the presubmit timeout "exclude-annotation": "androidx.test.filters.LargeTest"