Make RESTEasy Reactive the default codestart

Also only add the default codestart if no extensions have been selected. Quarkus is more than a REST framework and this behavior is annoying for anything that is not REST and does not have a codestart.
quarkusio · Mar 18, 2022 · 2da75de · 2da75de
1 parent 70432fd
commit 2da75de
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 7 deletions.
diff --git a/devtools/cli/src/test/java/io/quarkus/cli/CliProjectMavenTest.java b/devtools/cli/src/test/java/io/quarkus/cli/CliProjectMavenTest.java
@@ -51,8 +51,8 @@ public void testCreateAppDefaults() throws Exception {
         String pomContent = validateBasicIdentifiers(CreateProjectHelper.DEFAULT_GROUP_ID,
                 CreateProjectHelper.DEFAULT_ARTIFACT_ID,
                 CreateProjectHelper.DEFAULT_VERSION);
-        Assertions.assertTrue(pomContent.contains("<artifactId>quarkus-resteasy</artifactId>"),
-                "pom.xml should contain quarkus-resteasy:\n" + pomContent);
+        Assertions.assertTrue(pomContent.contains("<artifactId>quarkus-resteasy-reactive</artifactId>"),
+                "pom.xml should contain quarkus-resteasy-reactive:\n" + pomContent);
 
         CliDriver.valdiateGeneratedSourcePackage(project, "org/acme");
 

diff --git a/...-common/src/main/java/io/quarkus/devtools/codestarts/quarkus/QuarkusCodestartCatalog.java b/...-common/src/main/java/io/quarkus/devtools/codestarts/quarkus/QuarkusCodestartCatalog.java
@@ -66,7 +66,7 @@ public enum Tooling implements DataKey {
 
     public enum ExtensionCodestart implements DataKey {
         RESTEASY,
-        RESTEAST_REACTIVE,
+        RESTEASY_REACTIVE,
         SPRING_WEB
     }
 
@@ -127,14 +127,15 @@ protected Collection<Codestart> select(QuarkusCodestartProjectInput projectInput
                 .filter(c -> !isExample(c) || projectInput.getExample() == null || c.matches(projectInput.getExample()))
                 .collect(Collectors.toCollection(ArrayList::new));
 
-        // include default codestarts if no code selected
+        // include default codestarts if no code selected and no extensions have been selected
         if (projectInput.getAppContent().contains(CODE)
+                && projectInput.getExtensions().isEmpty()
                 && projectCodestarts.stream()
                         .noneMatch(c -> c.getType() == CodestartType.CODE && !c.getSpec().isPreselected())) {
             final Codestart defaultCodestart = codestarts.stream()
-                    .filter(c -> c.matches(ExtensionCodestart.RESTEASY.key()))
+                    .filter(c -> c.matches(ExtensionCodestart.RESTEASY_REACTIVE.key()))
                     .findFirst().orElseThrow(() -> new CodestartStructureException(
-                            ExtensionCodestart.RESTEASY.key() + " codestart not found"));
+                            ExtensionCodestart.RESTEASY_REACTIVE.key() + " codestart not found"));
             final String languageName = findLanguageName(projectCodestarts);
             if (defaultCodestart.implementsLanguage(languageName)) {
                 projectCodestarts.add(defaultCodestart);

diff --git a/integration-tests/maven/src/test/java/io/quarkus/maven/it/CreateProjectMojoIT.java b/integration-tests/maven/src/test/java/io/quarkus/maven/it/CreateProjectMojoIT.java
@@ -93,7 +93,7 @@ public void testProjectGenerationFromScratch() throws MavenInvocationException,
                         && d.getType().equals("pom"))).isTrue();
 
         assertThat(
-                model.getDependencies().stream().anyMatch(d -> d.getArtifactId().equalsIgnoreCase("quarkus-resteasy")
+                model.getDependencies().stream().anyMatch(d -> d.getArtifactId().equalsIgnoreCase("quarkus-resteasy-reactive")
                         && d.getVersion() == null)).isTrue();
 
         assertThat(model.getProfiles()).hasSize(1);