Add expect_tests and testing_support to infra recipe.

Add expect_tests and testing_support to infra recipe.

For more better access :)

R=eseidel@chromium.org, stip@chromium.org
BUG=

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=290190

[ghost stip (do not use), 2014-08-12 21:12:00]

nuts but lgtm

https://chromiumcodereview.appspot.com/466923002/diff/1/recipes/infra.py
File recipes/infra.py (right):

https://chromiumcodereview.appspot.com/466923002/diff/1/recipes/infra.py#newc...
recipes/infra.py:17: solution = lambda name, path_infix = None: {
remove spaces so you get path_infix=None

https://chromiumcodereview.appspot.com/466923002/diff/1/recipes/infra.py#newc...
recipes/infra.py:17: solution = lambda name, path_infix = None: {
this is kind of nuts imo

[iannucci, 2014-08-17 17:53:00]

The CQ bit was checked by iannucci@chromium.org

[commit-bot: I haz the power, 2014-08-17 17:53:14]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/iannucci@chromium.org/466923002/1

[commit-bot: I haz the power, 2014-08-17 17:55:05]

Committed patchset #1 (1) as 290190

[eseidel, 2014-08-18 15:47:40]

What is this for?  Why do we need to modify the defauld "fetch infra" checkout?
Why wouldn't "fetch infra" want these?  Or a "fetch infra_developer" or some way
to make whatever you're trying to do easy for humans to do?

[iannucci, 2014-08-18 16:56:43]

On 2014/08/18 15:47:40, eseidel wrote:
> What is this for?  Why do we need to modify the defauld "fetch infra"
checkout?
> Why wouldn't "fetch infra" want these?  Or a "fetch infra_developer" or some
way
> to make whatever you're trying to do easy for humans to do?

It's so that `fetch infra

[iannucci, 2014-08-18 16:57:26]

Oops, sent too early.

It's so that `fetch infra` gets all of the repos that are maintained by infra.
Is there a problem with fetching these other repos?

[eseidel, 2014-08-18 16:59:14]

Sorry, I am confusing recipies.  I thought this was the try-job recipe we wrote
which you were modifying to do more than 'fetch infra' did, but only on the bots
(which seemed bad).  This is the 'fetch infra' recipe instead?

[iannucci, 2014-08-18 16:59:51]

On 2014/08/18 16:59:14, eseidel wrote:
> Sorry, I am confusing recipies.  I thought this was the try-job recipe we
wrote
> which you were modifying to do more than 'fetch infra' did, but only on the
bots
> (which seemed bad).  This is the 'fetch infra' recipe instead?

Yes, this is the depot_tools fetch recipe for infra.

[iannucci, 2014-08-18 17:00:08]

On 2014/08/18 16:59:51, iannucci wrote:
> On 2014/08/18 16:59:14, eseidel wrote:
> > Sorry, I am confusing recipies.  I thought this was the try-job recipe we
> wrote
> > which you were modifying to do more than 'fetch infra' did, but only on the
> bots
> > (which seemed bad).  This is the 'fetch infra' recipe instead?
> 
> Yes, this is the depot_tools fetch recipe for infra.

(for humans, not for bots)

[iannucci, 2014-08-18 17:00:47]

On 2014/08/18 17:00:08, iannucci wrote:
> On 2014/08/18 16:59:51, iannucci wrote:
> > On 2014/08/18 16:59:14, eseidel wrote:
> > > Sorry, I am confusing recipies.  I thought this was the try-job recipe we
> > wrote
> > > which you were modifying to do more than 'fetch infra' did, but only on
the
> > bots
> > > (which seemed bad).  This is the 'fetch infra' recipe instead?
> > 
> > Yes, this is the depot_tools fetch recipe for infra.
> 
> (for humans, not for bots)

((and yes, having them both be called recipes was confusing, though at the time
we thought they'd converge... optimism, eh?))