Merge lp:~stylesen/lava-dispatcher/fix-empty-parse-pattern into lp:lava-dispatcher

[Senthil Kumaran S]

[Bug #1079642 ] Fix for test definitions which does not have a parse pattern.

[Andy Doan (doanac)]

Two comments:

1) you could also do this with only 2 lines of change with:
 if 'parse' not in testdef:
     return []

2) I don't completely agree with this bug. Its going to be odd to have a test that has no results. And this could confuse someone. So, at a minimum I think we should log a warning like:

 if 'parse' not in testdef:
     logging.warning('The test definition contains no result parsing rule')
     return []

[lp:~stylesen/lava-dispatcher/fix-empty-parse-pattern]

453. By Senthil Kumaran S on 2012-11-19

[Bug #1079642] Give a default pattern to parse, if one is not specified in
the test definition.

[Andy Doan (doanac)]

small typo "fal":

 defpat = "(?P<test_case_id>.*-*)\\s+:\\s+(?P<result>(PASS|FAIL|pass|fal))"

You also lefted out "skip|SKIP"

[Andy Doan (doanac)]

review: Approve

[lp:~stylesen/lava-dispatcher/fix-empty-parse-pattern]

454. By Senthil Kumaran S on 2012-11-19

Fix a small typo in the parse pattern and add more default result values.