Browse Source

AMBARI-7860. Download client configs not working, returns 404 (dlysnichenko)

Lisnichenko Dmitro 10 years ago
parent
commit
47929dce2e

+ 2 - 0
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProvider.java

@@ -331,6 +331,8 @@ public class ClientConfigResourceProvider extends AbstractControllerResourceProv
       throw new SystemException("Controller error ", e);
       throw new SystemException("Controller error ", e);
     }
     }
 
 
+    Resource resource = new ResourceImpl(Resource.Type.ClientConfig);
+    resources.add(resource);
     return resources;
     return resources;
   }
   }
 
 

+ 3 - 3
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProviderTest.java

@@ -43,6 +43,7 @@ import java.lang.reflect.Method;
 import java.util.*;
 import java.util.*;
 
 
 import static org.easymock.EasyMock.*;
 import static org.easymock.EasyMock.*;
+import static org.junit.Assert.assertFalse;
 
 
 /**
 /**
  * TaskResourceProvider tests.
  * TaskResourceProvider tests.
@@ -293,9 +294,8 @@ public class ClientConfigResourceProviderTest {
             runtime, process);
             runtime, process);
     PowerMock.replayAll();
     PowerMock.replayAll();
 
 
-    provider.getResources(request, predicate);
-
-
+    Set<Resource> resources = provider.getResources(request, predicate);
+    assertFalse(resources.isEmpty());
 
 
     // verify
     // verify
     verify(managementController, clusters, cluster, ambariMetaInfo, stackId, componentInfo,commandScriptDefinition,
     verify(managementController, clusters, cluster, ambariMetaInfo, stackId, componentInfo,commandScriptDefinition,