robotframework / SeleniumLibrary

Web testing library for Robot Framework
Apache License 2.0
1.36k stars 751 forks source link

Discuss/Consider using a dictionary or single argument for timing parameters on library import #1830

Open emanlove opened 1 year ago

emanlove commented 1 year ago

With the recent addition of the page load timeout [#1535] and the action chain delay [#1768] parameters along with the upcoming addition of waiting for expected conditions [#1827] and getters for the timeout settings [#1825], I raised up the question of whether or not we have to many library import arguments. In the same light as the way multiple timeout settings may be inquired, could we make the timeout / timing / delay attributes a single argument, maybe as a dictionary, that can be passed to the library import.

Snooz82 commented 9 months ago

I personally would consider a dictionary an unnecessary complexity.

multiple arguments are specially for library inits better for users.