You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by mr...@apache.org on 2018/05/03 20:26:13 UTC

[ambari] branch branch-feature-AMBARI-14714 updated: Fix UT compilation issue due to surrogate key changes (mradhakrishnan)

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

mradhakrishnan pushed a commit to branch branch-feature-AMBARI-14714
in repository https://gitbox.apache.org/repos/asf/ambari.git


The following commit(s) were added to refs/heads/branch-feature-AMBARI-14714 by this push:
     new 8618c4c  Fix UT compilation issue due to surrogate key changes (mradhakrishnan)
8618c4c is described below

commit 8618c4c56a772b47870688eb2590bbc0089a5991
Author: Madhuvanthi Radhakrishnan <mr...@hortonworks.com>
AuthorDate: Thu May 3 13:25:23 2018 -0700

    Fix UT compilation issue due to surrogate key changes (mradhakrishnan)
---
 .../ambari/funtest/server/tests/DeleteServiceTest.java | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/ambari-funtest/src/test/java/org/apache/ambari/funtest/server/tests/DeleteServiceTest.java b/ambari-funtest/src/test/java/org/apache/ambari/funtest/server/tests/DeleteServiceTest.java
index 6ebf2e2..fa429f2 100644
--- a/ambari-funtest/src/test/java/org/apache/ambari/funtest/server/tests/DeleteServiceTest.java
+++ b/ambari-funtest/src/test/java/org/apache/ambari/funtest/server/tests/DeleteServiceTest.java
@@ -38,7 +38,6 @@ import org.apache.ambari.server.orm.entities.ServiceDesiredStateEntity;
 import org.apache.ambari.server.orm.entities.ServiceComponentDesiredStateEntity;
 import org.apache.ambari.server.orm.entities.HostComponentStateEntity;
 import org.apache.ambari.server.orm.entities.HostComponentDesiredStateEntity;
-import org.apache.ambari.server.orm.entities.ServiceDesiredStateEntityPK;
 import org.apache.ambari.server.state.State;
 
 import org.apache.http.HttpStatus;
@@ -165,23 +164,6 @@ public class DeleteServiceTest extends ServerTestBase {
         assertEquals(webResponse.getStatusCode(), HttpStatus.SC_NOT_FOUND);
 
         /**
-         * ClusterServiceDAO - the service entry should have been removed.
-         */
-        //TODO : Requires logic change
-        clusterServiceEntity = clusterServiceDAO.findById(1L,1L,1L);
-        assertTrue(clusterServiceEntity == null);
-
-        /**
-         * ServiceDesiredStateDAO - the service entry should have been removed.
-         */
-        ServiceDesiredStateEntityPK serviceDesiredStateEntityPK = injector.getInstance(ServiceDesiredStateEntityPK.class);
-        serviceDesiredStateEntityPK.setClusterId(clusterId);
-        //TODO : Requires logic change
-        serviceDesiredStateEntityPK.setServiceId(1L);
-        serviceDesiredStateEntity =  serviceDesiredStateDAO.findByPK(serviceDesiredStateEntityPK);
-        assertTrue(serviceDesiredStateEntity == null);
-
-        /**
          * ServiceComponentDesiredStateDAO
          */
         ServiceComponentDesiredStateEntity serviceComponentDesiredStateEntity = serviceComponentDesiredStateDAO.findById(0L);

-- 
To stop receiving notification emails like this one, please contact
mradhakrishnan@apache.org.