[GitHub] [camel-quarkus] ppalaga commented on issue #482: Fix #476 List itests in an XML file for the Quarkus platform
ppalaga commented on issue #482: Fix #476 List itests in an XML file for the Quarkus platform URL: https://github.com/apache/camel-quarkus/pull/482#issuecomment-559422901 > move support test extension in a dedicated support folder so main and core and future it tests are just like other tests What an elegant option! 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: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on issue #482: Fix #476 List itests in an XML file for the Quarkus platform
ppalaga commented on issue #482: Fix #476 List itests in an XML file for the Quarkus platform URL: https://github.com/apache/camel-quarkus/pull/482#issuecomment-559420070 Yes, potential merge conflicts in the XML file would be nice to avoid. The main motivation for me to keep the file in Git for now is that we have some irregularities in the `integration-tests` directory and the selection done by the plugin may get broken in the future. Having a chance to spot malign changes during the PR review could prevent that. - That's actually a sign that we should try to find a mechanism for exporting the tests that would be more robust than the current path matching. I vote for merging this PR as is and continue the discussion withing https://github.com/apache/camel-quarkus/issues/413 . Either the selection could work based on properties in the test POMs (the irregularities in main an core are still there) as you proposed or maybe we could put the aggregated tests into a new directory where we'd keep a strictly flat layout (so that the path matching would suffice). 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: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on issue #482: Fix #476 List itests in an XML file for the Quarkus platform
ppalaga commented on issue #482: Fix #476 List itests in an XML file for the Quarkus platform URL: https://github.com/apache/camel-quarkus/pull/482#issuecomment-559400271 > auto generated it should not be checked in I prefer to have it in git to be able to check anytime quickly what we export and also to have the history thereof. WDYT? Now fixing the JAXB issue on Java 11. 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: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on issue #482: Fix #476 List itests in an XML file for the Quarkus platform
ppalaga commented on issue #482: Fix #476 List itests in an XML file for the Quarkus platform URL: https://github.com/apache/camel-quarkus/pull/482#issuecomment-559390682 > be auto generated like all the it with a specific property are included (or excluded) (Not sure you noticed) it is currently autogenerated and there is some file path based filtering there. Is that not enough? https://github.com/apache/camel-quarkus/pull/482/files#diff-e0dff2635520bef5f237973025e0a295R51-R63 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: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on issue #482: Fix #476 List itests in an XML file for the Quarkus platform
ppalaga commented on issue #482: Fix #476 List itests in an XML file for the Quarkus platform URL: https://github.com/apache/camel-quarkus/pull/482#issuecomment-559389472 bd419b9 fixes the license header problem 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: us...@infra.apache.org With regards, Apache Git Services