@UnlessBuildProfile not working as expected

[rareddy]

Describe the bug

When @UnlessBuildProfile used on @ApplicationScoped bean, there are a couple of issues I am seeing.

1. When you build a profile that does not match to the annotation, even in that profile the bean is available.
2. When the bean is available it does not do the @Inject operations correctly.

for example, I have @UnlessBuildProfile("prod"), and when building the prod I still see the bean available. I could verify that because in my case it is rest controller.

Expected behavior

The bean be not available

To Reproduce

@ApplicationScoped
@UnlessBuildProfile("prod")
@Path("/")
public class MockControlPlane {

  public String hello(){
     return "hello";
  } 
}

build with -Dquarkus.profile=prod

[FroMage]

CC @mkouba

[FroMage]

Actually, I wonder if this is not a combination of RESTEasy scanning and taking over discovery because CDI would not claim that bean?

[mkouba]

Actually, I wonder if this is not a combination of RESTEasy scanning and taking over discovery because CDI would not claim that bean?

Yes, I think so. The bean is not registered but there's a fallback used in the QuarkusInjectorFactory that just creates the resource instance via new MockControlPlane() and that's why CDI injection does not work there.

[mkouba]

TBH I'm not sure what the correct behavior would be but I think that the fallback is there since the beginning (and an "invisible" DEBUG message is logged).

[mkouba]

CC @geoand

[geoand]

Yeah, I think what you guys mention is what is going on

[FroMage]

So perhaps RESTEasy Classic and Reactive should both special-case @UnlessBuildProfile?

[geoand]

That will likely be needed

[geoand]

This actually should be working as we have merged: #15215

So I'll need to have another look

[rareddy]

I am using 1.11.1.Final version of Quarkus if the above is after that I would need validate with a newer release

[geoand]

OK, that explains it then, as I just tried with the latest and it works.

So you need to use 1.13.0.Final or newer to get this feature