Skip to content
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

Add get_keyword_tags to dynamic library API and remote interface #37

Closed
MikkoLeppaaho opened this issue Mar 16, 2017 · 11 comments
Closed

Comments

@MikkoLeppaaho
Copy link

Regarding the Get Keyword Tags -change in RF (robotframework/robotframework#2538)
As you probably had noticed, the remote api in latest robot framework (3.0.2) contains this new feature that is currently not supported by Robot Java Remote Server. Lack of this support seems not to be a really a big deal at least for our test environments as the only thing at the moment that this causes is the annoying error: ERROR org.apache.xmlrpc.server.XmlRpcErrorLogger - No such handler: get_keyword_tags which visible in the logs. Currently we use the JMSLibrary with the Robot Remote Server.

Of course in the future this support might be handy so if you have any spare time to spend with the project to add this, it would be great :-)

@pekkaklarck
Copy link
Member

Do you get that error only once or multiple times? If multiple, we should change the Remote library itself to just try once. Please submit an issue to Robot Framework's tracker in that case.

@MikkoLeppaaho
Copy link
Author

When the remote server is up and running (we use simple bat file to launch it), the error comes each time the pybot (from Ride or cmd line) is launched with test suite that uses the remote library. In our test suites the test suite setup contains keywords that init the jms library connection and that's the first point where remote is used (at least I think so) and that seems to be the exact moment when error pops up in java remote server.

@maxdesp
Copy link

maxdesp commented Dec 19, 2017

I have the same error with robot framework 3.0.2, which version should I use to avoid server crashing on running tests ?

@ombre42
Copy link
Collaborator

ombre42 commented Feb 22, 2018

@maxdesp is the server really crashing due to this or is it just an error?

@moijes12
Copy link

moijes12 commented Apr 4, 2018

@ombre42 I don't see the server crashing. The called keyword method returns the correct response but the exception trace is still there.

@karniemi
Copy link

karniemi commented Jan 23, 2019

@pekkaklarck @ombre42 @rainmanwy : ...we are seeing the exception 10s/100s of times per test run with jrobotremoteserver...and it is pretty ugly. It would be really nice if either the method is added to jrobotremoteserver or if the robotframework stopped complaining about it( as the method was supposed to be optional anyway...).

As a consequence, people get mixed up, and think that it's critical failure...which causes mess like this: rainmanwy/robotframework-SikuliLibrary#33 ....where we can see that some unrelated real connection problems are thought as related to this ugly message.

(BTW: ...there seems to be an implementation for this already in: aenniw#14 ...though...that fork is already some 60 commits ahead of upstream...)

@MichaelHallik
Copy link

MichaelHallik commented Jan 7, 2020

Issue still relevant with RF 3.1.2 (on Python 3.7.6) and jrobotremote 3.0.

EDIT: at my current assignment (Dutch tax authority) this issue is actually a reason for them to abandon the Java database library and promote the Python database lib for any future projects/teams.

@Hi-Fi
Copy link
Collaborator

Hi-Fi commented Jan 18, 2020

Exception should not be there with version 4.

The response with the call is empty, as JavalibCore is not having annotation to give tags.

@karniemi
Copy link

Exception should not be there with version 4.

The response with the call is empty, as JavalibCore is not having annotation to give tags.

version 4 of which piece of sw?

@Hi-Fi
Copy link
Collaborator

Hi-Fi commented Jan 18, 2020

Jrobotremoteserver 4.0.0

@Hi-Fi
Copy link
Collaborator

Hi-Fi commented Mar 24, 2022

Included in version 4.0.0, closing.

@Hi-Fi Hi-Fi closed this as completed Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants