[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [test-API PATCH] Fix a bug in the case that clean flag is used

On 04/23/2012 05:23 PM, Osier Yang wrote:
On 2012年04月23日 16:22, Guannan Ren wrote:
     When clean flag is used in testcase, the framework


     reports 'KeyError'.
     The bug is caused is by getting mod_case earlier


     than checking whether the 'clean' flag is present or not.

I'd think it doesn't explain the bug well, though I'm
not sure how to give a good explaination too. Perhaps
an example will be good. (It looks to me there is wrong
order of the string manipulation, and thus got a unknown

        Yes, you are right.

     The testcases without flag set doesn't be affected.

without 'clean' flag set are not affected.

BTW, could you destroy the "4 leading spaces" in commit
message, though it's fine, but nobody uses it. :-)

        ok, thanks

  src/generator.py |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/generator.py b/src/generator.py
index 6a46889..2e4ae07 100644
--- a/src/generator.py
+++ b/src/generator.py
@@ -115,11 +115,11 @@ class FuncGen(object):
              clean_flag = False

              mod_case_func = self.case_name_list[i]
-            mod_case = mod_case_func.rsplit(":", 1)[0]
              if mod_case_func.endswith(':clean'):
                  mod_case_func = mod_case_func[:-6]
                  clean_flag = True

+            mod_case = mod_case_func.rsplit(":", 1)[0]
              self.fmt.print_start(mod_case, env_logger)

              case_params = self.case_params_list[i]

Anyway, it looks right fix, ACK with the nits fixed.


     Thanks and pushed with nits fixed.

     Guannan Ren

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]