Skip to content

Commit 0ed20a1

Browse files
committed
#23 - when using Lombok @slf4j getting Argument passed to when() is not a mock
1 parent 29f1624 commit 0ed20a1

File tree

3 files changed

+26
-7
lines changed

3 files changed

+26
-7
lines changed

pom.xml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,10 +40,17 @@
4040

4141
<!-- test dependencies -->
4242

43+
<dependency>
44+
<groupId>org.projectlombok</groupId>
45+
<artifactId>lombok</artifactId>
46+
<version>1.18.6</version>
47+
<scope>test</scope>
48+
</dependency>
49+
4350
<dependency>
4451
<groupId>io.dinject</groupId>
4552
<artifactId>dinject-generator</artifactId>
46-
<version>1.4</version>
53+
<version>1.6</version>
4754
<scope>test</scope>
4855
</dependency>
4956

src/main/java/io/dinject/core/DBuilderExtn.java

Lines changed: 15 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -43,17 +43,26 @@ public boolean isAddBeanFor(Class<?> addForType, Class<?> injectTarget) {
4343
@SuppressWarnings("unchecked")
4444
@Override
4545
public Object enrich(Object bean, Class<?>[] types) {
46-
EnrichBean enrich = enrichMap.get(typeOf(bean, types));
46+
EnrichBean enrich = getEnrich(bean, types);
4747
return enrich != null ? enrich.enrich(bean) : bean;
4848
}
4949

5050
/**
51-
* Return the type to lookup for enrichment.
51+
* Search for EnrichBean on the bean or any of the interface types.
5252
*/
53-
private Class<?> typeOf(Object bean, Class<?>... types) {
54-
if (types.length > 0) {
55-
return types[0];
53+
private EnrichBean getEnrich(Object bean, Class<?>[] types) {
54+
55+
EnrichBean enrich = enrichMap.get(bean.getClass());
56+
if (enrich != null) {
57+
return enrich;
58+
}
59+
for (Class<?> type : types) {
60+
enrich = enrichMap.get(type);
61+
if (enrich != null) {
62+
return enrich;
63+
}
5664
}
57-
return bean.getClass();
65+
return null;
5866
}
67+
5968
}

src/test/java/org/example/coffee/grind/Grinder.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,12 @@
11
package org.example.coffee.grind;
22

3+
import lombok.extern.slf4j.Slf4j;
4+
35
import javax.annotation.PostConstruct;
46
import javax.inject.Inject;
57
import javax.inject.Singleton;
68

9+
@Slf4j
710
@Singleton
811
public class Grinder {
912

0 commit comments

Comments
 (0)