Add SeleniumLibrary Set Screenshot Directory keyword counterpart

Is your feature request related to a problem? Please describe.

I am working in a project, where we would like to redefine the output directory of Take Screenshot keyword. Currently the path can´t be changed. Therefore we would like the library to have a common keyword to set the screenshot directory path.

Describe the solution you’d like

A SeleniumLibrary Set Screenshot Directory keyword counterpart could to be added to Browser library. Then it would be possible to define needed screenshot related setting with two library keywords. Like this:

Configure Screenshot Settings
   Browser.Set Screenshot Directory    ${WANTED_SCREENSHOT_DIRECTORY_PATH}
   Browser.Take Screenshot             filename=robotframework-browser-screenshot-{index}

Describe alternatives you’ve considered

Adding a path argument to Take Screenshot keyword could also be an option.

[leeuwe]

Hi @Virvatuli. In my experience Browser.Take Screenshot already takes a full path, meaning you can easily create your own userkeyword to achieve this, like:

Make Screenshot
    [arguments]       ${filename}=robotframework-browser-screenshot-{index}     ${selector}=None
    ${ImgDir}=        Set Variable                  ${EXECDIR}${/}mydir${/}
    Take Screenshot   ${ImgDir}${filename}          ${selector}

[Virvatuli]

@leeuwe Thank you for your reply and advice on this matter. I managed to implement your given solution to our project and it works nicely. I didn´t realize you can do this by just reading the keyword’s documentation, but I am still a beginner.