[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #973: Clean up poms: introduce test BOM, manage only where necessary

[GitBox <gi...@apache.org>]

ppalaga commented on a change in pull request #973: Clean up poms: introduce test BOM, manage only where necessary
URL: https://github.com/apache/camel-quarkus/pull/973#discussion_r397664603
 
 

 ##########
 File path: integration-tests/aws/pom.xml
 ##########
 @@ -39,6 +39,18 @@
         <mvnd.builder.rule>camel-quarkus-aws-ec2-deployment,camel-quarkus-aws-ecs-deployment,camel-quarkus-aws-eks-deployment,camel-quarkus-aws-iam-deployment,camel-quarkus-aws-kinesis-deployment,camel-quarkus-aws-kms-deployment,camel-quarkus-aws-lambda-deployment,camel-quarkus-aws-s3-deployment,camel-quarkus-aws-sns-deployment,camel-quarkus-aws-sqs-deployment,camel-quarkus-aws-translate-deployment,camel-quarkus-log-deployment,camel-quarkus-support-policy-deployment,camel-quarkus-timer-deployment</mvnd.builder.rule>
     </properties>
 
+    <dependencyManagement>
+        <dependencies>
+            <dependency>
+                <groupId>org.apache.camel.quarkus</groupId>
+                <artifactId>camel-quarkus-bom-test</artifactId>
+                <version>${project.version}</version>
+                <type>pom</type>
+                <scope>import</scope>
+            </dependency>
+        </dependencies>
+    </dependencyManagement>
 
 Review comment:
   I thought of that option and I think that this way is better: We now have itests also under `extensions-jvm/*/integration-test`. Those will be easier to move to `integration-tests` when all tests are self-contained.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services