4 Commits

Author	SHA1	Message	Date
Chris Sosa	f53a8c769b	Add logs for update_engine, devserver, and cros_run_vm_update to test artifacts. ... Change-Id: Ib5617612be38bb6d7dafcb2edab887c998a676ab BUG=chromium-os:12951 TEST=Ran it with Simple. Saw logs generated (1_update) and in root/dev_server.log. Ran unittests for cros_build_lib. Review URL: http://codereview.chromium.org/6672007	2011-03-11 15:42:31 -08:00
Don Garrett	e1b78a7efd	Reintroduce RunCommand cleanup and unit tests. Fixed previous issue in which exceptions were thrown if exit_code was set, and added a new unittest to prove that. ... This reintroduces the change after commit 3635aaa55e, which reverted 01c8423582. BUG=chromium-os:11717 TEST=Ran new unit tests, and did full build. Review URL: http://codereview.chromium.org/6579048 Change-Id: I598e459ddaa156af47d47d0482cac16ce44f07a4	2011-03-01 14:22:41 -08:00
Luigi Semenzato	3635aaa55e	Revert "RunCommand was catching and rethrowing an exception, which seemed to generate" ... This reverts commit 01c8423582. BUG=chromium-os:11717 TEST=none Review URL: http://codereview.chromium.org/6575022 Change-Id: I4e66e7b2fcf01a4cb35557bdf208f990e6d8eb2b	2011-02-23 19:14:53 -08:00
Don Garrett	01c8423582	RunCommand was catching and rethrowing an exception, which seemed to generate ... extra noise in the final exception output. This change stops catching and rethrowing the same exception, and generally makes the RunCommand slightly more readable (if longer). Some unit tests are added, but they only test RunCommand, and not all options to RunCommand. BUG=chromium-os:11717 TEST=Manual, and new lib/cros_build_lib_unittest Review URL: http://codereview.chromium.org/6576016 Change-Id: Ibe9b13ff302621e93196008f86914cb65a232d61	2011-02-23 17:36:52 -08:00