tree 0347ab1dcfd9f88f755939e504aaf7c01b2bbd11
parent 69b455425a52c2cc4b3679ab4c43282f445292d5
author Dana Goyette <dgoyette@chromium.org> 1571951505 -0700
committer Commit Bot <commit-bot@chromium.org> 1578513509 +0000

faft: Separate the run_shell_command method that returns a result.

In CL:1863838, the behavior for block=True changed, breaking tests.
Before, it waited for process to exit; after, it waited for stdout
to be read as well.

This commit changes the method back to its old behavior, and adds a
separate method to give a result with stdout, stderr, and returncode.

BUG=b:147227865
TEST=Run firmware_FAFTRPC.system

Change-Id: Ie516a8baed2b5d1a19ca764e62f037e224da16e3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/1988732
Reviewed-by: Mary Ruthven <mruthven@chromium.org>
Tested-by: Dana Goyette <dgoyette@chromium.org>
Commit-Queue: Dana Goyette <dgoyette@chromium.org>