You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by or...@apache.org on 2022/05/31 15:23:15 UTC

[camel] branch main updated (032a7cded45 -> 4111c3196a9)

This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


    from 032a7cded45 Regen for commit c05c928da1b831c51c5d4617bbf81e0165b2bcf6
     new d82abf99339 (chores) camel-fhir: remove test annotations from non-test
     new fdcf4e92331 (chores) camel-jms: ensure tests have at least 1 assertion
     new 8c2c89964bf (chores) camel-jmx: ensure tests have at least 1 assertion
     new c25f08a4072 (chores) camel-nats: ensure tests have at least 1 assertion
     new 0ceaf1909e9 (chores) camel-azure-storage-blob: rename manual test
     new 784065cd48f (chores) camel-geocoder: rename manual test
     new 63742c5a54b (chores) camel-crypto: rename manual test
     new 32f4548696e (chores) camel-jetty: rename manual test
     new 1b8610e8cb5 (chores) camel-http: rename manual test
     new 4111c3196a9 (chores) camel-mina: rename manual test

The 10 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 ...CopyProducerIt.java => BlobCopyProducerManualIT.java} |  6 +++---
 .../camel/converter/crypto/HMACAccumulatorTest.java      |  5 +++++
 .../org/apache/camel/component/fhir/UrlFetcherTest.java  |  7 +++----
 ...GeoCoderProxyTest.java => GeoCoderProxyManualIT.java} |  4 ++--
 ...oducerLoadTest.java => HttpProducerLoadManualIT.java} |  6 +++---
 ...eTest.java => HttpProducerJMXBeansIssueManualIT.java} |  7 +++----
 .../JmsSendToAlotOfDestinationWithSameEndpointTest.java  | 16 +++++++++++-----
 .../camel/component/jmx/JMXObjectPropertiesTest.java     | 10 +++++++++-
 components/camel-mina/pom.xml                            |  2 +-
 ...utdownTest.java => MinaProducerShutdownManualIT.java} |  6 +++---
 .../camel/component/nats/NatsConsumerStopTest.java       |  9 +++------
 .../component/nats/integration/NatsAuthProducerIT.java   |  4 +++-
 .../nats/integration/NatsAuthTokenProducerIT.java        |  4 +++-
 .../camel/component/nats/integration/NatsProducerIT.java |  4 +++-
 14 files changed, 55 insertions(+), 35 deletions(-)
 rename components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/{BlobCopyProducerIt.java => BlobCopyProducerManualIT.java} (93%)
 rename components/camel-geocoder/src/test/java/org/apache/camel/component/geocoder/{GeoCoderProxyTest.java => GeoCoderProxyManualIT.java} (96%)
 rename components/camel-http/src/test/java/org/apache/camel/component/http/{HttpProducerLoadTest.java => HttpProducerLoadManualIT.java} (96%)
 rename components/camel-jetty/src/test/java/org/apache/camel/component/jetty/{HttpProducerJMXBeansIssueTest.java => HttpProducerJMXBeansIssueManualIT.java} (92%)
 rename components/camel-mina/src/test/java/org/apache/camel/component/mina/{MinaProducerShutdownTest.java => MinaProducerShutdownManualIT.java} (95%)


[camel] 01/10: (chores) camel-fhir: remove test annotations from non-test

Posted by or...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit d82abf993394734c75132d815a45988558f37f57
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Tue May 31 14:49:17 2022 +0200

    (chores) camel-fhir: remove test annotations from non-test
---
 .../test/java/org/apache/camel/component/fhir/UrlFetcherTest.java  | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/components/camel-fhir/camel-fhir-component/src/test/java/org/apache/camel/component/fhir/UrlFetcherTest.java b/components/camel-fhir/camel-fhir-component/src/test/java/org/apache/camel/component/fhir/UrlFetcherTest.java
index 84c928c5cf9..bab36280058 100644
--- a/components/camel-fhir/camel-fhir-component/src/test/java/org/apache/camel/component/fhir/UrlFetcherTest.java
+++ b/components/camel-fhir/camel-fhir-component/src/test/java/org/apache/camel/component/fhir/UrlFetcherTest.java
@@ -20,13 +20,12 @@ import ca.uhn.fhir.context.FhirContext;
 import ca.uhn.fhir.rest.client.api.IGenericClient;
 import ca.uhn.fhir.rest.client.interceptor.LoggingInterceptor;
 import org.hl7.fhir.r4.model.Patient;
-import org.junit.jupiter.api.Disabled;
-import org.junit.jupiter.api.Test;
 
-@Disabled("Helper class to generate search URLs based on HAPI-FHIR's search API")
+/**
+ * Helper class to generate search URLs based on HAPI-FHIR's search API
+ */
 public class UrlFetcherTest {
 
-    @Test
     public void getUrlTest() {
         // Create a client to talk to your favorite test server
         FhirContext ctx = FhirContext.forR4();


[camel] 10/10: (chores) camel-mina: rename manual test

Posted by or...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 4111c3196a9ed0ccee51aaa210cca11d5c1da3cc
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Tue May 31 15:13:10 2022 +0200

    (chores) camel-mina: rename manual test
    
    This prevent it from being flagged with missing assertions in code analysis
---
 components/camel-mina/pom.xml                                       | 2 +-
 ...aProducerShutdownTest.java => MinaProducerShutdownManualIT.java} | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/components/camel-mina/pom.xml b/components/camel-mina/pom.xml
index 8f05d843afc..fa6de55b235 100644
--- a/components/camel-mina/pom.xml
+++ b/components/camel-mina/pom.xml
@@ -94,7 +94,7 @@
                 <groupId>org.codehaus.mojo</groupId>
                 <artifactId>exec-maven-plugin</artifactId>
                 <configuration>
-                    <mainClass>org.apache.camel.component.mina.MinaProducerShutdownTest</mainClass>
+                    <mainClass>org.apache.camel.component.mina.MinaProducerShutdownManualIT</mainClass>
                     <includePluginDependencies>false</includePluginDependencies>
                 </configuration>
             </plugin>
diff --git a/components/camel-mina/src/test/java/org/apache/camel/component/mina/MinaProducerShutdownTest.java b/components/camel-mina/src/test/java/org/apache/camel/component/mina/MinaProducerShutdownManualIT.java
similarity index 95%
rename from components/camel-mina/src/test/java/org/apache/camel/component/mina/MinaProducerShutdownTest.java
rename to components/camel-mina/src/test/java/org/apache/camel/component/mina/MinaProducerShutdownManualIT.java
index 4ace184ce97..ce11339a45e 100644
--- a/components/camel-mina/src/test/java/org/apache/camel/component/mina/MinaProducerShutdownTest.java
+++ b/components/camel-mina/src/test/java/org/apache/camel/component/mina/MinaProducerShutdownManualIT.java
@@ -33,15 +33,15 @@ import org.slf4j.LoggerFactory;
  * Run this test from maven: mvn exec:java and see the output if there is a error.
  */
 @Disabled
-public class MinaProducerShutdownTest {
+public class MinaProducerShutdownManualIT {
 
-    private static final Logger LOG = LoggerFactory.getLogger(MinaProducerShutdownTest.class);
+    private static final Logger LOG = LoggerFactory.getLogger(MinaProducerShutdownManualIT.class);
     private static final String URI = "mina:tcp://localhost:6321?textline=true&sync=false";
     private long start;
     private CamelContext context;
 
     public static void main(String[] args) throws Exception {
-        MinaProducerShutdownTest me = new MinaProducerShutdownTest();
+        MinaProducerShutdownManualIT me = new MinaProducerShutdownManualIT();
         me.testProducer();
     }
 


[camel] 09/10: (chores) camel-http: rename manual test

Posted by or...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 1b8610e8cb5180b9286989b83a48e171be1c7098
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Tue May 31 15:12:52 2022 +0200

    (chores) camel-http: rename manual test
    
    This prevent it from being flagged with missing assertions in code analysis
---
 .../{HttpProducerLoadTest.java => HttpProducerLoadManualIT.java}    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/components/camel-http/src/test/java/org/apache/camel/component/http/HttpProducerLoadTest.java b/components/camel-http/src/test/java/org/apache/camel/component/http/HttpProducerLoadManualIT.java
similarity index 96%
rename from components/camel-http/src/test/java/org/apache/camel/component/http/HttpProducerLoadTest.java
rename to components/camel-http/src/test/java/org/apache/camel/component/http/HttpProducerLoadManualIT.java
index e9fc12bef93..f6160d20a60 100644
--- a/components/camel-http/src/test/java/org/apache/camel/component/http/HttpProducerLoadTest.java
+++ b/components/camel-http/src/test/java/org/apache/camel/component/http/HttpProducerLoadManualIT.java
@@ -37,10 +37,10 @@ import org.slf4j.LoggerFactory;
 
 import static org.apache.camel.component.http.HttpMethods.GET;
 
-@Disabled("Manual test")
-public class HttpProducerLoadTest extends BaseHttpTest {
+@Disabled("Manual integration test")
+public class HttpProducerLoadManualIT extends BaseHttpTest {
 
-    private static final Logger LOG = LoggerFactory.getLogger(HttpProducerLoadTest.class);
+    private static final Logger LOG = LoggerFactory.getLogger(HttpProducerLoadManualIT.class);
 
     private HttpServer localServer;
 


[camel] 03/10: (chores) camel-jmx: ensure tests have at least 1 assertion

Posted by or...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 8c2c89964bfb20e894b2d3154874b618d1d26eab
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Tue May 31 14:57:55 2022 +0200

    (chores) camel-jmx: ensure tests have at least 1 assertion
---
 .../apache/camel/component/jmx/JMXObjectPropertiesTest.java    | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/components/camel-jmx/src/test/java/org/apache/camel/component/jmx/JMXObjectPropertiesTest.java b/components/camel-jmx/src/test/java/org/apache/camel/component/jmx/JMXObjectPropertiesTest.java
index 7fbc1e57e17..75acd9ad0ac 100644
--- a/components/camel-jmx/src/test/java/org/apache/camel/component/jmx/JMXObjectPropertiesTest.java
+++ b/components/camel-jmx/src/test/java/org/apache/camel/component/jmx/JMXObjectPropertiesTest.java
@@ -19,16 +19,24 @@ package org.apache.camel.component.jmx;
 import java.util.Hashtable;
 import java.util.Map;
 
+import javax.management.MalformedObjectNameException;
+
 import org.apache.camel.component.jmx.beans.ISimpleMXBean;
 import org.junit.jupiter.api.Test;
 
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
+
 /**
  * Tests that the objectName is created with the hashtable of objectProperties
  */
 public class JMXObjectPropertiesTest extends SimpleBeanFixture {
 
     @Test
-    public void testObjectProperties() throws Exception {
+    public void testObjectProperties() {
+        assertDoesNotThrow(() -> doTestObjectProperties());
+    }
+
+    private void doTestObjectProperties() throws MalformedObjectNameException, InterruptedException {
         ISimpleMXBean bean = getSimpleMXBean();
         bean.touch();
         getMockFixture().waitForMessages();


[camel] 06/10: (chores) camel-geocoder: rename manual test

Posted by or...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 784065cd48fc59aeab4eaf68e1a7120260828813
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Tue May 31 15:12:07 2022 +0200

    (chores) camel-geocoder: rename manual test
    
    This prevent it from being flagged with missing assertions in code analysis
---
 .../geocoder/{GeoCoderProxyTest.java => GeoCoderProxyManualIT.java}   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/components/camel-geocoder/src/test/java/org/apache/camel/component/geocoder/GeoCoderProxyTest.java b/components/camel-geocoder/src/test/java/org/apache/camel/component/geocoder/GeoCoderProxyManualIT.java
similarity index 96%
rename from components/camel-geocoder/src/test/java/org/apache/camel/component/geocoder/GeoCoderProxyTest.java
rename to components/camel-geocoder/src/test/java/org/apache/camel/component/geocoder/GeoCoderProxyManualIT.java
index af64af3abd4..df29e9723f4 100644
--- a/components/camel-geocoder/src/test/java/org/apache/camel/component/geocoder/GeoCoderProxyTest.java
+++ b/components/camel-geocoder/src/test/java/org/apache/camel/component/geocoder/GeoCoderProxyManualIT.java
@@ -27,9 +27,9 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @Disabled("This test should be executed manually with a proxy set up")
-public class GeoCoderProxyTest extends GeoCoderApiKeyTestBase {
+public class GeoCoderProxyManualIT extends GeoCoderApiKeyTestBase {
 
-    private static final Logger LOG = LoggerFactory.getLogger(GeoCoderProxyTest.class);
+    private static final Logger LOG = LoggerFactory.getLogger(GeoCoderProxyManualIT.class);
 
     @Test
     public void testGeoCoderProxyNoAuth() throws Exception {


[camel] 02/10: (chores) camel-jms: ensure tests have at least 1 assertion

Posted by or...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit fdcf4e9233104a221cec050cbb0d650261dc49da
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Tue May 31 14:54:55 2022 +0200

    (chores) camel-jms: ensure tests have at least 1 assertion
---
 .../JmsSendToAlotOfDestinationWithSameEndpointTest.java  | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/components/camel-jms/src/test/java/org/apache/camel/component/jms/issues/JmsSendToAlotOfDestinationWithSameEndpointTest.java b/components/camel-jms/src/test/java/org/apache/camel/component/jms/issues/JmsSendToAlotOfDestinationWithSameEndpointTest.java
index 99b760ee863..1a72136cca2 100644
--- a/components/camel-jms/src/test/java/org/apache/camel/component/jms/issues/JmsSendToAlotOfDestinationWithSameEndpointTest.java
+++ b/components/camel-jms/src/test/java/org/apache/camel/component/jms/issues/JmsSendToAlotOfDestinationWithSameEndpointTest.java
@@ -25,6 +25,8 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.context.support.AbstractXmlApplicationContext;
 
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
+
 public class JmsSendToAlotOfDestinationWithSameEndpointTest extends CamelSpringTestSupport {
 
     private static final Logger LOG = LoggerFactory.getLogger(JmsSendToAlotOfDestinationWithSameEndpointTest.class);
@@ -32,24 +34,28 @@ public class JmsSendToAlotOfDestinationWithSameEndpointTest extends CamelSpringT
 
     @Test
     public void testSendToAlotOfMessageToQueues() {
+        assertDoesNotThrow(() -> sendToAlotOfMessagesToQueue());
+
+        // now we should be able to poll a message from each queue
+        // Thread.sleep(99999999);
+    }
+
+    private void sendToAlotOfMessagesToQueue() {
         int size = 100;
 
-        LOG.info("About to send " + size + " messages");
+        LOG.info("About to send {} messages", size);
 
         for (int i = 0; i < size; i++) {
             // use the same endpoint but provide a header with the dynamic queue we send to
             // this allows us to reuse endpoints and not create a new endpoint for each and every jms queue
             // we send to
             if (i > 0 && i % 50 == 0) {
-                LOG.info("Send " + i + " messages so far");
+                LOG.info("Sent {} messages so far", i);
             }
             template.sendBodyAndHeader(URI, ExchangePattern.InOnly, "Hello " + i, JmsConstants.JMS_DESTINATION_NAME, "foo" + i);
         }
 
         LOG.info("Send complete use jconsole to view");
-
-        // now we should be able to poll a message from each queue
-        // Thread.sleep(99999999);
     }
 
     @Override


[camel] 08/10: (chores) camel-jetty: rename manual test

Posted by or...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 32f4548696e1717a48cadc34d2d0e7c1db468bfd
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Tue May 31 15:12:41 2022 +0200

    (chores) camel-jetty: rename manual test
    
    This prevent it from being flagged with missing assertions in code analysis
---
 ...XBeansIssueTest.java => HttpProducerJMXBeansIssueManualIT.java} | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/HttpProducerJMXBeansIssueTest.java b/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/HttpProducerJMXBeansIssueManualIT.java
similarity index 92%
rename from components/camel-jetty/src/test/java/org/apache/camel/component/jetty/HttpProducerJMXBeansIssueTest.java
rename to components/camel-jetty/src/test/java/org/apache/camel/component/jetty/HttpProducerJMXBeansIssueManualIT.java
index cda48a6790d..d726176156a 100644
--- a/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/HttpProducerJMXBeansIssueTest.java
+++ b/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/HttpProducerJMXBeansIssueManualIT.java
@@ -27,9 +27,10 @@ import org.junit.jupiter.api.Test;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-public class HttpProducerJMXBeansIssueTest extends BaseJettyTest {
+@Disabled("This is a manual test")
+public final class HttpProducerJMXBeansIssueManualIT extends BaseJettyTest {
 
-    private static final Logger LOG = LoggerFactory.getLogger(HttpProducerJMXBeansIssueTest.class);
+    private static final Logger LOG = LoggerFactory.getLogger(HttpProducerJMXBeansIssueManualIT.class);
 
     @Override
     @BeforeEach
@@ -57,13 +58,11 @@ public class HttpProducerJMXBeansIssueTest extends BaseJettyTest {
     }
 
     @Test
-    @Disabled("disabled as this is a manual test")
     public void testNothing() {
         // do nothing as this test is manual
     }
 
     @Test
-    @Disabled("disabled as this is a manual test")
     public void testSendAlot() throws Exception {
         Endpoint ep = context.getEndpoint("direct:leak");
         Producer p = ep.createProducer();


[camel] 04/10: (chores) camel-nats: ensure tests have at least 1 assertion

Posted by or...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit c25f08a40728edea37db59e9f7dc9941e15cf0cc
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Tue May 31 15:03:34 2022 +0200

    (chores) camel-nats: ensure tests have at least 1 assertion
---
 .../org/apache/camel/component/nats/NatsConsumerStopTest.java    | 9 +++------
 .../camel/component/nats/integration/NatsAuthProducerIT.java     | 4 +++-
 .../component/nats/integration/NatsAuthTokenProducerIT.java      | 4 +++-
 .../apache/camel/component/nats/integration/NatsProducerIT.java  | 4 +++-
 4 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/components/camel-nats/src/test/java/org/apache/camel/component/nats/NatsConsumerStopTest.java b/components/camel-nats/src/test/java/org/apache/camel/component/nats/NatsConsumerStopTest.java
index 7e67d7b9454..91c054a4ae5 100644
--- a/components/camel-nats/src/test/java/org/apache/camel/component/nats/NatsConsumerStopTest.java
+++ b/components/camel-nats/src/test/java/org/apache/camel/component/nats/NatsConsumerStopTest.java
@@ -19,20 +19,17 @@ package org.apache.camel.component.nats;
 import org.apache.camel.Consumer;
 import org.apache.camel.test.junit5.CamelTestSupport;
 import org.junit.jupiter.api.Test;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 
 /**
  * Test NATS consumer stop happens cleanly. See https://issues.apache.org/jira/browse/CAMEL-15834.
  */
 public class NatsConsumerStopTest extends CamelTestSupport {
-
-    private static final Logger LOG = LoggerFactory.getLogger(NatsConsumerStopTest.class);
-
     @Test
     public void testConsumerStop() throws Exception {
         NatsEndpoint endpoint = context.getEndpoint("nats:test?flushConnection=true", NatsEndpoint.class);
         Consumer consumer = endpoint.createConsumer(null);
-        consumer.stop();
+        assertDoesNotThrow(() -> consumer.stop());
     }
 }
diff --git a/components/camel-nats/src/test/java/org/apache/camel/component/nats/integration/NatsAuthProducerIT.java b/components/camel-nats/src/test/java/org/apache/camel/component/nats/integration/NatsAuthProducerIT.java
index d11b45ba165..e0809c29e52 100644
--- a/components/camel-nats/src/test/java/org/apache/camel/component/nats/integration/NatsAuthProducerIT.java
+++ b/components/camel-nats/src/test/java/org/apache/camel/component/nats/integration/NatsAuthProducerIT.java
@@ -19,11 +19,13 @@ package org.apache.camel.component.nats.integration;
 import org.apache.camel.builder.RouteBuilder;
 import org.junit.jupiter.api.Test;
 
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
+
 public class NatsAuthProducerIT extends NatsAuthITSupport {
 
     @Test
     public void sendTest() {
-        template.sendBody("direct:send", "pippo");
+        assertDoesNotThrow(() -> template.sendBody("direct:send", "pippo"));
     }
 
     @Override
diff --git a/components/camel-nats/src/test/java/org/apache/camel/component/nats/integration/NatsAuthTokenProducerIT.java b/components/camel-nats/src/test/java/org/apache/camel/component/nats/integration/NatsAuthTokenProducerIT.java
index 2d4c7252836..5ac9b90a6ef 100644
--- a/components/camel-nats/src/test/java/org/apache/camel/component/nats/integration/NatsAuthTokenProducerIT.java
+++ b/components/camel-nats/src/test/java/org/apache/camel/component/nats/integration/NatsAuthTokenProducerIT.java
@@ -19,11 +19,13 @@ package org.apache.camel.component.nats.integration;
 import org.apache.camel.builder.RouteBuilder;
 import org.junit.jupiter.api.Test;
 
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
+
 public class NatsAuthTokenProducerIT extends NatsAuthITSupport {
 
     @Test
     public void sendTest() {
-        template.sendBody("direct:send", "pippo");
+        assertDoesNotThrow(() -> template.sendBody("direct:send", "pippo"));
     }
 
     @Override
diff --git a/components/camel-nats/src/test/java/org/apache/camel/component/nats/integration/NatsProducerIT.java b/components/camel-nats/src/test/java/org/apache/camel/component/nats/integration/NatsProducerIT.java
index 1d563ead3a6..1d758a14ce4 100644
--- a/components/camel-nats/src/test/java/org/apache/camel/component/nats/integration/NatsProducerIT.java
+++ b/components/camel-nats/src/test/java/org/apache/camel/component/nats/integration/NatsProducerIT.java
@@ -19,11 +19,13 @@ package org.apache.camel.component.nats.integration;
 import org.apache.camel.builder.RouteBuilder;
 import org.junit.jupiter.api.Test;
 
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
+
 public class NatsProducerIT extends NatsITSupport {
 
     @Test
     public void sendTest() {
-        template.sendBody("direct:send", "pippo");
+        assertDoesNotThrow(() -> template.sendBody("direct:send", "pippo"));
     }
 
     @Override


[camel] 05/10: (chores) camel-azure-storage-blob: rename manual test

Posted by or...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 0ceaf1909e978e2b46ce5397147ba6374a2f47c4
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Tue May 31 15:11:36 2022 +0200

    (chores) camel-azure-storage-blob: rename manual test
    
    This prevent it from being flagged with missing assertions in code analysis
---
 .../{BlobCopyProducerIt.java => BlobCopyProducerManualIT.java}      | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobCopyProducerIt.java b/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobCopyProducerManualIT.java
similarity index 93%
rename from components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobCopyProducerIt.java
rename to components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobCopyProducerManualIT.java
index 766b89a50e1..80af145a521 100644
--- a/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobCopyProducerIt.java
+++ b/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobCopyProducerManualIT.java
@@ -21,11 +21,11 @@ import org.apache.camel.ProducerTemplate;
 import org.apache.camel.builder.RouteBuilder;
 import org.apache.camel.component.azure.storage.blob.BlobConstants;
 import org.apache.camel.test.junit5.CamelTestSupport;
-import org.junit.Ignore;
+import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
-@Ignore("Requires real credentials")
-class BlobCopyProducerIt extends CamelTestSupport {
+@Disabled("Manual test that requires real credentials")
+class BlobCopyProducerManualIT extends CamelTestSupport {
 
     @EndpointInject
     private ProducerTemplate template;


[camel] 07/10: (chores) camel-crypto: rename manual test

Posted by or...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 63742c5a54b9baf61c2f7e68d9a9078614a437d1
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Tue May 31 15:12:19 2022 +0200

    (chores) camel-crypto: rename manual test
    
    This prevent it from being flagged with missing assertions in code analysis
---
 .../java/org/apache/camel/converter/crypto/HMACAccumulatorTest.java  | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/components/camel-crypto/src/test/java/org/apache/camel/converter/crypto/HMACAccumulatorTest.java b/components/camel-crypto/src/test/java/org/apache/camel/converter/crypto/HMACAccumulatorTest.java
index f67a10098dd..54f93714c37 100644
--- a/components/camel-crypto/src/test/java/org/apache/camel/converter/crypto/HMACAccumulatorTest.java
+++ b/components/camel-crypto/src/test/java/org/apache/camel/converter/crypto/HMACAccumulatorTest.java
@@ -27,6 +27,7 @@ import org.apache.camel.converter.crypto.HMACAccumulator.CircularBuffer;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
 public class HMACAccumulatorTest {
@@ -163,6 +164,10 @@ public class HMACAccumulatorTest {
 
     @Test
     void testBufferCompare() {
+        assertDoesNotThrow(() -> doBufferCompare());
+    }
+
+    private void doBufferCompare() {
         CircularBuffer buffer = new CircularBuffer(payload.length * 2);
         buffer.write(new byte[payload.length >> 1], 0, payload.length >> 1);
         buffer.write(payload, 0, payload.length);