Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1637fc2a authored by William Escande's avatar William Escande Committed by Gerrit Code Review
Browse files

Merge changes Ic9061e7d,I847adfe0 into main

* changes:
  TEST: Only read local config when used
  TEST: Add logs to identify low flaky failure
parents ea5dd57a b3072a78
Loading
Loading
Loading
Loading
+2 −0
Original line number Diff line number Diff line
@@ -369,6 +369,7 @@ public class TestUtils {
                }
                if (line.startsWith("[")) {
                    if (line.charAt(line.length() - 1) != ']') {
                        Log.e(TAG, "readAdapterConfig: config line is not correct: " + line);
                        return null;
                    }
                    section = line.substring(1, line.length() - 1);
@@ -380,6 +381,7 @@ public class TestUtils {
                }
            }
        } catch (IOException e) {
            Log.e(TAG, "readAdapterConfig: Exception while reading the config" + e);
            return null;
        }
        return adapterConfig;
+12 −7
Original line number Diff line number Diff line
@@ -179,7 +179,6 @@ public class AdapterServiceTest {

    private PackageManager mMockPackageManager;
    private MockContentResolver mMockContentResolver;
    private HashMap<String, HashMap<String, String>> mAdapterConfig;
    private int mForegroundUserId;
    private TestLooper mLooper;

@@ -345,9 +344,6 @@ public class AdapterServiceTest {
        mLooper.dispatchAll();

        mAdapterService.registerRemoteCallback(mIBluetoothCallback);

        mAdapterConfig = TestUtils.readAdapterConfig();
        assertThat(mAdapterConfig).isNotNull();
    }

    @After
@@ -849,8 +845,11 @@ public class AdapterServiceTest {
    @Test
    @Ignore("b/296127545: This is a native test")
    public void testObfuscateBluetoothAddress_BluetoothDisabled() {
        HashMap<String, HashMap<String, String>> adapterConfig = TestUtils.readAdapterConfig();
        assertThat(adapterConfig).isNotNull();

        assertThat(mAdapterService.getState()).isEqualTo(STATE_OFF);
        byte[] metricsSalt = getMetricsSalt(mAdapterConfig);
        byte[] metricsSalt = getMetricsSalt(adapterConfig);
        assertThat(metricsSalt).isNotNull();
        BluetoothDevice device = TestUtils.getTestDevice(BluetoothAdapter.getDefaultAdapter(), 0);
        byte[] obfuscatedAddress = mAdapterService.obfuscateAddress(device);
@@ -866,10 +865,13 @@ public class AdapterServiceTest {
    @Test
    @Ignore("b/296127545: This is a native test")
    public void testObfuscateBluetoothAddress_BluetoothEnabled() {
        HashMap<String, HashMap<String, String>> adapterConfig = TestUtils.readAdapterConfig();
        assertThat(adapterConfig).isNotNull();

        assertThat(mAdapterService.getState()).isEqualTo(STATE_OFF);
        doEnable(false);
        assertThat(mAdapterService.getState()).isEqualTo(STATE_ON);
        byte[] metricsSalt = getMetricsSalt(mAdapterConfig);
        byte[] metricsSalt = getMetricsSalt(adapterConfig);
        assertThat(metricsSalt).isNotNull();
        BluetoothDevice device = TestUtils.getTestDevice(BluetoothAdapter.getDefaultAdapter(), 0);
        byte[] obfuscatedAddress = mAdapterService.obfuscateAddress(device);
@@ -882,8 +884,11 @@ public class AdapterServiceTest {
    @Test
    @Ignore("b/296127545: This is a native test")
    public void testObfuscateBluetoothAddress_PersistentBetweenToggle() {
        HashMap<String, HashMap<String, String>> adapterConfig = TestUtils.readAdapterConfig();
        assertThat(adapterConfig).isNotNull();

        assertThat(mAdapterService.getState()).isEqualTo(STATE_OFF);
        byte[] metricsSalt = getMetricsSalt(mAdapterConfig);
        byte[] metricsSalt = getMetricsSalt(adapterConfig);
        assertThat(metricsSalt).isNotNull();
        BluetoothDevice device = TestUtils.getTestDevice(BluetoothAdapter.getDefaultAdapter(), 0);
        byte[] obfuscatedAddress1 = mAdapterService.obfuscateAddress(device);