Arc: CDI Decorators and interface methods with default implementations

[mederel]

Describe the bug

Found a tricky bug while using CDI Decorators based on an interface that has methods with default implementations.

Example:

interface Interface {
  default int getWithDefault() { return 3; }
}

@Decorator
class InterfaceDecorator implements Interface {
  @Inject
  @Delegate
  Interface delegate;

  public int getWithDefault() {
    return delegate.getWithDefault();
  }
}

@Dependent
class ConcreteImplementation implements Interface {
  @Override
  public int getWithDefault() {
    return 4;
  }
}

class User {
  @Inject
  Interface anInterface;

  public void call() {
    assert: anInterface.getWithDefault() == 4; // value of concrete implementation
  }
}

This code will end in an AssertionError cause the concerte implementation never gets called.

Expected behavior

Interface methods with default implementation are still override by the container to forwarding calls from one decorator to another till the concrete implementation of the type.

Actual behavior

Calls to interface methods with default implementation are not forwarded by the container from the first decorator to the second decorator or the actual implementation.

How to Reproduce?

Reproducer:
reproducer-decorator-default-impl.zip

Run mvn clean install after unzip.

Output of uname -a or ver

6.2.0 Ubuntu

Output of java -version

20.0.2

GraalVM version (if different from Java)

23.0.1

Quarkus version or git rev

3.3.1

Build tool (ie. output of mvnw --version or gradlew --version)

Apache Maven 3.9.4

Additional information

No response

[quarkus-bot]

/cc @Ladicek (arc), @manovotn (arc), @mkouba (arc)

[mederel]

The default methods are definitely removed after this contribution:

https://github.com/quarkusio/quarkus/pull/33841/files#diff-f05a30ce973915337c900f71476c739f710d3fcc1ab6d148bbb418755a2bd5a9

But when I try to remove the condition, obviously few tests start failing again:

[ERROR] Errors: 
[ERROR]   MultipleDecoratorsWithTypeVariablesTest.testConsumer » NullPointer
[ERROR]   MultipleDecoratorsWithTypeVariablesTest.testFunction » NullPointer
[ERROR]   MultipleDecoratorsWithTypeVariablesTest.testSupplier » NullPointer
[ERROR]   SlightlyMoreComplexDecoratorWithTypeVariableTest.testFunction » NullPointer

I don't think I am in a position to propose a fix.