From 4fc2dbb970cd6a921a371eb35b17da957b79dff0 Mon Sep 17 00:00:00 2001 From: Mohannad Farrag Date: Fri, 15 Mar 2024 12:20:48 +0000 Subject: [PATCH] Switch Cronet MTS to Orchestrator * Switching Cronet tests to the orchestrator means that we will be able to run tests exactly like how they are run in Chromium, as seen in this CL, this has allowed running previously disabled tests. * This change comes with an introduced latency which is not hugely impactful (Measured to be around 400ms per test case) which takes our runtime from ~6 minutes to ~10 minutes on average. This is an acceptable impact for better coverage. If it increases beyond that then we can utilize sharding but that is not required at the moment or in the foreseeable future unless we have passed 1000 test case. Test: atest NetHttpCoverageTest Bug: 329823447 Change-Id: I1fff900341d958b8f00fc777e909b3583a83cc6a --- Cronet/tests/common/AndroidTest.xml | 13 ++----------- Cronet/tests/mts/AndroidTest.xml | 11 +---------- 2 files changed, 3 insertions(+), 21 deletions(-) diff --git a/Cronet/tests/common/AndroidTest.xml b/Cronet/tests/common/AndroidTest.xml index bded8fb695..ae6b65bcc7 100644 --- a/Cronet/tests/common/AndroidTest.xml +++ b/Cronet/tests/common/AndroidTest.xml @@ -35,26 +35,17 @@