Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Split Quartz test into Quartz and Cron #3086

Closed
ppalaga opened this issue Sep 9, 2021 · 1 comment · Fixed by #3090
Closed

Split Quartz test into Quartz and Cron #3086

ppalaga opened this issue Sep 9, 2021 · 1 comment · Fixed by #3090
Assignees
@ppalaga
Labels
test
Milestone
No fix/wont't fix

Comments

@ppalaga
Copy link
Contributor

ppalaga commented Sep 9, 2021

It is better to test those extensions in isolation.
@ppalaga ppalaga added the test label Sep 9, 2021
@ppalaga ppalaga self-assigned this Sep 9, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Sep 9, 2021
@ppalaga
Split Quartz test into Quartz and Cron apache#3086
d652fae
@ppalaga ppalaga mentioned this issue Sep 9, 2021
Merged
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Sep 10, 2021
@ppalaga
Split Quartz test into Quartz and Cron apache#3086
30be5ca
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Sep 10, 2021
@ppalaga
Split Quartz test into Quartz and Cron apache#3086
ec1bf6b
@ppalaga
Copy link
Contributor Author

ppalaga commented Sep 10, 2021

Oh, there is no point in splitting these, because as stated in ​https://camel.apache.org/components/latest/cron-component.html
Cron is an interface component, that does not contain a default implementation, instead it requires that the users plug the implementation of their choice, such as quartz.

@ppalaga ppalaga added this to the No fix/wont't fix milestone Sep 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ppalaga ppalaga

Labels
test
Projects
None yet
Milestone
No fix/wont't fix
Development

Successfully merging a pull request may close this issue.

Remove superfluous xml-jaxb dependency from the REST test ppalaga/camel-quarkus
1 participant
@ppalaga