Skip to content

Commit d985620

Browse files
committed
Clarify KVM system VM console viewer test
1 parent 7abc435 commit d985620

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

server/src/test/java/org/apache/cloudstack/consoleproxy/ConsoleAccessManagerImplTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -596,7 +596,7 @@ public void getConsoleConnectionDetailsDoesNotRequireNewViewerForKVMUserVmWhenMu
596596
}
597597

598598
@Test
599-
public void getConsoleConnectionDetailsDoesNotRequireNewViewerForKVMSystemVmWhenMultipleViewersEnabled() {
599+
public void getConsoleConnectionDetailsDoesNotRequireNewViewerForKVMSystemVm() {
600600
VirtualMachine vm = Mockito.mock(VirtualMachine.class);
601601
HostVO host = Mockito.mock(HostVO.class);
602602
String hostAddress = "192.168.1.100";
@@ -612,11 +612,11 @@ public void getConsoleConnectionDetailsDoesNotRequireNewViewerForKVMSystemVmWhen
612612
Mockito.when(managementServer.getVncPort(vm)).thenReturn(hostPortInfo);
613613
Mockito.doReturn(new Ternary<>(hostAddress, null, null))
614614
.when(consoleAccessManager).parseHostInfo(Mockito.anyString());
615-
616615
ConsoleConnectionDetails result = consoleAccessManager.getConsoleConnectionDetails(vm, host);
617616

618617
Assert.assertNotNull(result);
619618
Assert.assertFalse(result.isSessionRequiresNewViewer());
619+
Mockito.verify(consoleAccessManager, Mockito.never()).isKvmMultipleConsoleViewersEnabled();
620620
}
621621

622622
@Test

0 commit comments

Comments
 (0)