Chris Manchester (limited bugmail, email directly) Reporter
	Description • 11 years ago

This looks like the way to do it.

	Andrew Halberstadt [:ahal] Assignee
	Comment 2 • 11 years ago

Unfortunately this depends on bug 907895 landing on every single branch :(

We should probably get these lists from the in-tree config to avoid situations like this in the future.

	Andrew Halberstadt [:ahal] Assignee
	Comment 3 • 11 years ago

Comment on attachment 794060 [details] [diff] [review]
Install moztest in desktop and b2g mozharness scripts;

Review of attachment 794060 [details] [diff] [review]:
-----------------------------------------------------------------

This is good, but we'll need to wait on the dependency, so we can't land it yet.

	Andrew Halberstadt [:ahal] Assignee
	Comment 4 • 11 years ago

The previous patch can't land because the moztest fix couldn't be easily landed on the release branch. Instead we should emulate the solution used for the marionette script in bug 908356.

Ash run going here:
https://tbpl.mozilla.org/?tree=Ash&rev=31d60358dfc7

	Jonathan Griffin (:jgriffin)
	Comment 5 • 11 years ago

Comment on attachment 804665 [details] [diff] [review]
Patch 1.0 - use in tree requirements for desktop and b2g unittests

Review of attachment 804665 [details] [diff] [review]:
-----------------------------------------------------------------

::: scripts/desktop_unittest.py
@@ +195,5 @@
> +            self.register_virtualenv_module(m, url=os.path.join(mozbase_dir,
> +                m))
> +
> +        self.register_virtualenv_module('marionette', url=os.path.join('tests',
> +            'marionette'))

We don't use Marionette in desktop unit tests at all.

	Andrew Halberstadt [:ahal] Assignee
	Comment 6 • 11 years ago

I've removed the marionette line in my local patch. Missed an import though, new ash run:
https://tbpl.mozilla.org/?tree=Ash&rev=cfb9ed59d9da

	Andrew Halberstadt [:ahal] Assignee
	Comment 7 • 11 years ago

Ash run looks good:

https://hg.mozilla.org/build/mozharness/rev/df99445c2aba