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

camel-java quickstart has no test coverage #146

Closed
rsvoboda opened this issue Apr 11, 2019 · 2 comments
Closed

camel-java quickstart has no test coverage #146

rsvoboda opened this issue Apr 11, 2019 · 2 comments
Labels
good first issue Good for newcomers

Comments

@rsvoboda
Copy link
Member

camel-java quickstart has no test coverage
@rsvoboda rsvoboda added the good first issue Good for newcomers label Apr 11, 2019
@ppalaga
Copy link
Contributor

ppalaga commented Apr 25, 2019

@rsvoboda I could eventually try to fix this. Could you please point me at a quickstart whose tests are especially worth to be used as an inspiration?

@ppalaga
Copy link
Contributor

ppalaga commented Apr 25, 2019

@rsvoboda unping. I see that all other quickstarts' tests look the same, mostly testing over HTTP. The problem with this camel quickstart is that it does not expose any endpoint. It is just a timer based loop writing something to the log. Let me figure out if there is a way to access the log from a test.

ppalaga added a commit to ppalaga/quarkus-quickstarts that referenced this issue Apr 25, 2019
@ppalaga
Fix quarkusio#146 camel-java quickstart has no test coverage
4a146e5
rsvoboda added a commit that referenced this issue Apr 26, 2019
@rsvoboda
Merge pull request #167 from ppalaga/i146
53e69ce 
Fix #146 camel-java quickstart has no test coverage
ppalaga added a commit to ppalaga/quarkus-quickstarts that referenced this issue Apr 26, 2019
@ppalaga
Fixup quarkusio#146 camel-java quickstart has no test coverage
712e181
rsvoboda added a commit that referenced this issue Apr 26, 2019
@rsvoboda
Merge pull request #169 from ppalaga/i146.1
ecb988e 
Fixup #146 camel-java quickstart has no test coverage
lordofthejars pushed a commit to lordofthejars/quarkus-quickstarts that referenced this issue May 11, 2019
@ppalaga @lordofthejars
Fix quarkusio#146 camel-java quickstart has no test coverage
ad30887
lordofthejars pushed a commit to lordofthejars/quarkus-quickstarts that referenced this issue May 11, 2019
@ppalaga @lordofthejars
Fixup quarkusio#146 camel-java quickstart has no test coverage
5c35361
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rsvoboda @ppalaga