[autotest] Make run_suite.py boolean arguments sane and functional.
When run_suite.py is invoked by the autotest_rpc code, arguments are
formed by doing '--long_name=value'. Therefore, some value will always
be passed, with the empty string representing disabling the switch, and
any non-empty string representing enabling the switch. The current code
does not handle this properly.
However, having to invoke run_suite.py with |-f ''| or |--file_bugs=|
looks silly, and is somewhat unintuitive, so instead just make each flag
take |True| or |False|. Therefore, the correct invocation is now
|-f True| or |--file_bugs=True|. Unspecified options default to |False|.
This can be safely updated with no dependant work as autotest_rpc is not
currently using either of these flags.
TEST=local run_suite.py
BUG=None
Change-Id: I3b6d0958757677d2ce17b00afcdf700f71d305ee
Reviewed-on: https://gerrit.chromium.org/gerrit/34553
Tested-by: Alex Miller <milleral@chromium.org>
Reviewed-by: Chris Masone <cmasone@chromium.org>
Reviewed-by: Scott Zawalski <scottz@chromium.org>
Commit-Ready: Alex Miller <milleral@chromium.org>
1 file changed