jboss-fuse / wsdl2rest

A tool allowing quick migration from existing JAX-WS services to REST
Apache License 2.0
25 stars 22 forks source link

issue60 - cannot obtain java method error with calculator.wsdl #63

Closed bfitzpat closed 5 years ago

bfitzpat commented 5 years ago

-- created JUnit test to reproduce issue -- fixed issue by making operation name conversion case insensitive

Signed-off-by: Brian Fitzpatrick bfitzpat@redhat.com

bfitzpat commented 5 years ago

@tdiesler - the issue in this case was the code was looking for "add" when the method name came back as "Add" - simple fix and doesn't seem to break anything else. Added a new test to slowly build up the library of wsdls we show support for.

tdiesler commented 5 years ago

ok to test

apupier commented 5 years ago

I don't reproduce the error from the CI. I tried to launch through Maven and through IDE Junit Test, test is passing in both cases. In the test, it seems that the test cannot find the wsdl test file which is in src/test/resources

jamesnetherton commented 5 years ago

Probably because on *nix, filenames are case sensitive. I.e Calculator.wsdl vs calculator.wsdl.

bfitzpat commented 5 years ago

Looks like this was merged by fusesource-ci, so will close it.