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

Expand MongoDb test coverage #2622

Closed
JiriOndrusek opened this issue May 19, 2021 · 4 comments · Fixed by #2642
Closed

Expand MongoDb test coverage #2622

JiriOndrusek opened this issue May 19, 2021 · 4 comments · Fixed by #2642
Assignees
Milestone

Comments

@JiriOndrusek
Copy link
Contributor

JiriOndrusek commented May 19, 2021

Add additional test coverage for:

  • Configuration of database in Spring XML. EDITED
  • Override an operation with dynamic operation. (Producer)
  • Tailable cursor consumer
  • Tailable cursor consumer with persistent tail tracking
  • Change streams consumer
  • Included converter
@JiriOndrusek JiriOndrusek self-assigned this May 19, 2021
@jamesnetherton
Copy link
Contributor

I think we can leave out testing for the Spring XML config.

@JiriOndrusek
Copy link
Contributor Author

JiriOndrusek commented May 19, 2021

@jamesnetherton this was my idea either. But it could be theoretically possible and because it is the first chapter in component doc, I kept it there. But if standard way is to ignore this and go only with quarkus configuration (through properties), no problem here.

@jamesnetherton
Copy link
Contributor

go only with quarkus configuration

Yeah that's the default test strategy. A lot of the component docs will mention Spring XML, but it's not our focus here so much. Also, not all of the Spring stuff will work since we do not fully support the framework.

@JiriOndrusek
Copy link
Contributor Author

ok, then I'm remopving this item from the summary, thanks for confirmation.

JiriOndrusek added a commit to JiriOndrusek/camel-quarkus that referenced this issue May 20, 2021
JiriOndrusek added a commit to JiriOndrusek/camel-quarkus that referenced this issue May 20, 2021
JiriOndrusek added a commit to JiriOndrusek/camel-quarkus that referenced this issue May 20, 2021
JiriOndrusek added a commit to JiriOndrusek/camel-quarkus that referenced this issue May 21, 2021
JiriOndrusek added a commit to JiriOndrusek/camel-quarkus that referenced this issue May 21, 2021
JiriOndrusek added a commit to JiriOndrusek/camel-quarkus that referenced this issue May 21, 2021
ppalaga pushed a commit that referenced this issue May 24, 2021
@ppalaga ppalaga added this to the 2.0.0-M1 milestone Jun 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants