Please cleanup the PR.

1. The robot framework output files and the geckodriver.log should never be part of the PR.
2. The Approval tests approved files should have not been changed.
3. There is failing unit test. Because of the added argument, mock does not match, please fix it.

And write tests for the change.

@colourmein13 do you still plan to work with the PR?

Closing due inactivity. Can be reopened if there is a need.

Closing due inactivity. Can be reopened if there is a need.

Hey. Apologies as I got busy with other stuff.

1. Should I add .log to my .gitignore? What output files are you talking about?
2. I haven't changed the content, but I see what you mean.
3. I have fixed the failed unittest. Should I create new PR for the new tests?

No worries, we do not have schedules and working in your own pace is always recommended.

1. Modifications to .gitignore should not be needed. It looks like you do not use atest/run.py to run the tests and therefore output files are not created in the already ignored folder. There is a readme file which explain in more details how running tests works out of the box.
2. It looks like you have deleted to whole branch from you fork and therefore I can not reopen this PR. I think most easiest way is to create the new PR whit all the changed needed.