-
Notifications
You must be signed in to change notification settings - Fork 15
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
Feature Request/ Question: Run tests as Integration Tests #32
Comments
Bump to this, any inputs? |
You will have to clarify exactly what you want to be done differently? Do you just mean having failsafe run them rather than surefire? Or do you mean actually launching the produced application and testing against it? |
I mean the difference between |
Was I clear enough? Does that make sense? |
Sure, but I am very unlikely to have the time to look at something like that. |
@GregJohnStewart any progress on this topic? I would like to get into this topic, as I think cucumber is perfect for integration tests. |
I had some code working towards it, but never quite got it working, definitely worth taking another look at (esp with Quarkus 3), and could probably use some assistance getting over the issue I was having. @stuartwdouglas Any thoughts since it's not just me asking? |
I would also be happy to contribute. Is there maybe something in the Quarkus core that can be used as a reference to get started? Or do you mind sharing your code? |
https://github.com/GregJohnStewart/quarkus-cucumber The actual code for the extension is in the @stuartwdouglas Happen to have the command used to run these tests handy? Any other notes for testing? |
Also thinking this would be a good opportunity to add some examples and docs |
This looks like a class loading problem.
|
I do not think that this is a class loading problem. In EDIT: |
While at it, reformat and unify code.
While at it, reformat and unify code.
@turing85 Thanks for taking point! Excited to see it in place. It is interesting CDI is called for, apparently. |
@GregJohnStewart no problem 🙂 but as mentioned in the (updated) documentation... it's not perfect. Also, I'd like to refactor the test afterwards. I think we need multipel test modules in order to automate some testing. |
Hello,
Unfortunately, this is not working, and I looked at the code in CucumberQuarkusTest, it seems to be linked to this part of code:
The correction would be to
If you agree with this solution, I can provide a patch with changes explained Thank you |
any news on this? |
With the new @stuartwdouglas @christophd (and possibly @mkouba) can we get some traction on this issue? |
I feel that Cucumber lends itself to integration type testing. I believe, based on observation, that the tests run by this extension are run in a 'regular' test manner.
It would be fantastic for there to be a way to configure these tests to be run in an integration context.
For Maven, I imagine it would be a matter of tweaking the
pom
, but unsure what it would take in GradleThe text was updated successfully, but these errors were encountered: