Closed Bug 1340305 Opened 7 years ago Closed 7 years ago

Add docs (and make it easier) to run selenium tests locally

Categories

(Tree Management :: Treeherder, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: wlach, Assigned: davehunt)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

In bug 1339547, I added some selenium tests to Treeherder. We don't really have a good story for running them locally though. I think we need to do the following:

* Install known-good copies of firefox, geckodriver, and xvfb inside the vagrant environment using puppet,
* Add some testing glue to make sure that they're set up and used when runtests.sh is run.
Depends on: 1364045
(In reply to William Lachance (:wlach) (use needinfo!) from comment #0)
> * Install known-good copies of firefox, geckodriver, and xvfb inside the
> vagrant environment using puppet,

Since this bug was filed, Firefox now supports headless mode, so we can skip the xvfb nonsense :-)

The steps are therefore:
* Copy the geckodriver/Firefox download & MOZ_HEADLESS=1 steps from here: https://github.com/mozilla/treeherder/blob/6a132dee0c56b1c4d13da54bbc5ca56a00b777f4/bin/travis-setup.sh#L52-L62
* Add them to vagrant/setup.sh, adjusting slightly to ideally ensure they only run if changes are required
* Adjust runtests.sh to invoke the selenium tests too
* Update the testing docs here (for people who run the tests manually outside of runtests.sh): https://treeherder.readthedocs.io/common_tasks.html#running-the-tests
(In reply to Ed Morley [:emorley] from comment #1)
> * Add them to vagrant/setup.sh

Or for MOZ_HEADLESS/PATH adjustments: vagrant/env.sh


> adjusting slightly to ideally ensure they only run if changes are required

For the Firefox beta download this won't really be possible I guess, since it's not version-pinned.
Assignee: nobody → dave.hunt
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Blocks: 1401048
Blocks: 1416266
No longer depends on: 1364045
Component: Treeherder: Docs & Development → TreeHerder
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: