Allow quarkus.application.name to be overridden for Spring Cloud Config Server

[thibaultmeyer]

• Resolves: Allow quarkus.application.name to be overridden for Spring Cloud Config Server #41011

[thibaultmeyer]

@geoand

[geoand]

Very nice!

Any chance we can get a test?

[thibaultmeyer]

yes, I'll try to add some unit tests

[geoand]

I was thinking of something similar to SpringCloudConfigClientGatewayTest

[thibaultmeyer]

I've added new unit tests, but I'm not 100% sure it's OK.

[geoand]

Please see output above for the command that needs to be run in order to properly format the code

[thibaultmeyer]

Please see output above for the command that needs to be run in order to properly format the code

done

[thibaultmeyer]

Import are now sorted.

mvn -f . net.revelc.code:impsort-maven-plugin:1.10.0:sort

[thibaultmeyer]

I repushed an import modification following the structure of this file: LambdaHttpHandlerTest.java.

EDIT : I definitevely dont understand how work import structure rules in Quarkus. And mvn command fail on my computer :

Error:  The build could not read 1 project -> [Help 1]
Error:    
Error:    The project io.quarkus:quarkus-spring-cloud-config-client:999-SNAPSHOT (/home/runner/work/quarkus/quarkus/extensions/spring-cloud-config-client/runtime/pom.xml) has 12 errors
Error:      Non-resolvable import POM: The following artifacts could not be resolved: io.quarkus:quarkus-bom:pom:999-SNAPSHOT (absent): Could not find artifact io.quarkus:quarkus-bom:pom:999-SNAPSHOT @ io.quarkus:quarkus-build-parent:999-SNAPSHOT, 

@geoand can you please help me?

EDIT 2 : seems OK, to many new line between import and class definition ;)

[quarkus-bot]

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 92aadc7.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

[geoand]

Great!