robotframework / JavalibCore

Base for implementing Java test libraries to be used with Robot Framework
Other
42 stars 25 forks source link

Update ArgumentConverter.java #15

Closed eramitmittal closed 4 years ago

eramitmittal commented 9 years ago

Allowing ${None} to be passed to keywords. currently it results in null pointer exception

jussimalinen commented 8 years ago

As we havent touched this library in a long time, I sort of forgot this issue. I'll make a new release with this before the end of the year.

eramitmittal commented 7 years ago

Hi,

Any news on this?

Thanks, Amit

hleskien commented 4 years ago

Hi, please integrate this pull request. It may not fix all NullPointerExceptions, but at least it's a start. Thank you, Holger

pekkaklarck commented 4 years ago

Unfortunately this tool hasn't been actively maintained lately. Would be great if someone could take a look at the PRs and create a new release. Have you @mihaiparvu, or your team, looked at JavalibCore at all? It's used by SwingLibrary isn't it? How about @Hi-Fi?

Hi-Fi commented 4 years ago

Sorry, I missed this completely when checked the code.

All my libraries are using this, and there are some things I have thought that need fixing for some time. E.g. argument parsing.

pekkaklarck commented 4 years ago

Are you @Hi-Fi interested in taking maintainer responsibilities, possibly with help from @mihaiparvu? I'm sure @jussimalinen can help getting started if needed.

I doubt there's that much work needed in general. First thing would be going through the issues and PRs here and possibly creating a release. RF 3.1 also introduced some new features to the library API such type support and named-only arguments. Latter releases could add support for them if they make sense in this context.

mihaiparvu commented 4 years ago

@pekkaklarck JavaLibCore is indeed used by SwingLibrary, but we didn't touch it at all. I don't think we even have maintainer rights on that repo. We could help test future versions with SwingLibrary, but don't have much time for development these days.

Hi-Fi commented 4 years ago

@pekkaklarck I can try at least.

Hi-Fi commented 4 years ago

ArgumentCollector removed, so this is not valid anymore.