Skip to content

Minor updates in the Java Backend API to improve reusability #257

Closed
wants to merge 1 commit into from

2 participants

@jeromebenois

Could you open more your Java Backend API in order to improve reusability? I need to call two internal methods in my Cucumber's Play'Framework plugin.

Thanks,
Jérôme.

@aslakhellesoy
Cucumber member

Thanks for that Jérôme. Is your Cucumber Play plugin source available anywhere?

@jeromebenois

Yes I plan to publish next week on my GitHub on Apache2 license. Are you interested?

@aslakhellesoy
Cucumber member

I won't be using Play, but I'm interested in looking at the code.

@aslakhellesoy
Cucumber member

@jeromebenois I ended up doing this a little differently. I don't want to open up too much of the internals because it makes it harder to refactor in the future.

Now you should be able to simply do this in your PlayBackend (not tested):

for (Method method : glueCodeClass.getMethods()) {
    loadGlue(glue, method);
}           

Does that work for you?

@hutchy2570 hutchy2570 added a commit to lgi-kitd/cucumber-jvm that referenced this pull request Mar 21, 2012
@aslakhellesoy aslakhellesoy Don't allow inheritance in glue code (stepdefs and hooks). Also fixes #…
…257 in a more encapsulated way.
4a718d8
@jeromebenois

Thanks, It works !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.