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

Possibility to reset remote connection #2937

Open
juhoarvid opened this issue Aug 21, 2018 · 6 comments
Open

Possibility to reset remote connection #2937

juhoarvid opened this issue Aug 21, 2018 · 6 comments

Comments

@juhoarvid
Copy link

There would be need to do reset for remote connection at least in following conditions:

  • remote server is requested to do restart for testing purpose
  • remote server is requested to do something which is expected to broke remote connection for a while
juhoarvid pushed a commit to juhoarvid/robotframework that referenced this issue Aug 21, 2018
- remote library instace connection can be resetted
- needs remote library used in own library, no possibility to straight
  call "Reset Connection" kw from robot scripts
@juhoarvid
Copy link
Author

Any comments or review for:
#2938

@pekkaklarck
Copy link
Member

I guess this could be useful. There are some problems with the code in the PR that ought to be pretty easy to fix. More importantly it would need tests and documentation.

@juhoarvid
Copy link
Author

I guess this could be useful. There are some problems with the code in the PR that ought to be pretty easy to fix. More importantly it would need tests and documentation.

Can I do something? Is there documentation rule follow, test procedure to execute and descritption what should be fixed?

@pekkaklarck
Copy link
Member

CONTRIBUTING.rst in the project root ought to get you started with tests and docs. I can help more here or on the #devel channel on Slack if needed.

Did you try would Reload Library keyword work in your case? Is that a different use case?

@juhoarvid
Copy link
Author

Short check with "Reload Library" shows it is working a bit differently, but may lead to same situation. Will need a bit more time that I can verify it fulfills my need

@pekkaklarck
Copy link
Member

What's the situation with this issue/PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants