This is an automated email from the ASF dual-hosted git repository. orpiske pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git
The following commit(s) were added to refs/heads/master by this push: new 7c0983c CAMEL-16400: split unit and integration tests (azure components) (#5325) 7c0983c is described below commit 7c0983c7ac2ce25adf88473eb7279f3bd5ad0fd9 Author: Otavio Rodolfo Piske <orpi...@users.noreply.github.com> AuthorDate: Fri Apr 9 16:24:08 2021 +0200 CAMEL-16400: split unit and integration tests (azure components) (#5325) * split unit and integration tests for camel-azure-eventhubs * split unit and integration tests for camel-azure-storage-blob * split unit and integration tests for camel-azure-storage-datalake * split unit and integration tests for camel-azure-storage-queue --- .../camel-azure/camel-azure-eventhubs/pom.xml | 38 +++++----------------- .../{ => integration}/EventHubsConsumerIT.java | 8 ++++- .../{ => integration}/EventHubsProducerIT.java | 8 ++++- .../{ => integration}/EventProcessorIT.java | 7 +++- .../operations/EventHubsProducerOperationsIT.java | 6 +++- .../camel-azure/camel-azure-storage-blob/pom.xml | 9 +++++ .../blob/integration/{BaseIT.java => Base.java} | 2 +- ...BlobConsumerITTest.java => BlobConsumerIT.java} | 2 +- ...sITTest.java => BlobContainerOperationsIT.java} | 2 +- ...OperationsITTest.java => BlobOperationsIT.java} | 2 +- ...BlobProducerITTest.java => BlobProducerIT.java} | 2 +- .../camel-azure-storage-datalake/pom.xml | 38 +++++----------------- .../integration/{BaseIT.java => Base.java} | 9 ++--- .../datalake/integration/DataLakeConsumerIT.java | 4 ++- .../integration/DataLakeFileOperationIT.java | 6 ++-- .../integration/DataLakeFileSystemOperationIT.java | 6 ++-- .../datalake/integration/DataLakeProducerIT.java | 4 ++- .../camel-azure/camel-azure-storage-queue/pom.xml | 38 +++++----------------- .../azure/storage/queue/QueueTestUtils.java | 5 +-- .../queue/{ => integration}/QueueConsumerIT.java | 7 +++- .../queue/{ => integration}/QueueProducerIT.java | 8 ++++- .../{ => integration}/QueueOperationsIT.java | 7 +++- .../QueueServiceOperationsIT.java | 7 +++- 23 files changed, 109 insertions(+), 116 deletions(-) diff --git a/components/camel-azure/camel-azure-eventhubs/pom.xml b/components/camel-azure/camel-azure-eventhubs/pom.xml index 92034b0..bfb7917 100644 --- a/components/camel-azure/camel-azure-eventhubs/pom.xml +++ b/components/camel-azure/camel-azure-eventhubs/pom.xml @@ -91,33 +91,13 @@ <scope>test</scope> </dependency> </dependencies> - <profiles> - <profile> - <id>fullTests</id> - <build> - <plugins> - <plugin> - <artifactId>maven-surefire-plugin</artifactId> - <version>${maven-surefire-plugin-version}</version> - <executions> - <execution> - <phase>integration-test</phase> - <goals> - <goal>test</goal> - </goals> - <configuration> - <excludes> - <exclude>none</exclude> - </excludes> - <includes> - <include>**/*IT</include> - </includes> - </configuration> - </execution> - </executions> - </plugin> - </plugins> - </build> - </profile> - </profiles> + + <build> + <plugins> + <plugin> + <groupId>org.apache.maven.plugins</groupId> + <artifactId>maven-failsafe-plugin</artifactId> + </plugin> + </plugins> + </build> </project> diff --git a/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/EventHubsConsumerIT.java b/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/integration/EventHubsConsumerIT.java similarity index 91% rename from components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/EventHubsConsumerIT.java rename to components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/integration/EventHubsConsumerIT.java index 88f04cb..3ed0bfe 100644 --- a/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/EventHubsConsumerIT.java +++ b/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/integration/EventHubsConsumerIT.java @@ -14,7 +14,7 @@ * See the License for the specific language governing permissions and * limitations under the License. */ -package org.apache.camel.component.azure.eventhubs; +package org.apache.camel.component.azure.eventhubs.integration; import java.util.Collections; import java.util.HashMap; @@ -32,6 +32,9 @@ import org.apache.camel.CamelContext; import org.apache.camel.EndpointInject; import org.apache.camel.Exchange; import org.apache.camel.builder.RouteBuilder; +import org.apache.camel.component.azure.eventhubs.EventHubsConfiguration; +import org.apache.camel.component.azure.eventhubs.EventHubsConstants; +import org.apache.camel.component.azure.eventhubs.TestUtils; import org.apache.camel.component.azure.eventhubs.client.EventHubsClientFactory; import org.apache.camel.component.mock.MockEndpoint; import org.apache.camel.test.junit5.CamelTestSupport; @@ -40,10 +43,13 @@ import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestInstance; +import org.junit.jupiter.api.condition.EnabledIfSystemProperty; import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertNotNull; +@EnabledIfSystemProperty(named = "connectionString", matches = ".*", + disabledReason = "Make sure to supply azure eventHubs connectionString, e.g: mvn verify -DconnectionString=string -DblobAccountName=blob -DblobAccessKey=key") @TestInstance(TestInstance.Lifecycle.PER_CLASS) class EventHubsConsumerIT extends CamelTestSupport { diff --git a/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/EventHubsProducerIT.java b/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/integration/EventHubsProducerIT.java similarity index 90% rename from components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/EventHubsProducerIT.java rename to components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/integration/EventHubsProducerIT.java index 30e6a32..fb62262 100644 --- a/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/EventHubsProducerIT.java +++ b/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/integration/EventHubsProducerIT.java @@ -14,7 +14,7 @@ * See the License for the specific language governing permissions and * limitations under the License. */ -package org.apache.camel.component.azure.eventhubs; +package org.apache.camel.component.azure.eventhubs.integration; import java.util.Properties; import java.util.concurrent.CompletableFuture; @@ -28,6 +28,9 @@ import org.apache.camel.EndpointInject; import org.apache.camel.Exchange; import org.apache.camel.ProducerTemplate; import org.apache.camel.builder.RouteBuilder; +import org.apache.camel.component.azure.eventhubs.EventHubsConfiguration; +import org.apache.camel.component.azure.eventhubs.EventHubsConstants; +import org.apache.camel.component.azure.eventhubs.TestUtils; import org.apache.camel.component.azure.eventhubs.client.EventHubsClientFactory; import org.apache.camel.component.mock.MockEndpoint; import org.apache.camel.test.junit5.CamelTestSupport; @@ -36,10 +39,13 @@ import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestInstance; +import org.junit.jupiter.api.condition.EnabledIfSystemProperty; import static org.junit.jupiter.api.Assertions.assertNotNull; import static org.junit.jupiter.api.Assertions.assertTrue; +@EnabledIfSystemProperty(named = "connectionString", matches = ".*", + disabledReason = "Make sure to supply azure eventHubs connectionString, e.g: mvn verify -DconnectionString=string -DblobAccountName=blob -DblobAccessKey=key") @TestInstance(TestInstance.Lifecycle.PER_CLASS) class EventHubsProducerIT extends CamelTestSupport { diff --git a/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/EventProcessorIT.java b/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/integration/EventProcessorIT.java similarity index 89% rename from components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/EventProcessorIT.java rename to components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/integration/EventProcessorIT.java index f0273a0..371b02a 100644 --- a/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/EventProcessorIT.java +++ b/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/integration/EventProcessorIT.java @@ -14,7 +14,7 @@ * See the License for the specific language governing permissions and * limitations under the License. */ -package org.apache.camel.component.azure.eventhubs; +package org.apache.camel.component.azure.eventhubs.integration; import java.util.Collections; import java.util.HashMap; @@ -33,6 +33,8 @@ import com.azure.messaging.eventhubs.models.EventContext; import com.azure.messaging.eventhubs.models.EventPosition; import com.azure.messaging.eventhubs.models.SendOptions; import com.azure.storage.blob.BlobContainerAsyncClient; +import org.apache.camel.component.azure.eventhubs.EventHubsConfiguration; +import org.apache.camel.component.azure.eventhubs.TestUtils; import org.apache.camel.component.azure.eventhubs.client.EventHubsClientFactory; import org.apache.commons.lang3.RandomStringUtils; import org.awaitility.Awaitility; @@ -40,9 +42,12 @@ import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestInstance; +import org.junit.jupiter.api.condition.EnabledIfSystemProperty; import static org.junit.jupiter.api.Assertions.assertTrue; +@EnabledIfSystemProperty(named = "connectionString", matches = ".*", + disabledReason = "Make sure to supply azure eventHubs connectionString, e.g: mvn verify -DconnectionString=string -DblobAccountName=blob -DblobAccessKey=key") @TestInstance(TestInstance.Lifecycle.PER_CLASS) public class EventProcessorIT { diff --git a/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/operations/EventHubsProducerOperationsIT.java b/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/integration/operations/EventHubsProducerOperationsIT.java similarity index 94% rename from components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/operations/EventHubsProducerOperationsIT.java rename to components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/integration/operations/EventHubsProducerOperationsIT.java index e5d7960..ad9cca3 100644 --- a/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/operations/EventHubsProducerOperationsIT.java +++ b/components/camel-azure/camel-azure-eventhubs/src/test/java/org/apache/camel/component/azure/eventhubs/integration/operations/EventHubsProducerOperationsIT.java @@ -14,7 +14,7 @@ * See the License for the specific language governing permissions and * limitations under the License. */ -package org.apache.camel.component.azure.eventhubs.operations; +package org.apache.camel.component.azure.eventhubs.integration.operations; import java.time.Duration; import java.util.LinkedList; @@ -31,6 +31,7 @@ import org.apache.camel.component.azure.eventhubs.EventHubsConfiguration; import org.apache.camel.component.azure.eventhubs.EventHubsConstants; import org.apache.camel.component.azure.eventhubs.TestUtils; import org.apache.camel.component.azure.eventhubs.client.EventHubsClientFactory; +import org.apache.camel.component.azure.eventhubs.operations.EventHubsProducerOperations; import org.apache.camel.support.DefaultExchange; import org.apache.camel.test.junit5.CamelTestSupport; import org.awaitility.Awaitility; @@ -38,7 +39,10 @@ import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestInstance; +import org.junit.jupiter.api.condition.EnabledIfSystemProperty; +@EnabledIfSystemProperty(named = "connectionString", matches = ".*", + disabledReason = "Make sure to supply azure eventHubs connectionString, e.g: mvn verify -DconnectionString=string -DblobAccountName=blob -DblobAccessKey=key") @TestInstance(TestInstance.Lifecycle.PER_CLASS) class EventHubsProducerOperationsIT extends CamelTestSupport { diff --git a/components/camel-azure/camel-azure-storage-blob/pom.xml b/components/camel-azure/camel-azure-storage-blob/pom.xml index 0cb01f2..9622a7c 100644 --- a/components/camel-azure/camel-azure-storage-blob/pom.xml +++ b/components/camel-azure/camel-azure-storage-blob/pom.xml @@ -112,4 +112,13 @@ <scope>test</scope> </dependency> </dependencies> + + <build> + <plugins> + <plugin> + <groupId>org.apache.maven.plugins</groupId> + <artifactId>maven-failsafe-plugin</artifactId> + </plugin> + </plugins> + </build> </project> diff --git a/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BaseIT.java b/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/Base.java similarity index 98% rename from components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BaseIT.java rename to components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/Base.java index 522458b..9852a4e 100644 --- a/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BaseIT.java +++ b/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/Base.java @@ -32,7 +32,7 @@ import org.junit.jupiter.api.TestInstance; import org.junit.jupiter.api.extension.RegisterExtension; @TestInstance(TestInstance.Lifecycle.PER_CLASS) -public class BaseIT extends CamelTestSupport { +public class Base extends CamelTestSupport { @RegisterExtension public static AzureService service; diff --git a/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobConsumerITTest.java b/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobConsumerIT.java similarity index 99% rename from components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobConsumerITTest.java rename to components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobConsumerIT.java index 4a18931..f0f3200 100644 --- a/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobConsumerITTest.java +++ b/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobConsumerIT.java @@ -42,7 +42,7 @@ import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertNotNull; import static org.junit.jupiter.api.Assertions.assertTrue; -class BlobConsumerITTest extends BaseIT { +class BlobConsumerIT extends Base { @TempDir static Path testDir; diff --git a/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobContainerOperationsITTest.java b/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobContainerOperationsIT.java similarity index 98% rename from components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobContainerOperationsITTest.java rename to components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobContainerOperationsIT.java index 2e45bde..3e91bbb 100644 --- a/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobContainerOperationsITTest.java +++ b/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobContainerOperationsIT.java @@ -36,7 +36,7 @@ import org.junit.jupiter.api.Test; import static org.junit.jupiter.api.Assertions.assertNotNull; import static org.junit.jupiter.api.Assertions.assertTrue; -class BlobContainerOperationsITTest extends BaseIT { +class BlobContainerOperationsIT extends Base { private BlobServiceClientWrapper blobServiceClientWrapper; diff --git a/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobOperationsITTest.java b/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobOperationsIT.java similarity index 99% rename from components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobOperationsITTest.java rename to components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobOperationsIT.java index fbf60be..ccc1560 100644 --- a/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobOperationsITTest.java +++ b/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobOperationsIT.java @@ -57,7 +57,7 @@ import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertNotNull; import static org.junit.jupiter.api.Assertions.assertTrue; -class BlobOperationsITTest extends BaseIT { +class BlobOperationsIT extends Base { private BlobContainerClientWrapper blobContainerClientWrapper; diff --git a/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobProducerITTest.java b/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobProducerIT.java similarity index 99% rename from components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobProducerITTest.java rename to components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobProducerIT.java index f4ad930..9f373c6 100644 --- a/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobProducerITTest.java +++ b/components/camel-azure/camel-azure-storage-blob/src/test/java/org/apache/camel/component/azure/storage/blob/integration/BlobProducerIT.java @@ -40,7 +40,7 @@ import org.junit.jupiter.api.Test; import static org.junit.jupiter.api.Assertions.assertNotNull; import static org.junit.jupiter.api.Assertions.assertTrue; -class BlobProducerITTest extends BaseIT { +class BlobProducerIT extends Base { @EndpointInject private ProducerTemplate template; diff --git a/components/camel-azure/camel-azure-storage-datalake/pom.xml b/components/camel-azure/camel-azure-storage-datalake/pom.xml index eef1a12..35d9ae2 100644 --- a/components/camel-azure/camel-azure-storage-datalake/pom.xml +++ b/components/camel-azure/camel-azure-storage-datalake/pom.xml @@ -109,34 +109,14 @@ <scope>test</scope> </dependency> </dependencies> - <profiles> - <profile> - <id>fullTests</id> - <build> - <plugins> - <plugin> - <artifactId>maven-surefire-plugin</artifactId> - <version>${maven-surefire-plugin-version}</version> - <executions> - <execution> - <phase>integration-test</phase> - <goals> - <goal>test</goal> - </goals> - <configuration> - <excludes> - <exclude>none</exclude> - </excludes> - <includes> - <include>**/*IT</include> - </includes> - </configuration> - </execution> - </executions> - </plugin> - </plugins> - </build> - </profile> - </profiles> + + <build> + <plugins> + <plugin> + <groupId>org.apache.maven.plugins</groupId> + <artifactId>maven-failsafe-plugin</artifactId> + </plugin> + </plugins> + </build> </project> diff --git a/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/BaseIT.java b/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/Base.java similarity index 93% rename from components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/BaseIT.java rename to components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/Base.java index ae466e2..bb6d4d2 100644 --- a/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/BaseIT.java +++ b/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/Base.java @@ -33,17 +33,16 @@ import org.apache.camel.test.infra.azure.storage.datalake.services.AzureStorageD import org.apache.camel.test.junit5.CamelTestSupport; import org.apache.commons.lang3.RandomStringUtils; import org.apache.commons.lang3.StringUtils; -import org.junit.Ignore; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.TestInstance; import org.junit.jupiter.api.extension.RegisterExtension; -@Ignore +//@EnabledIfSystemProperty(named = "azure.instance.type", matches = "remote") @TestInstance(TestInstance.Lifecycle.PER_CLASS) -public class BaseIT extends CamelTestSupport { +public class Base extends CamelTestSupport { @RegisterExtension - public static AzureService service; + public AzureService service = AzureStorageDataLakeServiceFactory.createService(); protected DataLakeServiceClient serviceClient; protected DataLakeConfiguration configuration; @@ -51,7 +50,6 @@ public class BaseIT extends CamelTestSupport { static { initCredentials(); - service = AzureStorageDataLakeServiceFactory.createService(); } @Override @@ -68,7 +66,6 @@ public class BaseIT extends CamelTestSupport { if (StringUtils.isNotEmpty(accountName) && StringUtils.isNotEmpty(accessKey)) { System.setProperty(AzureConfigs.ACCOUNT_NAME, accountName); System.setProperty(AzureConfigs.ACCOUNT_KEY, accessKey); - System.setProperty("azure.instance.type", "remote"); } } diff --git a/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeConsumerIT.java b/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeConsumerIT.java index 4bd0062..6a6687e 100644 --- a/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeConsumerIT.java +++ b/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeConsumerIT.java @@ -42,6 +42,7 @@ import org.apache.commons.lang3.RandomStringUtils; import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.condition.EnabledIfSystemProperty; import org.junit.jupiter.api.io.TempDir; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -50,7 +51,8 @@ import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertNotNull; import static org.junit.jupiter.api.Assertions.assertTrue; -public class DataLakeConsumerIT extends BaseIT { +@EnabledIfSystemProperty(named = "azure.instance.type", matches = "remote") +public class DataLakeConsumerIT extends Base { @TempDir static Path testDir; diff --git a/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeFileOperationIT.java b/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeFileOperationIT.java index c51233c..28ef403 100644 --- a/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeFileOperationIT.java +++ b/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeFileOperationIT.java @@ -38,16 +38,16 @@ import org.apache.camel.component.azure.storage.datalake.operations.DataLakeOper import org.apache.camel.support.DefaultExchange; import org.apache.commons.io.FileUtils; import org.apache.commons.lang3.RandomStringUtils; -import org.junit.Ignore; import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.condition.EnabledIfSystemProperty; import org.junit.jupiter.api.io.TempDir; import static org.junit.jupiter.api.Assertions.*; -@Ignore -public class DataLakeFileOperationIT extends BaseIT { +@EnabledIfSystemProperty(named = "azure.instance.type", matches = "remote") +public class DataLakeFileOperationIT extends Base { private DataLakeFileSystemClientWrapper fileSystemClientWrapper; private String randomFileName; diff --git a/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeFileSystemOperationIT.java b/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeFileSystemOperationIT.java index 72bd016..3eacc8f 100644 --- a/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeFileSystemOperationIT.java +++ b/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeFileSystemOperationIT.java @@ -25,15 +25,15 @@ import org.apache.camel.component.azure.storage.datalake.client.DataLakeServiceC import org.apache.camel.component.azure.storage.datalake.operations.DataLakeFileSystemOperations; import org.apache.camel.component.azure.storage.datalake.operations.DataLakeOperationResponse; import org.awaitility.Awaitility; -import org.junit.Ignore; import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.condition.EnabledIfSystemProperty; import static org.junit.jupiter.api.Assertions.*; -@Ignore -public class DataLakeFileSystemOperationIT extends BaseIT { +@EnabledIfSystemProperty(named = "azure.instance.type", matches = "remote") +public class DataLakeFileSystemOperationIT extends Base { private DataLakeServiceClientWrapper serviceClientWrapper; diff --git a/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeProducerIT.java b/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeProducerIT.java index d306d77..de87788 100644 --- a/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeProducerIT.java +++ b/components/camel-azure/camel-azure-storage-datalake/src/test/java/org/apache/camel/component/azure/storage/datalake/integration/DataLakeProducerIT.java @@ -27,10 +27,12 @@ import org.apache.commons.lang3.RandomStringUtils; import org.assertj.core.api.Assertions; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.condition.EnabledIfSystemProperty; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -public class DataLakeProducerIT extends BaseIT { +@EnabledIfSystemProperty(named = "azure.instance.type", matches = "remote") +public class DataLakeProducerIT extends Base { private static final Logger LOG = LoggerFactory.getLogger(DataLakeProducerIT.class); diff --git a/components/camel-azure/camel-azure-storage-queue/pom.xml b/components/camel-azure/camel-azure-storage-queue/pom.xml index d57dd53..f7c4780 100644 --- a/components/camel-azure/camel-azure-storage-queue/pom.xml +++ b/components/camel-azure/camel-azure-storage-queue/pom.xml @@ -76,33 +76,13 @@ <scope>test</scope> </dependency> </dependencies> - <profiles> - <profile> - <id>fullTests</id> - <build> - <plugins> - <plugin> - <artifactId>maven-surefire-plugin</artifactId> - <version>${maven-surefire-plugin-version}</version> - <executions> - <execution> - <phase>integration-test</phase> - <goals> - <goal>test</goal> - </goals> - <configuration> - <excludes> - <exclude>none</exclude> - </excludes> - <includes> - <include>**/*IT</include> - </includes> - </configuration> - </execution> - </executions> - </plugin> - </plugins> - </build> - </profile> - </profiles> + + <build> + <plugins> + <plugin> + <groupId>org.apache.maven.plugins</groupId> + <artifactId>maven-failsafe-plugin</artifactId> + </plugin> + </plugins> + </build> </project> diff --git a/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/QueueTestUtils.java b/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/QueueTestUtils.java index 8f60b1c..ec5cb59 100644 --- a/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/QueueTestUtils.java +++ b/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/QueueTestUtils.java @@ -21,6 +21,8 @@ import java.io.InputStream; import java.util.Objects; import java.util.Properties; +import static org.junit.jupiter.api.Assertions.fail; + public final class QueueTestUtils { private QueueTestUtils() { @@ -41,8 +43,7 @@ public final class QueueTestUtils { public static Properties loadAzureAccessFromJvmEnv() throws Exception { final Properties properties = new Properties(); if (System.getProperty("accountName") == null || System.getProperty("accessKey") == null) { - throw new Exception( - "Make sure to supply azure accessKey or accountName, e.g: mvn verify -PfullTests -DaccountName=myacc -DaccessKey=mykey"); + fail("Make sure to supply azure accessKey or accountName, e.g: mvn verify -PfullTests -DaccountName=myacc -DaccessKey=mykey"); } properties.setProperty("account_name", System.getProperty("accountName")); properties.setProperty("access_key", System.getProperty("accessKey")); diff --git a/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/QueueConsumerIT.java b/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/integration/QueueConsumerIT.java similarity index 90% rename from components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/QueueConsumerIT.java rename to components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/integration/QueueConsumerIT.java index 8778559..130a254 100644 --- a/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/QueueConsumerIT.java +++ b/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/integration/QueueConsumerIT.java @@ -14,7 +14,7 @@ * See the License for the specific language governing permissions and * limitations under the License. */ -package org.apache.camel.component.azure.storage.queue; +package org.apache.camel.component.azure.storage.queue.integration; import java.util.List; import java.util.Properties; @@ -25,6 +25,8 @@ import org.apache.camel.CamelContext; import org.apache.camel.EndpointInject; import org.apache.camel.Exchange; import org.apache.camel.builder.RouteBuilder; +import org.apache.camel.component.azure.storage.queue.QueueConfiguration; +import org.apache.camel.component.azure.storage.queue.QueueTestUtils; import org.apache.camel.component.azure.storage.queue.client.QueueClientFactory; import org.apache.camel.component.azure.storage.queue.client.QueueClientWrapper; import org.apache.camel.component.azure.storage.queue.client.QueueServiceClientWrapper; @@ -35,9 +37,12 @@ import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestInstance; +import org.junit.jupiter.api.condition.EnabledIfSystemProperty; import static org.junit.jupiter.api.Assertions.assertEquals; +@EnabledIfSystemProperty(named = "accountName", matches = ".*", + disabledReason = "Make sure to supply azure accessKey or accountName, e.g: mvn verify -DaccountName=myacc -DaccessKey=mykey") @TestInstance(TestInstance.Lifecycle.PER_CLASS) class QueueConsumerIT extends CamelTestSupport { diff --git a/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/QueueProducerIT.java b/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/integration/QueueProducerIT.java similarity index 93% rename from components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/QueueProducerIT.java rename to components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/integration/QueueProducerIT.java index a404b55..5db9801 100644 --- a/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/QueueProducerIT.java +++ b/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/integration/QueueProducerIT.java @@ -14,7 +14,7 @@ * See the License for the specific language governing permissions and * limitations under the License. */ -package org.apache.camel.component.azure.storage.queue; +package org.apache.camel.component.azure.storage.queue.integration; import java.util.List; import java.util.Map; @@ -29,6 +29,9 @@ import org.apache.camel.EndpointInject; import org.apache.camel.ExchangePattern; import org.apache.camel.ProducerTemplate; import org.apache.camel.builder.RouteBuilder; +import org.apache.camel.component.azure.storage.queue.QueueConfiguration; +import org.apache.camel.component.azure.storage.queue.QueueConstants; +import org.apache.camel.component.azure.storage.queue.QueueTestUtils; import org.apache.camel.component.azure.storage.queue.client.QueueClientFactory; import org.apache.camel.component.azure.storage.queue.client.QueueServiceClientWrapper; import org.apache.camel.component.mock.MockEndpoint; @@ -37,10 +40,13 @@ import org.apache.commons.lang3.RandomStringUtils; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestInstance; +import org.junit.jupiter.api.condition.EnabledIfSystemProperty; import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertTrue; +@EnabledIfSystemProperty(named = "accountName", matches = ".*", + disabledReason = "Make sure to supply azure accessKey or accountName, e.g: mvn verify -DaccountName=myacc -DaccessKey=mykey") @TestInstance(TestInstance.Lifecycle.PER_CLASS) class QueueProducerIT extends CamelTestSupport { diff --git a/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/operations/QueueOperationsIT.java b/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/operations/integration/QueueOperationsIT.java similarity index 96% rename from components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/operations/QueueOperationsIT.java rename to components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/operations/integration/QueueOperationsIT.java index 1bc3667..ae6564b 100644 --- a/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/operations/QueueOperationsIT.java +++ b/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/operations/integration/QueueOperationsIT.java @@ -14,7 +14,7 @@ * See the License for the specific language governing permissions and * limitations under the License. */ -package org.apache.camel.component.azure.storage.queue.operations; +package org.apache.camel.component.azure.storage.queue.operations.integration; import java.time.Duration; import java.util.List; @@ -32,6 +32,8 @@ import org.apache.camel.component.azure.storage.queue.QueueTestUtils; import org.apache.camel.component.azure.storage.queue.client.QueueClientFactory; import org.apache.camel.component.azure.storage.queue.client.QueueClientWrapper; import org.apache.camel.component.azure.storage.queue.client.QueueServiceClientWrapper; +import org.apache.camel.component.azure.storage.queue.operations.QueueOperationResponse; +import org.apache.camel.component.azure.storage.queue.operations.QueueOperations; import org.apache.camel.support.DefaultExchange; import org.apache.camel.test.junit5.CamelTestSupport; import org.apache.commons.lang3.RandomStringUtils; @@ -39,6 +41,7 @@ import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestInstance; +import org.junit.jupiter.api.condition.EnabledIfSystemProperty; import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertFalse; @@ -46,6 +49,8 @@ import static org.junit.jupiter.api.Assertions.assertNotNull; import static org.junit.jupiter.api.Assertions.assertThrows; import static org.junit.jupiter.api.Assertions.assertTrue; +@EnabledIfSystemProperty(named = "accountName", matches = ".*", + disabledReason = "Make sure to supply azure accessKey or accountName, e.g: mvn verify -DaccountName=myacc -DaccessKey=mykey") @TestInstance(TestInstance.Lifecycle.PER_CLASS) class QueueOperationsIT extends CamelTestSupport { diff --git a/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/operations/QueueServiceOperationsIT.java b/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/operations/integration/QueueServiceOperationsIT.java similarity index 89% rename from components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/operations/QueueServiceOperationsIT.java rename to components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/operations/integration/QueueServiceOperationsIT.java index 94a395c..818ee48 100644 --- a/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/operations/QueueServiceOperationsIT.java +++ b/components/camel-azure/camel-azure-storage-queue/src/test/java/org/apache/camel/component/azure/storage/queue/operations/integration/QueueServiceOperationsIT.java @@ -14,7 +14,7 @@ * See the License for the specific language governing permissions and * limitations under the License. */ -package org.apache.camel.component.azure.storage.queue.operations; +package org.apache.camel.component.azure.storage.queue.operations.integration; import java.util.List; import java.util.Properties; @@ -26,15 +26,20 @@ import org.apache.camel.component.azure.storage.queue.QueueConfiguration; import org.apache.camel.component.azure.storage.queue.QueueTestUtils; import org.apache.camel.component.azure.storage.queue.client.QueueClientFactory; import org.apache.camel.component.azure.storage.queue.client.QueueServiceClientWrapper; +import org.apache.camel.component.azure.storage.queue.operations.QueueOperationResponse; +import org.apache.camel.component.azure.storage.queue.operations.QueueServiceOperations; import org.apache.commons.lang3.RandomStringUtils; import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestInstance; +import org.junit.jupiter.api.condition.EnabledIfSystemProperty; import static org.junit.jupiter.api.Assertions.assertNotNull; import static org.junit.jupiter.api.Assertions.assertTrue; +@EnabledIfSystemProperty(named = "accountName", matches = ".*", + disabledReason = "Make sure to supply azure accessKey or accountName, e.g: mvn verify -DaccountName=myacc -DaccessKey=mykey") @TestInstance(TestInstance.Lifecycle.PER_CLASS) class QueueServiceOperationsIT {