-
Notifications
You must be signed in to change notification settings - Fork 150
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
Fix 368 by also matching io.cucumber
annotations and interfaces
#369
Conversation
Just tried a local
So have not been able to test this yet. Any idea how to resolve? Do I need to update the binaries? Nvm: Looks to be Java 11 related; I'll see what I can fix locally. |
Great ! And welcome :)
It's done. 👍
Unfortunally, the plugin does not have many tests. Most of them are done manually. When I try the plugin manually, I generally play with
Since you introduce the support of the latest cucumber, the job should be done with the same samples with both cucmber v4 and v5.
Simply build it with JDK1.8 for now. |
Hmm; tried it just now on the calculator examples, but it appears the new step definitions are not yet picked up. I also suddenly had to configure my projects with the Cucumber nature, which is also a first for me. Any idea what might be going on? My debug options/skills for a plugin a pretty limited. |
// Then set Language of Java8-cuke-api | ||
if (decl.getElementName().matches(REGEX_JAVA8_CUKEAPI)) { | ||
String importDeclaration = decl.getElementName(); | ||
// setJava8CukeLang(importDeclaration); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This code appeared left over from before; Hope it's OK to remove.
As discussed in slack, the testing issues might seem from having to match cucumber expressions in the latest calculator examples, rather than regular expressions in a previous iteration. Not confirmed yet, but something to be aware of in testing this branch. |
@timtebeek could you create a ZIP archive for a local installation? I have V 1.0.0.201907020018 that is not working. |
@RusmanCool Unfortunately my changes are not working for me either; so a zip won't help you much. If you know how to fix the code please do! I'm at a loss here, and don't have the time to dive in fully. |
Are there any ongoing actions to correct this problem? because this is blocking for all new version (io.cucumber.*) |
@timtebeek Thanks for your contribution ! @sgrillon14 You can use the snapshot update-site to get this fix. |
@qvdk , where is the delivery please? i see https://marketplace.eclipse.org/content/cucumber-eclipse-plugin but last delivery is : |
I find https://github.com/cucumber/cucumber-eclipse-update-site-snapshot and it work update. But plugin do not work |
it's already better, the plugin finds the steps that are in the project. I still have a problem, it can't find the steps that are in a dependency (Maven) that contains generic steps. Reproduce my problem:
I try add package in Cucumber plugin configuration but do not work: |
As discussed in #368 ; First time developing on a plugin; Not tested yet but code seems straight forward. What's the flow needed to try this out?