Add test for cases with @GrpcClientBean and @GrpcClient usage. Fix exception message

Author: a-simeshin

grpc-client-spring-boot-autoconfigure/src/main/java/net/devh/boot/grpc/client/inject/GrpcClientBeanPostProcessor.java

@@ -117,8 +117,9 @@ public Object postProcessBeforeInitialization(final Object bean, final String be
                         beanFactory.registerSingleton(beanNameToCreate, beanValue);
                     } catch (final Exception e) {
                         throw new BeanCreationException(
-                                "Could not create and register grpc client bean: {} from class {}", beanNameToCreate,
-                                clazz.getSimpleName(), e);
+                                "Could not create and register grpc client bean " + beanNameToCreate + " from class " +
+                                        clazz.getSimpleName(),
+                                e);
                     }
                 }
             }

tests/src/test/java/net/devh/boot/grpc/test/inject/GrpcClientBeanInjectionNegativeTest.java

@@ -0,0 +1,95 @@
+/*
+ * Copyright (c) 2016-2021 Michael Zhang <[email protected]>
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated
+ * documentation files (the "Software"), to deal in the Software without restriction, including without limitation the
+ * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to
+ * permit persons to whom the Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE
+ * WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
+ * COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
+ * OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+ */
+
+package net.devh.boot.grpc.test.inject;
+
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+
+import org.junit.jupiter.api.Test;
+import org.springframework.beans.factory.BeanCreationException;
+import org.springframework.boot.SpringApplication;
+import org.springframework.boot.test.context.TestConfiguration;
+import org.springframework.context.annotation.Bean;
+import org.springframework.test.annotation.DirtiesContext;
+
+import lombok.extern.slf4j.Slf4j;
+import net.devh.boot.grpc.client.inject.GrpcClient;
+import net.devh.boot.grpc.client.inject.GrpcClientBean;
+import net.devh.boot.grpc.client.inject.GrpcClientBeans;
+import net.devh.boot.grpc.test.config.BaseAutoConfiguration;
+import net.devh.boot.grpc.test.config.InProcessConfiguration;
+import net.devh.boot.grpc.test.config.ServiceConfiguration;
+import net.devh.boot.grpc.test.proto.TestServiceGrpc;
+
+/**
+ * Test case should cover probable negative situations with @GrpcBeanClient and @GrpcClient usage
+ */
+@Slf4j
+@DirtiesContext
+public class GrpcClientBeanInjectionNegativeTest {
+
+    @Test
+    void twoSameGrpcClientBeansTest() {
+        final SpringApplication app = new SpringApplication(
+                TwoSameGrpcClientBeans.class,
+                InProcessConfiguration.class,
+                ServiceConfiguration.class,
+                BaseAutoConfiguration.class);
+        assertThrows(BeanCreationException.class, app::run);
+    }
+
+    @Test
+    void grpcClientBeanAndCustomGrpcClientFieldTest() {
+        final SpringApplication app = new SpringApplication(
+                GrpcClientBeanWithCustomGrpcClientFieldConfig.class,
+                InProcessConfiguration.class,
+                ServiceConfiguration.class,
+                BaseAutoConfiguration.class);
+        assertDoesNotThrow(() -> SpringApplication.exit(app.run()));
+    }
+
+    @TestConfiguration
+    @GrpcClientBeans(value = {
+            @GrpcClientBean(
+                    clazz = TestServiceGrpc.TestServiceBlockingStub.class,
+                    beanName = "blockingStub",
+                    client = @GrpcClient("test")),
+            @GrpcClientBean(
+                    clazz = TestServiceGrpc.TestServiceBlockingStub.class,
+                    beanName = "blockingStub",
+                    client = @GrpcClient("test")),
+    })
+    public static class TwoSameGrpcClientBeans {
+    }
+
+    @TestConfiguration
+    @GrpcClientBean(
+            clazz = TestServiceGrpc.TestServiceBlockingStub.class,
+            beanName = "blockingStub",
+            client = @GrpcClient("testGrpcBeanClient"))
+    public static class GrpcClientBeanWithCustomGrpcClientFieldConfig {
+
+        @GrpcClient("testGrpcBeanClient")
+        TestServiceGrpc.TestServiceBlockingStub stub;
+
+        @Bean
+        public TestServiceGrpc.TestServiceBlockingStub blockingStub() {
+            return stub;
+        }
+    }
+}