Set Vaadin's launch-browser property automatically

Closes gh-1552
spring-io · Aug 16, 2024 · 86c98ff · 86c98ff
1 parent 4aa69b4
commit 86c98ff
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/...o/spring/start/site/extension/dependency/vaadin/VaadinProjectGenerationConfiguration.java b/...o/spring/start/site/extension/dependency/vaadin/VaadinProjectGenerationConfiguration.java
@@ -21,6 +21,7 @@
 import io.spring.initializr.generator.condition.ConditionalOnBuildSystem;
 import io.spring.initializr.generator.condition.ConditionalOnRequestedDependency;
 import io.spring.initializr.generator.project.ProjectDescription;
+import io.spring.initializr.generator.spring.properties.ApplicationPropertiesCustomizer;
 import io.spring.initializr.generator.spring.scm.git.GitIgnoreCustomizer;
 import io.spring.initializr.metadata.InitializrMetadata;
 
@@ -31,6 +32,7 @@
  * Configuration for generation of projects that depend on Vaadin.
  *
  * @author Stephane Nicoll
+ * @author Moritz Halbritter
  */
 @Configuration(proxyBeanMethods = false)
 @ConditionalOnRequestedDependency("vaadin")
@@ -54,4 +56,9 @@ GitIgnoreCustomizer vaadinGitIgnoreCustomizer() {
 		return (gitignore) -> gitignore.getGeneral().add("node_modules");
 	}
 
+	@Bean
+	ApplicationPropertiesCustomizer launchBrowserApplicationPropertiesCustomizer() {
+		return (properties) -> properties.add("vaadin.launch-browser", true);
+	}
+
 }
diff --git a/...ing/start/site/extension/dependency/vaadin/VaadinProjectGenerationConfigurationTests.java b/...ing/start/site/extension/dependency/vaadin/VaadinProjectGenerationConfigurationTests.java
@@ -27,6 +27,7 @@
  * Tests for {@link VaadinProjectGenerationConfiguration}.
  *
  * @author Stephane Nicoll
+ * @author Moritz Halbritter
  */
 class VaadinProjectGenerationConfigurationTests extends AbstractExtensionTests {
 
@@ -110,6 +111,18 @@ void gitIgnoreWithoutVaadinDoesNotIgnoreNodeModules() {
 			.doesNotContain("node_modules");
 	}
 
+	@Test
+	void shouldAddLaunchBrowserProperty() {
+		assertThat(generateProject(createProjectRequest("vaadin"))).textFile("src/main/resources/application.properties")
+				.contains("vaadin.launch-browser=true");
+	}
+
+	@Test
+	void shouldNotAddLaunchBrowserPropertyIfVaadinIsNotSelected() {
+		assertThat(generateProject(createProjectRequest("data-jpa"))).textFile("src/main/resources/application.properties")
+				.doesNotContain("vaadin.launch-browser=true");
+	}
+
 	@Override
 	protected ProjectRequest createProjectRequest(String... dependencies) {
 		ProjectRequest request = super.createProjectRequest(dependencies);