----- "Lucas Meneghel Rodrigues" <[email protected]> wrote:
> Signed-off-by: Lucas Meneghel Rodrigues <[email protected]>
> ---
> client/tests/kvm/kvm_config.py | 14 +++++++-------
> client/tests/kvm/kvm_tests.py | 12 ++++++------
> 2 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/client/tests/kvm/kvm_config.py
> b/client/tests/kvm/kvm_config.py
> index 95eefcb..343c67d 100755
> --- a/client/tests/kvm/kvm_config.py
> +++ b/client/tests/kvm/kvm_config.py
> @@ -302,9 +302,9 @@ class config:
> # Found an operator?
> if op_found:
> if self.debug and not restricted:
> - self.__debug_print(indented_line,
> - "Parsing operator (%d dicts in current "
> - "context)" % len_list)
> + self.__debug_print(indented_line, ("Parsing operator "
^
I don't think we need the parentheses opened here |
because the parameters are already enclosed in parentheses.
> + "(%d dicts in current
> "
> + "context)" %
> len_list))
> (left, value) = self.split_and_strip(line, op_found)
> filters_and_key = self.split_and_strip(left, ":")
> filters = filters_and_key[:-1]
> @@ -349,10 +349,10 @@ class config:
> break
> list = filtered_list
> if self.debug and not restricted:
> - self.__debug_print(indented_line,
> - "Parsing no/only (%d dicts in current "
> - "context, %d remain)" %
> - (len_list, len(list)))
> + self.__debug_print(indented_line, ("Parsing no/only "
^
I don't think we need the parentheses opened here |
> + "(%d dicts in current
> "
> + "context, %d remain)"
> %
> + (len_list,
> len(list))))
>
> # Parse 'variants'
> elif line == "variants:":
> diff --git a/client/tests/kvm/kvm_tests.py
> b/client/tests/kvm/kvm_tests.py
> index 54d2a7a..4b87a76 100644
> --- a/client/tests/kvm/kvm_tests.py
> +++ b/client/tests/kvm/kvm_tests.py
> @@ -327,16 +327,16 @@ def run_autotest(test, params, env):
> logging.info(str(result))
> if result[1] == "FAIL":
> status_fail = True
> - message_fail = "Test '%s' ended with FAIL"
> - " (info: '%s')" % (result[0], result[3])
> + message_fail = ("Test '%s' ended with FAIL "
> + "(info: '%s')" % (result[0], result[3]))
> if result[1] == "ERROR":
> status_error = True
> - message_error = "Test '%s' ended with ERROR"
> - " (info: '%s')" % (result[0], result[3])
> + message_error = ("Test '%s' ended with ERROR "
> + "(info: '%s')" % (result[0], result[3]))
> if result[1] == "ABORT":
> status_error = True
> - message_error = "Test '%s' ended with ABORT"
> - " (info: '%s')" % (result[0], result[3])
> + message_error = ("Test '%s' ended with ABORT "
> + "(info: '%s')" % (result[0], result[3]))
>
> # Copy test results to the local bindir/guest_results
> logging.info("Copying results back from guest...")
> --
> 1.6.2.2
>
> _______________________________________________
> Autotest mailing list
> [email protected]
> http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
