From e6317e5caab717103467a3222eb7d0581d68e61e Mon Sep 17 00:00:00 2001 From: Treehugger Robot Date: Mon, 13 Jul 2020 09:35:07 +0000 Subject: [PATCH] Fix EthernetTetheringTest failure if tethering is not supported Do not run ethernet tethering tests if tethering is not supported because Tethering APIs would fail. Also skip the test if EthernetManager is not avaliable. EthernetManager would be null if there are no FEATURE_ETHERNET and FEATURE_USB_HOST. Bug: 159869957 Test: atest CtsTetheringTest Merged-In: I2b9be6799c6edeefc8cd74897a8704dbe80dd061 Change-Id: I2b9be6799c6edeefc8cd74897a8704dbe80dd061 --- .../src/android/net/EthernetTetheringTest.java | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/packages/Tethering/tests/integration/src/android/net/EthernetTetheringTest.java b/packages/Tethering/tests/integration/src/android/net/EthernetTetheringTest.java index e10bab4b36b59..9bb01ae5df1d1 100644 --- a/packages/Tethering/tests/integration/src/android/net/EthernetTetheringTest.java +++ b/packages/Tethering/tests/integration/src/android/net/EthernetTetheringTest.java @@ -102,17 +102,21 @@ public class EthernetTetheringTest { private UiAutomation mUiAutomation = InstrumentationRegistry.getInstrumentation().getUiAutomation(); + private boolean mRunTests; @Before public void setUp() throws Exception { - mHandlerThread = new HandlerThread(getClass().getSimpleName()); - mHandlerThread.start(); - mHandler = new Handler(mHandlerThread.getLooper()); - mTetheredInterfaceRequester = new TetheredInterfaceRequester(mHandler, mEm); // Needed to create a TestNetworkInterface, to call requestTetheredInterface, and to receive // tethered client callbacks. mUiAutomation.adoptShellPermissionIdentity( MANAGE_TEST_NETWORKS, NETWORK_SETTINGS, TETHER_PRIVILEGED); + mRunTests = mTm.isTetheringSupported() && mEm != null; + assumeTrue(mRunTests); + + mHandlerThread = new HandlerThread(getClass().getSimpleName()); + mHandlerThread.start(); + mHandler = new Handler(mHandlerThread.getLooper()); + mTetheredInterfaceRequester = new TetheredInterfaceRequester(mHandler, mEm); } private void cleanUp() throws Exception { @@ -136,7 +140,7 @@ public class EthernetTetheringTest { @After public void tearDown() throws Exception { try { - cleanUp(); + if (mRunTests) cleanUp(); } finally { mUiAutomation.dropShellPermissionIdentity(); }