Open Bug 1480337 Opened 2 years ago Updated 1 month ago

[wdspec] Move shared methods into file of appropriate command


(Testing :: geckodriver, defect, P3)

Version 3


(Not tracked)


(Reporter: whimboo, Unassigned)



(1 file)

The tests for all the commands specified by the WebDriver spec make use of shared methods inside their folders. To not have to duplicate those methods across all test files, move them into the file.
Comment on attachment 8997431 [details]
Bug 1480337 - [wdspec] Move shared methods into file for each command.

There’s a lot of test failures because the HTTP request functions
carry the same name as the module they’re in.  If you can solve
that, this looks OK.
Attachment #8997431 - Flags: review?(ato) → review+
The underlying problem here is actually not that the shared methods have the same name as the module, but that the fixtures don't care about the imported method, but try to make use of the global module. I assume that this happens because of the way how those are executed by pytest. Also it could mean that this is a bug in pytest. I will investigate and file a new issue against pytest if necessary.
Also please note that I'm not able to see those failures locally when I run the tests with `mach`. So maybe there is a different version of pytest in use, which might mean a new regression or a fix in one of the latest releases.

Currently I'm not working on this bug.

Assignee: hskupin → nobody
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.