Rely on Portage to run required stages before "test" stage.

Rely on Portage to run required stages before "test" stage.

Previously, the cros unit test function was telling Portage exactly which
flow stages to run leading up to the "test" stage by giving the "noauto"
FEATURE and specifying each stage explicitly.  This proved unnecessarily
brittle, so now we rely on Portage's default behavior to determine which
stages must be run before the requested "test" stage.  Also added explicit
"clean" stage after test to remove all temporary files from disk.

Change-Id: I7e0a9e0ecc29eee2b564e78f9078a8da43b96167

BUG=chromium-os:13634
TEST=./cros_run_unit_tests --board=x86-mario --packages=chromeos-wm

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=be2351f

[mtennant, 2011-04-07 17:00:46]

Please review.

[davidjames, 2011-04-07 17:02:07]

LGTM w/nit

http://codereview.chromium.org/6812021/diff/1/cros_run_unit_tests
File cros_run_unit_tests (right):

http://codereview.chromium.org/6812021/diff/1/cros_run_unit_tests#newcode50
cros_run_unit_tests:50: # Run the unit test for the given package, via ebuild
src_test() function
Could you add a trailing period?

[Daniel Erat, 2011-04-07 17:08:03]

LGTM.  Thanks!

[sosa, 2011-04-07 17:08:42]

LGTM here a well.  Can you just re-verify that all packages run?  Run
without --package set

On Thu, Apr 7, 2011 at 10:08 AM,  <derat@chromium.org> wrote:
> LGTM.  Thanks!
>
> http://codereview.chromium.org/6812021/
>

[mtennant, 2011-04-07 18:46:20]

I just completed a full sync and run of all package unit tests, which passed.

On 2011/04/07 17:08:42, sosa wrote:
> LGTM here a well.  Can you just re-verify that all packages run?  Run
> without --package set
> 
> On Thu, Apr 7, 2011 at 10:08 AM,  <mailto:derat@chromium.org> wrote:
> > LGTM. &nbsp;Thanks!
> >
> > http://codereview.chromium.org/6812021/
> >

[sosa, 2011-04-07 19:48:53]

SGTM.  Thanks for the extra testing!

On Thu, Apr 7, 2011 at 11:46 AM,  <mtennant@chromium.org> wrote:
> I just completed a full sync and run of all package unit tests, which
> passed.
>
> On 2011/04/07 17:08:42, sosa wrote:
>>
>> LGTM here a well.  Can you just re-verify that all packages run?  Run
>> without --package set
>
>> On Thu, Apr 7, 2011 at 10:08 AM,  <mailto:derat@chromium.org> wrote:
>> > LGTM. &nbsp;Thanks!
>> >
>> > http://codereview.chromium.org/6812021/
>> >
>
>
>
> http://codereview.chromium.org/6812021/
>

[mtennant, 2011-04-07 20:01:43]

Feedback addressed.  Pushing shortly.

http://codereview.chromium.org/6812021/diff/1/cros_run_unit_tests
File cros_run_unit_tests (right):

http://codereview.chromium.org/6812021/diff/1/cros_run_unit_tests#newcode50
cros_run_unit_tests:50: # Run the unit test for the given package, via ebuild
src_test() function
On 2011/04/07 17:02:07, davidjames wrote:
> Could you add a trailing period?

Done.