When JSch session is null, open channel and execute command after create session and connect

Author: Dokyeongyun

src/main/java/root/common/server/implement/JschServer.java

@@ -77,7 +77,12 @@ public void disConnect(Session session) {
 
 	public Channel openExecChannel(Session session, String command) {
 		if(session == null) {
-			throw new NullPointerException("Session is null");
+			init();
+			try {
+				session = this.connect(this.getSession());
+			} catch (JSchException e) {
+				log.error(e.getMessage());
+			}
 		}
 
 		Channel channel = null;

src/test/java/root/common/server/implement/JschServerTest.java

@@ -103,10 +103,10 @@ public void testOpenExecChannel_Success() throws JSchException {
 	}
 
 	@Test
-	public void testOpenExecChannel_Fail_SessionIsNull() {
-		NullPointerException thrown = assertThrows(NullPointerException.class,
-				() -> jsch.openExecChannel(null, "echo 1"));
-		assertEquals("Session is null", thrown.getMessage());
+	public void testOpenExecChannel_Success_WhenSessionIsNull() throws JSchException {
+		Session session = null;
+		Channel channel = jsch.openExecChannel(session, "echo 1");
+		assertNotNull(channel);
 	}
 
 	@Test
@@ -125,6 +125,19 @@ public void testConnectChannel_Success() throws JSchException, IOException {
 		}
 	}
 
+	@Test
+	public void testConnectChannel_Success_WhenSessionIsNull() throws JSchException, IOException {
+		Session session = null;
+		Channel channel = jsch.openExecChannel(session, "echo 1");
+		InputStream in = jsch.connectChannel(channel);
+
+		assertNotNull(in);
+		try (BufferedReader br = new BufferedReader(new InputStreamReader(in))) {
+			String echoLine = br.readLine();
+			assertEquals(echoLine, "1");
+		}
+	}
+
 	@Test
 	public void testDisConnectChannel_Success() throws JSchException {
 		jsch.init();