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

Commit ae6c5b97 authored by Priyanka Advani's avatar Priyanka Advani Committed by Android (Google) Code Review
Browse files

Revert "Ravenwood/HostStubGen refactor and bug fixe" 

This reverts commit ec5e313e.

Reason for revert: Probable culprit for b/332911993. Will be verifying through ABTD for confirmation and before submitting the revert.

Change-Id: I030e2d27d6b897013106b9d73b57b50823ea87c9
parent ec5e313e

ravenwood/bivalenttest/test/com/android/platform/test/ravenwood/bivalenttest/RavenwoodAndroidApiTest.java

deleted100644 → 0
+0 −47
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2024 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */

package com.android.platform.test.ravenwood.bivalenttest;



import static org.junit.Assert.assertEquals;



import android.util.ArrayMap;

import android.util.Size;



import androidx.test.ext.junit.runners.AndroidJUnit4;



import org.junit.Test;

import org.junit.runner.RunWith;



import java.util.Map;



// Tests for calling simple Android APIs.

@RunWith(AndroidJUnit4.class)

public class RavenwoodAndroidApiTest {

    @Test

    public void testArrayMapSimple() {

        final Map<String, String> map = new ArrayMap<>();



        map.put("key1", "value1");

        assertEquals("value1", map.get("key1"));

    }



    @Test

    public void testSizeSimple() {

        final var size = new Size(1, 2);



        assertEquals(2, size.getHeight());

    }

}

ravenwood/bivalenttest/test/com/android/platform/test/ravenwood/bivalenttest/RavenwoodClassRuleDeviceOnlyTest.java

deleted100644 → 0
+0 −39
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2024 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */

package com.android.platform.test.ravenwood.bivalenttest;



import android.platform.test.annotations.DisabledOnRavenwood;

import android.platform.test.ravenwood.RavenwoodClassRule;

import android.platform.test.ravenwood.RavenwoodRule;



import androidx.test.ext.junit.runners.AndroidJUnit4;



import org.junit.Assert;

import org.junit.ClassRule;

import org.junit.Test;

import org.junit.runner.RunWith;



@RunWith(AndroidJUnit4.class)

@DisabledOnRavenwood

public class RavenwoodClassRuleDeviceOnlyTest {

    @ClassRule

    public static final RavenwoodClassRule sRavenwood = new RavenwoodClassRule();



    @Test

    public void testDeviceOnly() {

        Assert.assertFalse(RavenwoodRule.isOnRavenwood());

    }

}

ravenwood/bivalenttest/test/com/android/platform/test/ravenwood/bivalenttest/RavenwoodClassRuleRavenwoodOnlyTest.java

deleted100644 → 0
+0 −43
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2024 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */

package com.android.platform.test.ravenwood.bivalenttest;



import android.platform.test.ravenwood.RavenwoodClassRule;

import android.platform.test.ravenwood.RavenwoodRule;



import androidx.test.ext.junit.runners.AndroidJUnit4;



import org.junit.Assert;

import org.junit.ClassRule;

import org.junit.Ignore;

import org.junit.Test;

import org.junit.runner.RunWith;



@RunWith(AndroidJUnit4.class)

// TODO: atest RavenwoodBivalentTest_device fails with the following message.

// `RUNNER ERROR: Instrumentation reported numtests=7 but only ran 6`

// @android.platform.test.annotations.DisabledOnNonRavenwood

// Figure it out and then make DisabledOnNonRavenwood support TYPEs as well.

@Ignore

public class RavenwoodClassRuleRavenwoodOnlyTest {

    @ClassRule

    public static final RavenwoodClassRule sRavenwood = new RavenwoodClassRule();



    @Test

    public void testRavenwoodOnly() {

        Assert.assertTrue(RavenwoodRule.isOnRavenwood());

    }

}

ravenwood/coretest/test/com/android/platform/test/ravenwood/coretest/RavenwoodTestRunnerValidationTest.java

+1 −1
Original line number Diff line number Diff line
@@ -40,7 +40,7 @@ public class RavenwoodTestRunnerValidationTest { 
    public final RuleChain chain = RuleChain.outerRule(mThrown).around(mRavenwood);



    public RavenwoodTestRunnerValidationTest() {

        Assume.assumeTrue(RavenwoodRule._$RavenwoodPrivate.isOptionalValidationEnabled());

        Assume.assumeTrue(mRavenwood._ravenwood_private$isOptionalValidationEnabled());

        // Because RavenwoodRule will throw this error before executing the test method,

        // we can't do it in the test method itself.

        // So instead, we initialize it here.

ravenwood/junit-src/android/platform/test/annotations/DisabledOnNonRavenwood.java

+1 −5
Original line number Diff line number Diff line
@@ -31,17 +31,13 @@ import java.lang.annotation.Target; 
 * which means if a test class has this annotation, you can't negate it in subclasses or

 * on a per-method basis.

 *

 * THIS ANNOTATION CANNOT BE ADDED TO CLASSES AT THIS PONINT.

 * See {@link com.android.platform.test.ravenwood.bivalenttest.RavenwoodClassRuleRavenwoodOnlyTest}

 * for the reason.

 *

 * The {@code RAVENWOOD_RUN_DISABLED_TESTS} environmental variable won't work because it won't be

 * propagated to the device. (We may support it in the future, possibly using a debug. sysprop.)

 *

 * @hide

 */

@Inherited

@Target({ElementType.METHOD})

@Target({ElementType.METHOD, ElementType.TYPE})

@Retention(RetentionPolicy.RUNTIME)

public @interface DisabledOnNonRavenwood {

    /**