erickalendra commented on code in PR #10686: URL: https://github.com/apache/cloudstack/pull/10686#discussion_r2037803726
########## server/src/test/java/com/cloud/network/NetworkServiceImplTest.java: ########## @@ -141,11 +146,11 @@ public class NetworkServiceImplTest { @Mock VpcManager vpcMgr; @Mock - NetworkOrchestrationService networkManager; + NetworkOrchestrationService _networkMgr; @Mock AlertManager alertManager; @Mock - DataCenterDao dcDao; + DataCenterDao _dcDao; Review Comment: Error message if code change is accepted: ``` [ERROR] testPerformBasicChecksPromiscuousTypeExpectedCommunitySet(com.cloud.network.NetworkServiceImplTest) Time elapsed: 0.002 s <<< ERROR! org.mockito.exceptions.base.MockitoException: Mockito couldn't inject mock dependency on field 'com.cloud.dc.dao.DataCenterDao com.cloud.network.NetworkServiceImpl._dcDao' that is annotated with @InjectMocks in your test, because there were multiple matching mocks (i.e. fields annotated with @Mock and having matching type): job, networkOfferingServiceMapDao, routerDao, accountMock, routedIpv4Manager, dataCenterDao, alertManager, physicalNetworkDao, entityMgr, commandSetupHelper, networkService, networkOrchestrationService, nicDao, networkHelper, ipv6Service, publicMtuKey, serviceOfferingVoMock, networkOfferingDao, domainVOMock, _responseObject, ipAddressDao, userChangeMtuKey, publicIpQuarantineDaoMock, bgpService, privateMtuKey, userDao, ipAddressManagerMock, vpcMgr, ipMock, vpcDao, _networkMgr, dcDao, serviceOfferingDaoMock, updateQuarantinedIpCmdMock, callContextMock, domainDaoMock, publicIpQuarantineVOMock, networkDao, networkModel, accountVOMock, ipAddressVOMock, nsxProviderDao. If you have multiple fields of same type in your class under test then consider naming the @Mock fields identically to the respective class under test's fields, so Mockito can match them by name. [ERROR] testPerformBasicChecksPromiscuousTypeExpectedCommunitySet(com.cloud.network.NetworkServiceImplTest) Time elapsed: 0.002 s <<< ERROR! java.lang.NullPointerException: Cannot invoke "org.mockito.MockedStatic.close()" because "this.callContextMocked" is null at com.cloud.network.NetworkServiceImplTest.tearDown(NetworkServiceImplTest.java:334) ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org