Move terminal ui framework to a dedicated module

Resolves #847

Author: fmbenhassine

pom.xml

@@ -82,6 +82,7 @@
 
 	<modules>
 		<module>spring-shell-table</module>
+		<module>spring-shell-tui</module>
 		<module>spring-shell-core</module>
 		<module>spring-shell-standard</module>
 		<module>spring-shell-test</module>

spring-shell-autoconfigure/src/main/java/org/springframework/shell/boot/ComponentFlowAutoConfiguration.java

@@ -27,9 +27,9 @@
 import org.springframework.context.annotation.Scope;
 import org.springframework.core.annotation.Order;
 import org.springframework.core.io.ResourceLoader;
-import org.springframework.shell.component.flow.ComponentFlow;
-import org.springframework.shell.component.flow.ComponentFlow.Builder;
-import org.springframework.shell.style.TemplateExecutor;
+import org.springframework.shell.tui.component.flow.ComponentFlow;
+import org.springframework.shell.tui.component.flow.ComponentFlow.Builder;
+import org.springframework.shell.tui.style.TemplateExecutor;
 
 /**
  * {@link EnableAutoConfiguration Auto-configuration} for {@link ComponentFlow}.

spring-shell-autoconfigure/src/main/java/org/springframework/shell/boot/ComponentFlowCustomizer.java

@@ -15,7 +15,7 @@
  */
 package org.springframework.shell.boot;
 
-import org.springframework.shell.component.flow.ComponentFlow;
+import org.springframework.shell.tui.component.flow.ComponentFlow;
 
 /**
  * Callback interface that can be used to customize a {@link ComponentFlow.Builder}.

spring-shell-autoconfigure/src/main/java/org/springframework/shell/boot/StandardCommandsAutoConfiguration.java

@@ -40,7 +40,7 @@
 import org.springframework.shell.standard.commands.Script;
 import org.springframework.shell.standard.commands.Stacktrace;
 import org.springframework.shell.standard.commands.Version;
-import org.springframework.shell.style.TemplateExecutor;
+import org.springframework.shell.tui.style.TemplateExecutor;
 
 /**
  * Creates beans for standard commands.

spring-shell-autoconfigure/src/main/java/org/springframework/shell/boot/TerminalUIAutoConfiguration.java

@@ -23,13 +23,13 @@
 import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Scope;
-import org.springframework.shell.component.ViewComponentBuilder;
-import org.springframework.shell.component.ViewComponentExecutor;
-import org.springframework.shell.component.view.TerminalUI;
-import org.springframework.shell.component.view.TerminalUIBuilder;
-import org.springframework.shell.component.view.TerminalUICustomizer;
-import org.springframework.shell.style.ThemeActive;
-import org.springframework.shell.style.ThemeResolver;
+import org.springframework.shell.tui.component.ViewComponentBuilder;
+import org.springframework.shell.tui.component.ViewComponentExecutor;
+import org.springframework.shell.tui.component.view.TerminalUI;
+import org.springframework.shell.tui.component.view.TerminalUIBuilder;
+import org.springframework.shell.tui.component.view.TerminalUICustomizer;
+import org.springframework.shell.tui.style.ThemeActive;
+import org.springframework.shell.tui.style.ThemeResolver;
 
 @AutoConfiguration
 @ConditionalOnClass(TerminalUI.class)

spring-shell-autoconfigure/src/main/java/org/springframework/shell/boot/ThemingAutoConfiguration.java

@@ -20,12 +20,12 @@
 import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
 import org.springframework.boot.context.properties.EnableConfigurationProperties;
 import org.springframework.context.annotation.Bean;
-import org.springframework.shell.style.TemplateExecutor;
-import org.springframework.shell.style.Theme;
-import org.springframework.shell.style.ThemeActive;
-import org.springframework.shell.style.ThemeRegistry;
-import org.springframework.shell.style.ThemeResolver;
-import org.springframework.shell.style.ThemeSettings;
+import org.springframework.shell.tui.style.TemplateExecutor;
+import org.springframework.shell.tui.style.Theme;
+import org.springframework.shell.tui.style.ThemeActive;
+import org.springframework.shell.tui.style.ThemeRegistry;
+import org.springframework.shell.tui.style.ThemeResolver;
+import org.springframework.shell.tui.style.ThemeSettings;
 import org.springframework.util.StringUtils;
 
 @AutoConfiguration

spring-shell-autoconfigure/src/test/java/org/springframework/shell/boot/TerminalUIAutoConfigurationTests.java

@@ -25,12 +25,12 @@
 import org.springframework.boot.test.context.runner.ApplicationContextRunner;
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
-import org.springframework.shell.component.view.TerminalUI;
-import org.springframework.shell.component.view.TerminalUIBuilder;
-import org.springframework.shell.component.view.TerminalUICustomizer;
-import org.springframework.shell.style.ThemeActive;
-import org.springframework.shell.style.ThemeRegistry;
-import org.springframework.shell.style.ThemeResolver;
+import org.springframework.shell.tui.component.view.TerminalUI;
+import org.springframework.shell.tui.component.view.TerminalUIBuilder;
+import org.springframework.shell.tui.component.view.TerminalUICustomizer;
+import org.springframework.shell.tui.style.ThemeActive;
+import org.springframework.shell.tui.style.ThemeRegistry;
+import org.springframework.shell.tui.style.ThemeResolver;
 import org.springframework.test.util.ReflectionTestUtils;
 
 import static org.assertj.core.api.Assertions.assertThat;

spring-shell-autoconfigure/src/test/java/org/springframework/shell/boot/ThemingAutoConfigurationTests.java

@@ -25,14 +25,14 @@
 import org.springframework.boot.test.context.runner.ApplicationContextRunner;
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
-import org.springframework.shell.style.FigureSettings;
-import org.springframework.shell.style.SpinnerSettings;
-import org.springframework.shell.style.StyleSettings;
-import org.springframework.shell.style.TemplateExecutor;
-import org.springframework.shell.style.Theme;
-import org.springframework.shell.style.ThemeRegistry;
-import org.springframework.shell.style.ThemeResolver;
-import org.springframework.shell.style.ThemeSettings;
+import org.springframework.shell.tui.style.FigureSettings;
+import org.springframework.shell.tui.style.SpinnerSettings;
+import org.springframework.shell.tui.style.StyleSettings;
+import org.springframework.shell.tui.style.TemplateExecutor;
+import org.springframework.shell.tui.style.Theme;
+import org.springframework.shell.tui.style.ThemeRegistry;
+import org.springframework.shell.tui.style.ThemeResolver;
+import org.springframework.shell.tui.style.ThemeSettings;
 import org.springframework.util.StringUtils;
 
 import static org.assertj.core.api.Assertions.assertThat;

spring-shell-core/pom.xml

@@ -31,17 +31,11 @@
 	</scm>
 
 	<dependencies>
-		<!-- TODO move TerminalSizeAware to core and remove this dependency -->
 		<dependency>
 			<groupId>org.springframework.shell</groupId>
 			<artifactId>spring-shell-table</artifactId>
 			<version>${project.parent.version}</version>
 		</dependency>
-		<dependency>
-			<groupId>io.projectreactor</groupId>
-			<artifactId>reactor-core</artifactId>
-			<version>${reactor.version}</version>
-		</dependency>
 		<dependency>
 			<groupId>org.springframework</groupId>
 			<artifactId>spring-context</artifactId>
@@ -57,16 +51,6 @@
 			<artifactId>jline</artifactId>
 			<version>${jline.version}</version>
 		</dependency>
-		<dependency>
-			<groupId>org.antlr</groupId>
-			<artifactId>ST4</artifactId>
-			<version>${antlr-st4.version}</version>
-		</dependency>
-		<dependency>
-			<groupId>commons-io</groupId>
-			<artifactId>commons-io</artifactId>
-			<version>${commons-io.version}</version>
-		</dependency>
 		<dependency>
 			<groupId>org.slf4j</groupId>
 			<artifactId>slf4j-api</artifactId>
@@ -79,6 +63,12 @@
 		</dependency>
 
 		<!-- Test dependencies -->
+		<dependency>
+			<groupId>org.springframework.shell</groupId>
+			<artifactId>spring-shell-tui</artifactId>
+			<version>${project.parent.version}</version>
+			<scope>test</scope>
+		</dependency>
 		<dependency>
 			<groupId>org.springframework.boot</groupId>
 			<artifactId>spring-boot-test</artifactId>
@@ -103,24 +93,6 @@
 			<version>${mockito.version}</version>
 			<scope>test</scope>
 		</dependency>
-		<dependency>
-			<groupId>com.google.jimfs</groupId>
-			<artifactId>jimfs</artifactId>
-			<version>${jimfs.version}</version>
-			<scope>test</scope>
-		</dependency>
-		<dependency>
-			<groupId>io.projectreactor</groupId>
-			<artifactId>reactor-test</artifactId>
-			<version>${reactor.version}</version>
-			<scope>test</scope>
-		</dependency>
-		<dependency>
-			<groupId>org.awaitility</groupId>
-			<artifactId>awaitility</artifactId>
-			<version>${awaitility.version}</version>
-			<scope>test</scope>
-		</dependency>
 		<dependency>
 			<groupId>org.hibernate.validator</groupId>
 			<artifactId>hibernate-validator</artifactId>

spring-shell-core/src/test/java/org/springframework/shell/jline/InteractiveShellRunnerTests.java

@@ -30,8 +30,6 @@
 import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
-import org.springframework.boot.ApplicationArguments;
-import org.springframework.boot.DefaultApplicationArguments;
 import org.springframework.shell.ExitRequest;
 
 import static org.assertj.core.api.Assertions.assertThat;
@@ -152,8 +150,4 @@ void testExitWithCtrlD() throws Exception {
         writeThread.join();
     }
 
-	private static ApplicationArguments ofApplicationArguments(String... args) {
-		return new DefaultApplicationArguments(args);
-	}
-
 }