Skip to content

Commit 4d55256

Browse files
committed
Fixed jacoco init method injection and causing test failure.
1 parent 3a78e08 commit 4d55256

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

src/main/java/io/github/majusko/pulsar/collector/ConsumerCollector.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import io.github.majusko.pulsar.annotation.PulsarConsumer;
44
import org.springframework.beans.factory.config.BeanPostProcessor;
5+
import org.springframework.context.annotation.Configuration;
56
import org.springframework.stereotype.Component;
67

78
import java.util.Arrays;
@@ -10,7 +11,7 @@
1011
import java.util.concurrent.ConcurrentHashMap;
1112
import java.util.stream.Collectors;
1213

13-
@Component
14+
@Configuration
1415
public class ConsumerCollector implements BeanPostProcessor {
1516

1617
private Map<String, ConsumerHolder> consumers = new ConcurrentHashMap<>();
@@ -25,6 +26,8 @@ public Object postProcessBeforeInitialization(Object bean, String beanName) {
2526
method -> beanClass.getName() + "#" + method.getName(),
2627
method -> new ConsumerHolder(method.getAnnotation(PulsarConsumer.class), method, bean))));
2728

29+
System.out.println(consumers.toString());
30+
2831
return bean;
2932
}
3033

src/test/java/io/github/majusko/pulsar/PulsarJavaSpringBootStarterApplicationTests.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
import java.util.Set;
2828

2929
@SpringBootTest
30-
@Import(TestProducerConfiguration.class)
30+
@Import({TestProducerConfiguration.class, TestConsumerConfiguration.class})
3131
class PulsarJavaSpringBootStarterApplicationTests {
3232

3333
@Autowired
@@ -60,10 +60,10 @@ void testConsumerRegistration1() throws Exception {
6060
}
6161

6262
@Test
63-
void testConsumerRegistration2() throws Exception {
63+
void testConsumerRegistration2() {
6464
final Class<TestConsumerConfiguration> clazz = TestConsumerConfiguration.class;
65-
final String descriptor = clazz.getName() + "#" + clazz.getDeclaredMethods()[0].getName();
66-
final ConsumerHolder consumerHolder = consumerCollector.getConsumer(descriptor).orElseThrow(Exception::new);
65+
final String descriptor = clazz.getName() + "#" + clazz.getMethods()[0].getName();
66+
final ConsumerHolder consumerHolder = consumerCollector.getConsumer(descriptor).orElse(null);
6767

6868
Assertions.assertNotNull(consumerHolder);
6969
Assertions.assertEquals("mock-topic", consumerHolder.getAnnotation().topic());

0 commit comments

Comments
 (0)