Skip to content

Commit c4a57f9

Browse files
committed
update 2: fix build/test errors in 4.19
1 parent 4e25ea2 commit c4a57f9

File tree

3 files changed

+6
-7
lines changed

3 files changed

+6
-7
lines changed

server/src/test/java/com/cloud/network/vpc/NetworkACLServiceImplTest.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -622,8 +622,8 @@ public void validateProtocolTestProtocolIsStringValid() {
622622

623623
@Test(expected = InvalidParameterValueException.class)
624624
public void validateProtocolTestProtocolNotIcmpWithIcmpConfigurations() {
625-
Mockito.when(networkAclItemVoMock.getIcmpCode()).thenReturn(1);
626-
Mockito.when(networkAclItemVoMock.getIcmpType()).thenReturn(1);
625+
Mockito.when(networkAclItemVoMock.getIcmpCode()).thenReturn(2);
626+
Mockito.when(networkAclItemVoMock.getIcmpType()).thenReturn(3);
627627

628628
Mockito.when(networkAclItemVoMock.getProtocol()).thenReturn("tcp");
629629
networkAclServiceImpl.validateProtocol(networkAclItemVoMock);
@@ -647,8 +647,8 @@ public void validateProtocolTestProtocolNotIcmpWithSourcePorts() {
647647

648648
@Test
649649
public void validateProtocolTestProtocolIcmpWithIcmpConfigurations() {
650-
Mockito.when(networkAclItemVoMock.getIcmpCode()).thenReturn(1);
651-
Mockito.when(networkAclItemVoMock.getIcmpType()).thenReturn(1);
650+
Mockito.when(networkAclItemVoMock.getIcmpCode()).thenReturn(2);
651+
Mockito.when(networkAclItemVoMock.getIcmpType()).thenReturn(3);
652652

653653
Mockito.when(networkAclItemVoMock.getSourcePortStart()).thenReturn(null);
654654
Mockito.when(networkAclItemVoMock.getSourcePortEnd()).thenReturn(null);

utils/src/test/java/com/cloud/utils/net/NetUtilsTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -855,7 +855,6 @@ private void validateIcmpTypeAndCodeInternal(Integer icmpType, Integer icmpCode,
855855
}
856856

857857
@Test
858-
@PrepareForTest(NetUtils.class)
859858
public void validateIcmpTypeAndCodes() {
860859
validateIcmpTypeAndCodeInternal(-1, -1, true);
861860
validateIcmpTypeAndCodeInternal(3, 2, true);

utils/src/test/java/com/cloud/utils/net/NetworkProtocolsTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,10 @@
2323
import org.junit.Test;
2424

2525
import org.junit.runner.RunWith;
26-
import org.powermock.modules.junit4.PowerMockRunner;
26+
import org.mockito.runners.MockitoJUnitRunner;
2727

2828

29-
@RunWith(PowerMockRunner.class)
29+
@RunWith(MockitoJUnitRunner.class)
3030
public class NetworkProtocolsTest {
3131

3232
@Test

0 commit comments

Comments
 (0)