MuraliKola / robotframework-seleniumlibrary

Automatically exported from code.google.com/p/robotframework-seleniumlibrary
Apache License 2.0
1 stars 0 forks source link

It should be possible to initialize the library with a full url #181

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
It should be possible to initialize the library with a full url e.g.:

http://192.168.52.1:4444

The background is that we start the selenium server with the jenkins 
seleniumrc-plugin and recive the selenium-variable in the form described above.

Original issue reported on code.google.com by nicolai....@gmail.com on 10 Mar 2011 at 12:01

GoogleCodeExporter commented 8 years ago
Sounds fine to me. I think this should work so that the server_host argument 
given to the library could be either just host (current behavior), host:port, 
or http://host:port. The library itself needs host and port separately but 
splitting them from these formats is easy.

The library also takes separate server_port argument. If this port is given, it 
should have precedence over the port given as part of the server_host argument.

Are you nicolai (I don't see the whole name) interested in implementing this?

Original comment by pekka.klarck on 10 Mar 2011 at 12:12

GoogleCodeExporter commented 8 years ago
Hello Pekka,

Yes, I am interested in this feature being implemented.
And I would implement this enhancement by myself, but I'm too busy at the 
moment, sorry...

Another additional note: I forgot to mention that the url we recieve via the 
selenium-variable ends up with a "/" e.g: http://192.168.52.1:4444/

Original comment by nicolai....@gmail.com on 10 Mar 2011 at 2:17

GoogleCodeExporter commented 8 years ago
It's unlikely that the core development team has time to implement this either 
in the near future so let us know when you have some time available. The URL 
can be parsed e.g. with the urlparse module so possible slashes at the end are 
easy to handle.

Original comment by pekka.klarck on 10 Mar 2011 at 7:37

GoogleCodeExporter commented 8 years ago

Original comment by janne.t....@gmail.com on 27 Apr 2011 at 11:40

GoogleCodeExporter commented 8 years ago
This issue was updated by revision 8ab90c4cfaa0.

Original comment by janne.t....@gmail.com on 29 Apr 2011 at 10:38

GoogleCodeExporter commented 8 years ago

Original comment by pekka.klarck on 30 Apr 2011 at 9:19