Cucumber Scala - Java reflection in ScalaDsl - random test errors

[jkrawczyk]

The code introduced in the commit b589ced seems to have a bug in functionParams method.
The comment from the commit:

The functionParams method has to filter out JVM bridge methods to ensure
the correct version of the apply() method (with type parameter
information) is captured. Function objects only have one defined method
named apply() so taking the head of the list should always be safe.

It turns out that there may be more methods defined, e.g. inner, helper methods or synthetic methods when accessing Step's variables. In that case taking head after filtering out bridge methods is not enough. Cucumber tests fail randomly because getDeclaredMethods returns Methods without sorting or any particular order.

A quick solution is to add getName to filter but maybe there is a better fix for that.

f.getClass.getDeclaredMethods.filter(m => "apply".equals(m.getName) && !m.isBridge).head.getGenericParameterTypes

[manuelbernhardt]

+1 on this issue, it's rather an annoyance as it renders the Scala module unusable.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.