Merge "Use consistent naming for allocating SPI." am: 37303d5e94 am: 4753ed4b1b
am: b6bca5a59d
Change-Id: Ib6845ab0d7f385fbd1d59c0179771542e12c8be0
This commit is contained in:
@@ -80,7 +80,7 @@ public class IpSecManagerTest {
|
||||
int resourceId = 1;
|
||||
IpSecSpiResponse spiResp =
|
||||
new IpSecSpiResponse(IpSecManager.Status.OK, resourceId, DROID_SPI);
|
||||
when(mMockIpSecService.reserveSecurityParameterIndex(
|
||||
when(mMockIpSecService.allocateSecurityParameterIndex(
|
||||
eq(IpSecTransform.DIRECTION_IN),
|
||||
eq(GOOGLE_DNS_4.getHostAddress()),
|
||||
eq(DROID_SPI),
|
||||
@@ -88,7 +88,7 @@ public class IpSecManagerTest {
|
||||
.thenReturn(spiResp);
|
||||
|
||||
IpSecManager.SecurityParameterIndex droidSpi =
|
||||
mIpSecManager.reserveSecurityParameterIndex(
|
||||
mIpSecManager.allocateSecurityParameterIndex(
|
||||
IpSecTransform.DIRECTION_IN, GOOGLE_DNS_4, DROID_SPI);
|
||||
assertEquals(DROID_SPI, droidSpi.getSpi());
|
||||
|
||||
@@ -102,7 +102,7 @@ public class IpSecManagerTest {
|
||||
int resourceId = 1;
|
||||
IpSecSpiResponse spiResp =
|
||||
new IpSecSpiResponse(IpSecManager.Status.OK, resourceId, DROID_SPI);
|
||||
when(mMockIpSecService.reserveSecurityParameterIndex(
|
||||
when(mMockIpSecService.allocateSecurityParameterIndex(
|
||||
eq(IpSecTransform.DIRECTION_OUT),
|
||||
eq(GOOGLE_DNS_4.getHostAddress()),
|
||||
eq(IpSecManager.INVALID_SECURITY_PARAMETER_INDEX),
|
||||
@@ -110,7 +110,7 @@ public class IpSecManagerTest {
|
||||
.thenReturn(spiResp);
|
||||
|
||||
IpSecManager.SecurityParameterIndex randomSpi =
|
||||
mIpSecManager.reserveSecurityParameterIndex(
|
||||
mIpSecManager.allocateSecurityParameterIndex(
|
||||
IpSecTransform.DIRECTION_OUT, GOOGLE_DNS_4);
|
||||
|
||||
assertEquals(DROID_SPI, randomSpi.getSpi());
|
||||
@@ -127,12 +127,13 @@ public class IpSecManagerTest {
|
||||
public void testAllocSpiResUnavaiableExeption() throws Exception {
|
||||
IpSecSpiResponse spiResp =
|
||||
new IpSecSpiResponse(IpSecManager.Status.RESOURCE_UNAVAILABLE, 0, 0);
|
||||
when(mMockIpSecService.reserveSecurityParameterIndex(
|
||||
when(mMockIpSecService.allocateSecurityParameterIndex(
|
||||
anyInt(), anyString(), anyInt(), anyObject()))
|
||||
.thenReturn(spiResp);
|
||||
|
||||
try {
|
||||
mIpSecManager.reserveSecurityParameterIndex(IpSecTransform.DIRECTION_OUT, GOOGLE_DNS_4);
|
||||
mIpSecManager.allocateSecurityParameterIndex(
|
||||
IpSecTransform.DIRECTION_OUT, GOOGLE_DNS_4);
|
||||
fail("ResourceUnavailableException was not thrown");
|
||||
} catch (IpSecManager.ResourceUnavailableException e) {
|
||||
}
|
||||
@@ -144,12 +145,13 @@ public class IpSecManagerTest {
|
||||
@Test
|
||||
public void testAllocSpiSpiUnavaiableExeption() throws Exception {
|
||||
IpSecSpiResponse spiResp = new IpSecSpiResponse(IpSecManager.Status.SPI_UNAVAILABLE, 0, 0);
|
||||
when(mMockIpSecService.reserveSecurityParameterIndex(
|
||||
when(mMockIpSecService.allocateSecurityParameterIndex(
|
||||
anyInt(), anyString(), anyInt(), anyObject()))
|
||||
.thenReturn(spiResp);
|
||||
|
||||
try {
|
||||
mIpSecManager.reserveSecurityParameterIndex(IpSecTransform.DIRECTION_OUT, GOOGLE_DNS_4);
|
||||
mIpSecManager.allocateSecurityParameterIndex(
|
||||
IpSecTransform.DIRECTION_OUT, GOOGLE_DNS_4);
|
||||
fail("ResourceUnavailableException was not thrown");
|
||||
} catch (IpSecManager.ResourceUnavailableException e) {
|
||||
}
|
||||
@@ -161,7 +163,7 @@ public class IpSecManagerTest {
|
||||
@Test
|
||||
public void testRequestAllocInvalidSpi() throws Exception {
|
||||
try {
|
||||
mIpSecManager.reserveSecurityParameterIndex(
|
||||
mIpSecManager.allocateSecurityParameterIndex(
|
||||
IpSecTransform.DIRECTION_OUT, GOOGLE_DNS_4, 0);
|
||||
fail("Able to allocate invalid spi");
|
||||
} catch (IllegalArgumentException e) {
|
||||
|
||||
@@ -125,7 +125,7 @@ public class IpSecServiceParameterizedTest {
|
||||
.thenReturn(TEST_SPI_OUT);
|
||||
|
||||
IpSecSpiResponse spiResp =
|
||||
mIpSecService.reserveSecurityParameterIndex(
|
||||
mIpSecService.allocateSecurityParameterIndex(
|
||||
IpSecTransform.DIRECTION_OUT, mRemoteAddr, TEST_SPI_OUT, new Binder());
|
||||
assertEquals(IpSecManager.Status.OK, spiResp.status);
|
||||
assertEquals(TEST_SPI_OUT, spiResp.spi);
|
||||
@@ -142,7 +142,7 @@ public class IpSecServiceParameterizedTest {
|
||||
.thenReturn(TEST_SPI_OUT);
|
||||
|
||||
IpSecSpiResponse spiResp =
|
||||
mIpSecService.reserveSecurityParameterIndex(
|
||||
mIpSecService.allocateSecurityParameterIndex(
|
||||
IpSecTransform.DIRECTION_OUT, mRemoteAddr, TEST_SPI_OUT, new Binder());
|
||||
|
||||
mIpSecService.releaseSecurityParameterIndex(spiResp.resourceId);
|
||||
@@ -212,7 +212,7 @@ public class IpSecServiceParameterizedTest {
|
||||
.thenReturn(returnSpi);
|
||||
|
||||
IpSecSpiResponse spi =
|
||||
mIpSecService.reserveSecurityParameterIndex(
|
||||
mIpSecService.allocateSecurityParameterIndex(
|
||||
direction,
|
||||
NetworkUtils.numericToInetAddress(remoteAddress).getHostAddress(),
|
||||
IpSecManager.INVALID_SECURITY_PARAMETER_INDEX,
|
||||
|
||||
@@ -287,7 +287,7 @@ public class IpSecServiceTest {
|
||||
for (String address : invalidAddresses) {
|
||||
try {
|
||||
IpSecSpiResponse spiResp =
|
||||
mIpSecService.reserveSecurityParameterIndex(
|
||||
mIpSecService.allocateSecurityParameterIndex(
|
||||
IpSecTransform.DIRECTION_OUT, address, DROID_SPI, new Binder());
|
||||
fail("Invalid address was passed through IpSecService validation: " + address);
|
||||
} catch (IllegalArgumentException e) {
|
||||
@@ -368,7 +368,7 @@ public class IpSecServiceTest {
|
||||
// Reserve spis until it fails.
|
||||
for (int i = 0; i < MAX_NUM_SPIS; i++) {
|
||||
IpSecSpiResponse newSpi =
|
||||
mIpSecService.reserveSecurityParameterIndex(
|
||||
mIpSecService.allocateSecurityParameterIndex(
|
||||
0x1,
|
||||
InetAddress.getLoopbackAddress().getHostAddress(),
|
||||
DROID_SPI + i,
|
||||
@@ -384,7 +384,7 @@ public class IpSecServiceTest {
|
||||
|
||||
// Try to reserve one more SPI, and should fail.
|
||||
IpSecSpiResponse extraSpi =
|
||||
mIpSecService.reserveSecurityParameterIndex(
|
||||
mIpSecService.allocateSecurityParameterIndex(
|
||||
0x1,
|
||||
InetAddress.getLoopbackAddress().getHostAddress(),
|
||||
DROID_SPI + MAX_NUM_SPIS,
|
||||
@@ -398,7 +398,7 @@ public class IpSecServiceTest {
|
||||
|
||||
// Should successfully reserve one more spi.
|
||||
extraSpi =
|
||||
mIpSecService.reserveSecurityParameterIndex(
|
||||
mIpSecService.allocateSecurityParameterIndex(
|
||||
0x1,
|
||||
InetAddress.getLoopbackAddress().getHostAddress(),
|
||||
DROID_SPI + MAX_NUM_SPIS,
|
||||
|
||||
Reference in New Issue
Block a user