Skip to content

Commit

Permalink
Merge pull request #849 from michalvavrik/feature/fix-external-app-pl…
Browse files Browse the repository at this point in the history
…atform-group-id

Fix Quarkus Platform group id used for external applications as default Quickstarts group id changed
  • Loading branch information
michalvavrik authored Aug 3, 2023
2 parents b33a10e + 9f747b8 commit 0034a40
Showing 1 changed file with 13 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@

import static io.quarkus.test.services.quarkus.GitRepositoryResourceBuilderUtils.cloneRepository;
import static io.quarkus.test.services.quarkus.GitRepositoryResourceBuilderUtils.mavenBuild;
import static io.quarkus.test.services.quarkus.model.QuarkusProperties.PLATFORM_GROUP_ID;
import static io.quarkus.test.utils.MavenUtils.withProperty;

import java.nio.file.Path;
import java.util.List;
Expand Down Expand Up @@ -43,13 +45,20 @@ protected Path getApplicationFolder() {

@Override
protected List<String> prepareCommand(List<String> systemProperties) {
// Dev mode
final List<String> commands;

if (model.isDevMode()) {
return MavenUtils.devModeMavenCommand(model.getContext(), systemProperties);
// Dev mode
commands = MavenUtils.devModeMavenCommand(model.getContext(), systemProperties);
} else {
// JVM or Native
commands = super.prepareCommand(systemProperties);
}

// JVM or Native
return super.prepareCommand(systemProperties);
// set quarkus.platform.group-id
commands.add(withProperty(PLATFORM_GROUP_ID.getPropertyKey(), PLATFORM_GROUP_ID.get()));

return List.copyOf(commands);
}

@Override
Expand Down

0 comments on commit 0034a40

Please sign in to comment.