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.

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

Revisit core, main and foundation integration tests #2362

Closed
ppalaga opened this issue Mar 23, 2021 · 1 comment · Fixed by #2597
Closed

Revisit core, main and foundation integration tests #2362

ppalaga opened this issue Mar 23, 2021 · 1 comment · Fixed by #2597
Assignees
Labels
Milestone

Comments

@ppalaga
Copy link
Contributor

ppalaga commented Mar 23, 2021

Things to improve:

  • Minimize the dependencies (timer, log might get removed from the core test)
  • Consider splitting some of the modules to Increase isolation, e.g. timer, and log do not have their isolated modules
  • Consider introducing a grouped test to execute all modules at once on the CI
  • Make sure we have both consumer and producer assertions for direct
@ppalaga ppalaga added the test label Mar 23, 2021
@ppalaga ppalaga added this to the 1.9.0 milestone Apr 20, 2021
@JiriOndrusek
Copy link
Contributor

@ppalaga I'd like to be sure, that I understand this issue correctly:

  • We should try to create separate test modules for e.g. log, timer and therefore remove this dependencies from core test
  • We should verify that producer and consumer cases are tested for direct
  • I probably don't understand what does grouped test mean in this case - do you mean to introduce grouped test, which will contain core tests and these separated ones (like timer, log, ..)?

@JiriOndrusek JiriOndrusek self-assigned this May 12, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue May 14, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue May 14, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue May 14, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue May 14, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue May 14, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue May 14, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue May 17, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue May 17, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue May 17, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue May 17, 2021
ppalaga added a commit that referenced this issue May 17, 2021
ppalaga added a commit that referenced this issue May 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants