Skip to content

Commit

Permalink
Fix Quarkus Platform group id for external applications
Browse files Browse the repository at this point in the history
  • Loading branch information
michalvavrik committed Aug 3, 2023
1 parent b33a10e commit 9f747b8
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 9f747b8

Please sign in to comment.