Commit 38ea8753 authored by Amit Mahajan's avatar Amit Mahajan
Browse files

Mark flaky/failing tests as @Postsubmit.

This is done so that telephony tests can be enabled for presubmit.
All tests not passing at least 95% of the time are marked
@Postsubmit. Test passing metrics are based on
go/tel-nyc-mr1-apct-dashboard and go/tel-master-apct-dashboard.
These tests will be fixed as part of b/29579544.

Bug: 31041734
Change-Id: I5e65e94696b6526eb3c270238fd80795319ea2ff
parent 9cec95d6
......@@ -10,7 +10,8 @@ LOCAL_SRC_FILES := $(call all-subdir-java-files)
LOCAL_JAVA_LIBRARIES := android.test.runner telephony-common ims-common services.core
LOCAL_STATIC_JAVA_LIBRARIES := guava \
mockito-target \
android-support-test
android-support-test \
platform-test-annotations
LOCAL_PACKAGE_NAME := FrameworksTelephonyTests
......
......@@ -28,6 +28,7 @@ import android.os.Message;
import android.os.Parcel;
import android.os.SystemClock;
import android.os.UserHandle;
import android.platform.test.annotations.Postsubmit;
import android.telephony.CellInfo;
import android.telephony.CellInfoGsm;
import android.telephony.ServiceState;
......@@ -184,6 +185,7 @@ public class ServiceStateTrackerTest extends TelephonyTest {
assertTrue(sst.getPowerStateFromCarrier());
}
@Postsubmit
@Test
@MediumTest
public void testNoRilTrafficAfterSetRadioPower() {
......
......@@ -16,6 +16,7 @@
package com.android.internal.telephony;
import android.platform.test.annotations.Postsubmit;
import android.test.suitebuilder.annotation.SmallTest;
import org.junit.After;
......@@ -49,12 +50,14 @@ public class Sms7BitEncodingTranslatorTest extends TelephonyTest {
super.tearDown();
}
@Postsubmit
@Test
@SmallTest
public void testNoTranslate() {
assertEquals("123", Sms7BitEncodingTranslator.translate("123"));
}
@Postsubmit
@Test
@SmallTest
public void testCommonTranslate() {
......@@ -68,6 +71,7 @@ public class Sms7BitEncodingTranslatorTest extends TelephonyTest {
assertEquals("OIA", Sms7BitEncodingTranslator.translate(s));
}
@Postsubmit
@Test
@SmallTest
public void testGsmTranslate() {
......@@ -81,6 +85,7 @@ public class Sms7BitEncodingTranslatorTest extends TelephonyTest {
assertEquals("??Ç", Sms7BitEncodingTranslator.translate(s));
}
@Postsubmit
@Test
@SmallTest
public void testCdmaTranslate() {
......
......@@ -17,6 +17,7 @@
package com.android.internal.telephony;
import android.os.Looper;
import android.platform.test.annotations.Postsubmit;
import android.test.AndroidTestCase;
import android.test.suitebuilder.annotation.SmallTest;
......@@ -456,6 +457,7 @@ public class SmsUsageMonitorShortCodeTest extends AndroidTestCase {
new ShortCodeTest(null, "112", CATEGORY_NOT_SHORT_CODE),
};
@Postsubmit
@SmallTest
public void testSmsUsageMonitor() {
// InstrumentationTestRunner prepares a looper, but AndroidJUnitRunner does not.
......
......@@ -24,6 +24,7 @@ import android.os.HandlerThread;
import android.os.RemoteException;
import android.os.UserHandle;
import android.os.UserManager;
import android.platform.test.annotations.Postsubmit;
import android.provider.Telephony;
import android.test.mock.MockContentResolver;
import android.test.suitebuilder.annotation.MediumTest;
......@@ -159,6 +160,7 @@ public class CdmaInboundSmsHandlerTest extends TelephonyTest {
assertEquals("IdleState", getCurrentState().getName());
}
@Postsubmit
@Test
@MediumTest
public void testNewSms() {
......
......@@ -17,6 +17,7 @@
package com.android.internal.telephony.cdma;
import android.os.Parcel;
import android.platform.test.annotations.Postsubmit;
import android.telephony.SmsCbCmasInfo;
import android.telephony.SmsCbMessage;
import android.telephony.cdma.CdmaSmsCbProgramData;
......@@ -482,6 +483,7 @@ public class CdmaSmsCbTest extends AndroidTestCase {
// VZW requirement is to discard message with unsupported charset. Verify that we return null
// for this unsupported character set.
@Postsubmit
@Test @SmallTest
public void testCmasUnsupportedCharSet() throws Exception {
SmsMessage msg = createCmasSmsMessage(SmsEnvelope.SERVICE_CATEGORY_CMAS_EXTREME_THREAT,
......
......@@ -22,6 +22,7 @@ import android.os.AsyncResult;
import android.os.Handler;
import android.os.HandlerThread;
import android.os.Message;
import android.platform.test.annotations.Postsubmit;
import android.telephony.CarrierConfigManager;
import android.telephony.ServiceState;
import android.test.suitebuilder.annotation.SmallTest;
......@@ -282,6 +283,7 @@ public class DataConnectionTest extends TelephonyTest {
hasCapability(NetworkCapabilities.NET_CAPABILITY_NOT_METERED));
}
@Postsubmit
@Test
@SmallTest
public void testNonMeteredCapability() throws Exception {
......
......@@ -28,6 +28,7 @@ import android.os.HandlerThread;
import android.os.Message;
import android.os.PersistableBundle;
import android.os.SystemProperties;
import android.platform.test.annotations.Postsubmit;
import android.telephony.CarrierConfigManager;
import android.test.suitebuilder.annotation.SmallTest;
......@@ -448,6 +449,7 @@ public class ImsPhoneTest extends TelephonyTest {
assertEquals(msg, messageArgumentCaptor.getValue().obj);
}
@Postsubmit
@Test
@SmallTest
public void testCallForwardingOption() throws Exception {
......@@ -573,6 +575,7 @@ public class ImsPhoneTest extends TelephonyTest {
assertEquals(false, mImsPhoneUT.getWakeLock().isHeld());
}
@Postsubmit
@Test
@SmallTest
public void testProcessDisconnectReason() throws Exception {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment