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

jialiang pushed a commit to branch upgrade/jdk-spring-dependencies
in repository https://gitbox.apache.org/repos/asf/ambari.git


The following commit(s) were added to 
refs/heads/upgrade/jdk-spring-dependencies by this push:
     new 998b33d715 AMBARI-26221: Fix StackAdvisorCommandTest error,remove 
unnecessary code. (#3865)
998b33d715 is described below

commit 998b33d71557192d98480e0801a66a90a81157bb
Author: coldless177 <[email protected]>
AuthorDate: Fri Nov 1 16:12:48 2024 +0800

    AMBARI-26221: Fix StackAdvisorCommandTest error,remove unnecessary code. 
(#3865)
---
 .../api/services/stackadvisor/commands/StackAdvisorCommandTest.java | 6 ------
 1 file changed, 6 deletions(-)

diff --git 
a/ambari-server/src/test/java/org/apache/ambari/server/api/services/stackadvisor/commands/StackAdvisorCommandTest.java
 
b/ambari-server/src/test/java/org/apache/ambari/server/api/services/stackadvisor/commands/StackAdvisorCommandTest.java
index 13ea6b7f32..4b290bc294 100644
--- 
a/ambari-server/src/test/java/org/apache/ambari/server/api/services/stackadvisor/commands/StackAdvisorCommandTest.java
+++ 
b/ambari-server/src/test/java/org/apache/ambari/server/api/services/stackadvisor/commands/StackAdvisorCommandTest.java
@@ -105,7 +105,6 @@ public class StackAdvisorCommandTest {
     int requestId = 0;
     StackAdvisorRunner saRunner = mock(StackAdvisorRunner.class);
     AmbariMetaInfo metaInfo = mock(AmbariMetaInfo.class);
-    
doReturn(Collections.emptyList()).when(metaInfo).getStackParentVersions(anyString(),
 anyString());
     StackAdvisorCommand<TestResource> command = spy(new 
TestStackAdvisorCommand(recommendationsDir, recommendationsArtifactsLifetime,
         ServiceInfo.ServiceAdvisorType.PYTHON, requestId, saRunner, metaInfo, 
null));
 
@@ -125,7 +124,6 @@ public class StackAdvisorCommandTest {
     int requestId = 0;
     StackAdvisorRunner saRunner = mock(StackAdvisorRunner.class);
     AmbariMetaInfo metaInfo = mock(AmbariMetaInfo.class);
-    
doReturn(Collections.emptyList()).when(metaInfo).getStackParentVersions(anyString(),
 anyString());
     StackAdvisorCommand<TestResource> command = spy(new 
TestStackAdvisorCommand(recommendationsDir, recommendationsArtifactsLifetime,
         ServiceInfo.ServiceAdvisorType.PYTHON, requestId, saRunner, metaInfo, 
null));
 
@@ -153,7 +151,6 @@ public class StackAdvisorCommandTest {
     int requestId = 0;
     StackAdvisorRunner saRunner = mock(StackAdvisorRunner.class);
     AmbariMetaInfo metaInfo = mock(AmbariMetaInfo.class);
-    
doReturn(Collections.emptyList()).when(metaInfo).getStackParentVersions(anyString(),
 anyString());
     StackAdvisorCommand<TestResource> command = spy(new 
TestStackAdvisorCommand(recommendationsDir, recommendationsArtifactsLifetime,
         ServiceInfo.ServiceAdvisorType.PYTHON, requestId, saRunner, metaInfo, 
null));
 
@@ -165,8 +162,6 @@ public class StackAdvisorCommandTest {
     doThrow(new 
WebApplicationException()).when(command).adjust(any(StackAdvisorData.class),
         any(StackAdvisorRequest.class));
 
-    doThrow(new StackAdvisorException("error")).when(saRunner)
-        .runScript(any(ServiceInfo.ServiceAdvisorType.class), 
any(StackAdvisorCommandType.class), any(File.class));
     command.invoke(request, ServiceInfo.ServiceAdvisorType.PYTHON);
 
 
@@ -182,7 +177,6 @@ public class StackAdvisorCommandTest {
     final int requestId = 2;
     StackAdvisorRunner saRunner = mock(StackAdvisorRunner.class);
     AmbariMetaInfo metaInfo = mock(AmbariMetaInfo.class);
-    
doReturn(Collections.emptyList()).when(metaInfo).getStackParentVersions(anyString(),
 anyString());
     final StackAdvisorCommand<TestResource> command = spy(new 
TestStackAdvisorCommand(
         recommendationsDir, recommendationsArtifactsLifetime, 
ServiceInfo.ServiceAdvisorType.PYTHON, requestId,
         saRunner, metaInfo, null));


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to