This is an automated email from the ASF dual-hosted git repository.

rohit pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


The following commit(s) were added to refs/heads/main by this push:
     new c83dee5851 server: fix exception in DeploymentPlanningManagerImplTest 
(#6799)
c83dee5851 is described below

commit c83dee585174ffa99b4f210a93fe208ada9ad327
Author: Wei Zhou <[email protected]>
AuthorDate: Tue Oct 4 07:27:12 2022 +0200

    server: fix exception in DeploymentPlanningManagerImplTest (#6799)
    
    previously
    ```
    [INFO] Running com.cloud.deploy.DeploymentPlanningManagerImplTest
    
    java.lang.NullPointerException
            at 
com.cloud.deploy.DeploymentPlanningManagerImpl.planDeployment(DeploymentPlanningManagerImpl.java:403)
            at 
com.cloud.deploy.DeploymentPlanningManagerImplTest.failEncRootLastHostNotSupportingEncryptionTest(DeploymentPlanningManagerImplTest.java:644)
    
    ...
    [INFO] Tests run: 27, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 
2.407 s - in com.cloud.deploy.DeploymentPlanningManagerImplTest
    
    ```
    
    with this PR
    
    ```
    [INFO] Running com.cloud.deploy.DeploymentPlanningManagerImplTest
    
    [INFO] Tests run: 27, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 
2.481 s - in com.cloud.deploy.DeploymentPlanningManagerImplTest
    [INFO]
    [INFO] Results:
    [INFO]
    [INFO] Tests run: 27, Failures: 0, Errors: 0, Skipped: 0
    ```
---
 .../test/java/com/cloud/deploy/DeploymentPlanningManagerImplTest.java  | 3 +++
 1 file changed, 3 insertions(+)

diff --git 
a/server/src/test/java/com/cloud/deploy/DeploymentPlanningManagerImplTest.java 
b/server/src/test/java/com/cloud/deploy/DeploymentPlanningManagerImplTest.java
index 41d5eaa223..262aab4478 100644
--- 
a/server/src/test/java/com/cloud/deploy/DeploymentPlanningManagerImplTest.java
+++ 
b/server/src/test/java/com/cloud/deploy/DeploymentPlanningManagerImplTest.java
@@ -193,6 +193,9 @@ public class DeploymentPlanningManagerImplTest {
     @Mock
     Host host;
 
+    @Mock
+    ConfigurationDao configDao;
+
     private static final long dataCenterId = 1L;
     private static final long instanceId = 123L;
     private static final long hostId = 0L;

Reply via email to