diff options
author | Chen Xu <fionaxu@google.com> | 2020-06-02 13:46:23 -0700 |
---|---|---|
committer | Chen Xu <fionaxu@google.com> | 2020-06-02 20:51:11 +0000 |
commit | f50c1b447dc4f8de9b94e898e0577efaf4d54833 (patch) | |
tree | e9d4cac34fea87e03e6d9406b0778c71faad4a98 /tests/telephonytests | |
parent | 3b2e21f702eabfc741a9c28e2171398d955598be (diff) | |
download | telephony-f50c1b447dc4f8de9b94e898e0577efaf4d54833.tar.gz |
use createStringArray API
public API readStringArray(int[]) will trigger Parcel.readInt() twice. Instead
we should use the public API createStringArray
Bug: 157879439
Test: atest AdnRecordTest
Change-Id: Idd33e2983b0b367100bd487ee30e95682e2d6434
Diffstat (limited to 'tests/telephonytests')
-rw-r--r-- | tests/telephonytests/src/com/android/internal/telephony/AdnRecordTest.java | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/tests/telephonytests/src/com/android/internal/telephony/AdnRecordTest.java b/tests/telephonytests/src/com/android/internal/telephony/AdnRecordTest.java index d647ae13b1..3b786b0162 100644 --- a/tests/telephonytests/src/com/android/internal/telephony/AdnRecordTest.java +++ b/tests/telephonytests/src/com/android/internal/telephony/AdnRecordTest.java @@ -16,6 +16,8 @@ package com.android.internal.telephony; +import android.os.Parcel; +import java.util.Arrays; import junit.framework.TestCase; import android.test.suitebuilder.annotation.SmallTest; @@ -174,6 +176,19 @@ public class AdnRecordTest extends TestCase { assertEquals("+18885551212,12345678", adn.getNumber()); assertFalse(adn.isEmpty()); } + + @SmallTest + public void testParcelUnParcel() throws Exception { + AdnRecord adn = new AdnRecord(0,0,"Voice Mail", + "+18056377243", new String[]{"adc@email.com"}); + Parcel p = Parcel.obtain(); + adn.writeToParcel(p, 0); + p.setDataPosition(0); + AdnRecord copy = AdnRecord.CREATOR.createFromParcel(p); + assertEquals(adn.getAlphaTag(), copy.getAlphaTag()); + assertEquals(adn.getNumber(), copy.getNumber()); + assertTrue(Arrays.equals(adn.getEmails(), copy.getEmails())); + } } |