This is an automated email from the ASF dual-hosted git repository. acosentino pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git
commit afa4f1ac22028891aeb855e02a4cb7273ec710cc Author: Andrea Cosentino <anco...@gmail.com> AuthorDate: Thu Aug 1 16:59:33 2019 +0200 CAMEL-13801 - camel3 - Use @BindToRegistry wherever possible, Camel-Swagger-Java --- .../swagger/RestSwaggerReaderApiDocsOverrideTest.java | 13 ++++++------- .../camel/swagger/RestSwaggerReaderApiDocsTest.java | 11 +++++------ .../camel/swagger/RestSwaggerReaderDayOfWeekTest.java | 11 +++++------ .../RestSwaggerReaderEnableVendorExtensionTest.java | 11 +++++------ .../swagger/RestSwaggerReaderModelApiSecurityTest.java | 11 +++++------ .../swagger/RestSwaggerReaderModelBookOrderTest.java | 11 +++++------ .../apache/camel/swagger/RestSwaggerReaderModelTest.java | 11 +++++------ .../swagger/RestSwaggerReaderPropertyPlaceholderTest.java | 15 +++++++-------- .../org/apache/camel/swagger/RestSwaggerReaderTest.java | 11 +++++------ .../apache/camel/swagger/producer/RestSwaggerGetTest.java | 9 +++------ .../swagger/producer/RestSwaggerGetUriParamTest.java | 9 +++------ .../component/syslog/NettyRfc5425LongMessageTest.java | 1 - .../apache/camel/component/syslog/NettyRfc5425Test.java | 1 - 13 files changed, 54 insertions(+), 71 deletions(-) diff --git a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderApiDocsOverrideTest.java b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderApiDocsOverrideTest.java index 3609574..969f7fb 100644 --- a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderApiDocsOverrideTest.java +++ b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderApiDocsOverrideTest.java @@ -21,21 +21,20 @@ import com.fasterxml.jackson.databind.ObjectMapper; import com.fasterxml.jackson.databind.SerializationFeature; import io.swagger.jaxrs.config.BeanConfig; import io.swagger.models.Swagger; + +import org.apache.camel.BindToRegistry; import org.apache.camel.builder.RouteBuilder; import org.apache.camel.impl.JndiRegistry; import org.apache.camel.impl.engine.DefaultClassResolver; import org.apache.camel.model.rest.RestParamType; +import org.apache.camel.swagger.producer.DummyRestProducerFactory; import org.apache.camel.test.junit4.CamelTestSupport; import org.junit.Test; public class RestSwaggerReaderApiDocsOverrideTest extends CamelTestSupport { - - @Override - protected JndiRegistry createRegistry() throws Exception { - JndiRegistry jndi = super.createRegistry(); - jndi.bind("dummy-rest", new DummyRestConsumerFactory()); - return jndi; - } + + @BindToRegistry("dummy-rest") + private DummyRestConsumerFactory factory = new DummyRestConsumerFactory(); @Override protected RouteBuilder createRouteBuilder() throws Exception { diff --git a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderApiDocsTest.java b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderApiDocsTest.java index 31c44c2..73b4b1c 100644 --- a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderApiDocsTest.java +++ b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderApiDocsTest.java @@ -21,21 +21,20 @@ import com.fasterxml.jackson.databind.ObjectMapper; import com.fasterxml.jackson.databind.SerializationFeature; import io.swagger.jaxrs.config.BeanConfig; import io.swagger.models.Swagger; + +import org.apache.camel.BindToRegistry; import org.apache.camel.builder.RouteBuilder; import org.apache.camel.impl.JndiRegistry; import org.apache.camel.impl.engine.DefaultClassResolver; import org.apache.camel.model.rest.RestParamType; +import org.apache.camel.swagger.producer.DummyRestProducerFactory; import org.apache.camel.test.junit4.CamelTestSupport; import org.junit.Test; public class RestSwaggerReaderApiDocsTest extends CamelTestSupport { - @Override - protected JndiRegistry createRegistry() throws Exception { - JndiRegistry jndi = super.createRegistry(); - jndi.bind("dummy-rest", new DummyRestConsumerFactory()); - return jndi; - } + @BindToRegistry("dummy-rest") + private DummyRestConsumerFactory factory = new DummyRestConsumerFactory(); @Override protected RouteBuilder createRouteBuilder() throws Exception { diff --git a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderDayOfWeekTest.java b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderDayOfWeekTest.java index a485bd2..b6520ab 100644 --- a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderDayOfWeekTest.java +++ b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderDayOfWeekTest.java @@ -21,21 +21,20 @@ import com.fasterxml.jackson.databind.ObjectMapper; import com.fasterxml.jackson.databind.SerializationFeature; import io.swagger.jaxrs.config.BeanConfig; import io.swagger.models.Swagger; + +import org.apache.camel.BindToRegistry; import org.apache.camel.builder.RouteBuilder; import org.apache.camel.impl.JndiRegistry; import org.apache.camel.impl.engine.DefaultClassResolver; import org.apache.camel.model.rest.RestParamType; +import org.apache.camel.swagger.producer.DummyRestProducerFactory; import org.apache.camel.test.junit4.CamelTestSupport; import org.junit.Test; public class RestSwaggerReaderDayOfWeekTest extends CamelTestSupport { - @Override - protected JndiRegistry createRegistry() throws Exception { - JndiRegistry jndi = super.createRegistry(); - jndi.bind("dummy-rest", new DummyRestConsumerFactory()); - return jndi; - } + @BindToRegistry("dummy-rest") + private DummyRestConsumerFactory factory = new DummyRestConsumerFactory(); @Override protected RouteBuilder createRouteBuilder() throws Exception { diff --git a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderEnableVendorExtensionTest.java b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderEnableVendorExtensionTest.java index 2feb6de..3e55813 100644 --- a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderEnableVendorExtensionTest.java +++ b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderEnableVendorExtensionTest.java @@ -21,21 +21,20 @@ import com.fasterxml.jackson.databind.ObjectMapper; import com.fasterxml.jackson.databind.SerializationFeature; import io.swagger.jaxrs.config.BeanConfig; import io.swagger.models.Swagger; + +import org.apache.camel.BindToRegistry; import org.apache.camel.builder.RouteBuilder; import org.apache.camel.impl.JndiRegistry; import org.apache.camel.impl.engine.DefaultClassResolver; import org.apache.camel.model.rest.RestParamType; +import org.apache.camel.swagger.producer.DummyRestProducerFactory; import org.apache.camel.test.junit4.CamelTestSupport; import org.junit.Test; public class RestSwaggerReaderEnableVendorExtensionTest extends CamelTestSupport { - @Override - protected JndiRegistry createRegistry() throws Exception { - JndiRegistry jndi = super.createRegistry(); - jndi.bind("dummy-rest", new DummyRestConsumerFactory()); - return jndi; - } + @BindToRegistry("dummy-rest") + private DummyRestConsumerFactory factory = new DummyRestConsumerFactory(); @Override protected RouteBuilder createRouteBuilder() throws Exception { diff --git a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderModelApiSecurityTest.java b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderModelApiSecurityTest.java index 4aeeb87..7fd9e18 100644 --- a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderModelApiSecurityTest.java +++ b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderModelApiSecurityTest.java @@ -21,21 +21,20 @@ import com.fasterxml.jackson.databind.ObjectMapper; import com.fasterxml.jackson.databind.SerializationFeature; import io.swagger.jaxrs.config.BeanConfig; import io.swagger.models.Swagger; + +import org.apache.camel.BindToRegistry; import org.apache.camel.builder.RouteBuilder; import org.apache.camel.impl.JndiRegistry; import org.apache.camel.impl.engine.DefaultClassResolver; import org.apache.camel.model.rest.RestParamType; +import org.apache.camel.swagger.producer.DummyRestProducerFactory; import org.apache.camel.test.junit4.CamelTestSupport; import org.junit.Test; public class RestSwaggerReaderModelApiSecurityTest extends CamelTestSupport { - @Override - protected JndiRegistry createRegistry() throws Exception { - JndiRegistry jndi = super.createRegistry(); - jndi.bind("dummy-rest", new DummyRestConsumerFactory()); - return jndi; - } + @BindToRegistry("dummy-rest") + private DummyRestConsumerFactory factory = new DummyRestConsumerFactory(); @Override protected RouteBuilder createRouteBuilder() throws Exception { diff --git a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderModelBookOrderTest.java b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderModelBookOrderTest.java index e8959ab..a0e22d2 100644 --- a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderModelBookOrderTest.java +++ b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderModelBookOrderTest.java @@ -21,21 +21,20 @@ import com.fasterxml.jackson.databind.ObjectMapper; import com.fasterxml.jackson.databind.SerializationFeature; import io.swagger.jaxrs.config.BeanConfig; import io.swagger.models.Swagger; + +import org.apache.camel.BindToRegistry; import org.apache.camel.builder.RouteBuilder; import org.apache.camel.impl.JndiRegistry; import org.apache.camel.impl.engine.DefaultClassResolver; import org.apache.camel.model.rest.RestParamType; +import org.apache.camel.swagger.producer.DummyRestProducerFactory; import org.apache.camel.test.junit4.CamelTestSupport; import org.junit.Test; public class RestSwaggerReaderModelBookOrderTest extends CamelTestSupport { - @Override - protected JndiRegistry createRegistry() throws Exception { - JndiRegistry jndi = super.createRegistry(); - jndi.bind("dummy-rest", new DummyRestConsumerFactory()); - return jndi; - } + @BindToRegistry("dummy-rest") + private DummyRestConsumerFactory factory = new DummyRestConsumerFactory(); @Override protected RouteBuilder createRouteBuilder() throws Exception { diff --git a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderModelTest.java b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderModelTest.java index 66e477c..8c71821 100644 --- a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderModelTest.java +++ b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderModelTest.java @@ -21,21 +21,20 @@ import com.fasterxml.jackson.databind.ObjectMapper; import com.fasterxml.jackson.databind.SerializationFeature; import io.swagger.jaxrs.config.BeanConfig; import io.swagger.models.Swagger; + +import org.apache.camel.BindToRegistry; import org.apache.camel.builder.RouteBuilder; import org.apache.camel.impl.JndiRegistry; import org.apache.camel.impl.engine.DefaultClassResolver; import org.apache.camel.model.rest.RestParamType; +import org.apache.camel.swagger.producer.DummyRestProducerFactory; import org.apache.camel.test.junit4.CamelTestSupport; import org.junit.Test; public class RestSwaggerReaderModelTest extends CamelTestSupport { - @Override - protected JndiRegistry createRegistry() throws Exception { - JndiRegistry jndi = super.createRegistry(); - jndi.bind("dummy-rest", new DummyRestConsumerFactory()); - return jndi; - } + @BindToRegistry("dummy-rest") + private DummyRestConsumerFactory factory = new DummyRestConsumerFactory(); @Override protected RouteBuilder createRouteBuilder() throws Exception { diff --git a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderPropertyPlaceholderTest.java b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderPropertyPlaceholderTest.java index fba836e..73de294 100644 --- a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderPropertyPlaceholderTest.java +++ b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderPropertyPlaceholderTest.java @@ -24,31 +24,30 @@ import com.fasterxml.jackson.databind.ObjectMapper; import com.fasterxml.jackson.databind.SerializationFeature; import io.swagger.jaxrs.config.BeanConfig; import io.swagger.models.Swagger; + +import org.apache.camel.BindToRegistry; import org.apache.camel.builder.RouteBuilder; import org.apache.camel.impl.JndiRegistry; import org.apache.camel.impl.engine.DefaultClassResolver; import org.apache.camel.model.rest.RestDefinition; import org.apache.camel.model.rest.RestParamType; +import org.apache.camel.swagger.producer.DummyRestProducerFactory; import org.apache.camel.test.junit4.CamelTestSupport; import org.junit.Ignore; import org.junit.Test; @Ignore("Does not run well on CI due test uses JMX mbeans") public class RestSwaggerReaderPropertyPlaceholderTest extends CamelTestSupport { - + + @BindToRegistry("dummy-rest") + private DummyRestProducerFactory factory = new DummyRestProducerFactory(); + @Override protected boolean useJmx() { return true; } @Override - protected JndiRegistry createRegistry() throws Exception { - JndiRegistry jndi = super.createRegistry(); - jndi.bind("dummy-rest", new DummyRestConsumerFactory()); - return jndi; - } - - @Override protected Properties useOverridePropertiesWithPropertiesComponent() { Properties prop = new Properties(); prop.put("foo", "hello"); diff --git a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderTest.java b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderTest.java index a2b8ee7..9407c92 100644 --- a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderTest.java +++ b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/RestSwaggerReaderTest.java @@ -21,21 +21,20 @@ import com.fasterxml.jackson.databind.ObjectMapper; import com.fasterxml.jackson.databind.SerializationFeature; import io.swagger.jaxrs.config.BeanConfig; import io.swagger.models.Swagger; + +import org.apache.camel.BindToRegistry; import org.apache.camel.builder.RouteBuilder; import org.apache.camel.impl.JndiRegistry; import org.apache.camel.impl.engine.DefaultClassResolver; import org.apache.camel.model.rest.RestParamType; +import org.apache.camel.swagger.producer.DummyRestProducerFactory; import org.apache.camel.test.junit4.CamelTestSupport; import org.junit.Test; public class RestSwaggerReaderTest extends CamelTestSupport { - @Override - protected JndiRegistry createRegistry() throws Exception { - JndiRegistry jndi = super.createRegistry(); - jndi.bind("dummy-rest", new DummyRestConsumerFactory()); - return jndi; - } + @BindToRegistry("dummy-rest") + private DummyRestConsumerFactory factory = new DummyRestConsumerFactory(); @Override protected RouteBuilder createRouteBuilder() throws Exception { diff --git a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/producer/RestSwaggerGetTest.java b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/producer/RestSwaggerGetTest.java index 1c53029..3fcc68f 100644 --- a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/producer/RestSwaggerGetTest.java +++ b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/producer/RestSwaggerGetTest.java @@ -16,6 +16,7 @@ */ package org.apache.camel.swagger.producer; +import org.apache.camel.BindToRegistry; import org.apache.camel.RoutesBuilder; import org.apache.camel.builder.RouteBuilder; import org.apache.camel.impl.JndiRegistry; @@ -24,12 +25,8 @@ import org.junit.Test; public class RestSwaggerGetTest extends CamelTestSupport { - @Override - protected JndiRegistry createRegistry() throws Exception { - JndiRegistry jndi = super.createRegistry(); - jndi.bind("dummy", new DummyRestProducerFactory()); - return jndi; - } + @BindToRegistry("dummy") + private DummyRestProducerFactory factory = new DummyRestProducerFactory(); @Test public void testSwaggerGet() throws Exception { diff --git a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/producer/RestSwaggerGetUriParamTest.java b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/producer/RestSwaggerGetUriParamTest.java index 802adf3..837d27d 100644 --- a/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/producer/RestSwaggerGetUriParamTest.java +++ b/components/camel-swagger-java/src/test/java/org/apache/camel/swagger/producer/RestSwaggerGetUriParamTest.java @@ -16,6 +16,7 @@ */ package org.apache.camel.swagger.producer; +import org.apache.camel.BindToRegistry; import org.apache.camel.RoutesBuilder; import org.apache.camel.builder.RouteBuilder; import org.apache.camel.impl.JndiRegistry; @@ -24,12 +25,8 @@ import org.junit.Test; public class RestSwaggerGetUriParamTest extends CamelTestSupport { - @Override - protected JndiRegistry createRegistry() throws Exception { - JndiRegistry jndi = super.createRegistry(); - jndi.bind("dummy", new DummyRestProducerFactory()); - return jndi; - } + @BindToRegistry("dummy") + private DummyRestProducerFactory factory = new DummyRestProducerFactory(); @Test public void testSwaggerGet() throws Exception { diff --git a/components/camel-syslog/src/test/java/org/apache/camel/component/syslog/NettyRfc5425LongMessageTest.java b/components/camel-syslog/src/test/java/org/apache/camel/component/syslog/NettyRfc5425LongMessageTest.java index d93b28a..f9340d0 100644 --- a/components/camel-syslog/src/test/java/org/apache/camel/component/syslog/NettyRfc5425LongMessageTest.java +++ b/components/camel-syslog/src/test/java/org/apache/camel/component/syslog/NettyRfc5425LongMessageTest.java @@ -24,7 +24,6 @@ import org.apache.camel.builder.RouteBuilder; import org.apache.camel.component.mock.MockEndpoint; import org.apache.camel.component.syslog.netty.Rfc5425Encoder; import org.apache.camel.component.syslog.netty.Rfc5425FrameDecoder; -import org.apache.camel.impl.JndiRegistry; import org.apache.camel.spi.DataFormat; import org.apache.camel.spi.Registry; import org.apache.camel.test.AvailablePortFinder; diff --git a/components/camel-syslog/src/test/java/org/apache/camel/component/syslog/NettyRfc5425Test.java b/components/camel-syslog/src/test/java/org/apache/camel/component/syslog/NettyRfc5425Test.java index 9b9093a..4a37351 100644 --- a/components/camel-syslog/src/test/java/org/apache/camel/component/syslog/NettyRfc5425Test.java +++ b/components/camel-syslog/src/test/java/org/apache/camel/component/syslog/NettyRfc5425Test.java @@ -22,7 +22,6 @@ import org.apache.camel.builder.RouteBuilder; import org.apache.camel.component.mock.MockEndpoint; import org.apache.camel.component.syslog.netty.Rfc5425Encoder; import org.apache.camel.component.syslog.netty.Rfc5425FrameDecoder; -import org.apache.camel.impl.JndiRegistry; import org.apache.camel.spi.DataFormat; import org.apache.camel.spi.Registry; import org.apache.camel.test.AvailablePortFinder;