qakit

Merge lp://staging/~canonical-platform-qa/qakit/spread-xunit into lp://staging/qakit

  1. spread-xunit
  2. Merge into trunk
Proposed by Víctor R. Ruiz
Status: Needs review
Proposed branch: lp://staging/~canonical-platform-qa/qakit/spread-xunit
Merge into: lp://staging/qakit
Diff against target: 1142 lines (+1127/-0)
3 files modified
spread-xunit/spread.py (+122/-0)
spread-xunit/tests-results-raspi-2.txt (+627/-0)
spread-xunit/tests-results-raspi-3.txt (+378/-0)
To merge this branch: bzr merge lp://staging/~canonical-platform-qa/qakit/spread-xunit
Reviewer Review Type Date Requested Status
Canonical Platform QA Team Pending
Review via email: mp+314743@code.staging.launchpad.net

Description of the change

Support to convert Spread results to XUnit format.

To post a comment you must log in.
Revision history for this message
Max Brustkern (nuclearbob) wrote :

This patch to spread.py would accept filenames as arguments:

-#!/usr/bin/python
+#!/usr/bin/python3
 # -*- coding: utf8 -*-
 ## Parse the test results from spread tool
 import re
+import sys
 from dateutil.parser import parse

 re_executing = re.compile('(^.*) Executing external\:(.*)\.\.\.')
@@ -116,7 +117,6 @@
     # print(test_re_error())
     # print(test_re_error_output())
     # print(test_re_restoring())
- (tests, errors, length) = parse_execute('tests-results-raspi-2.txt')
- print(xunify(tests, errors, length))
- (tests, errors, length) = parse_execute('tests-results-raspi-3.txt')
- print(xunify(tests, errors, length))
+ for filename in sys.argv[1:]:
+ (tests, errors, length) = parse_execute(filename)
+ print(xunify(tests, errors, length))

Does qakit use pep8? If so, I can propose changes for that as well.

Reply
lp://staging/~canonical-platform-qa/qakit/spread-xunit updated
203. By Víctor R. Ruiz

Accept filenames as arguments (Max Brustkern)

Unmerged revisions

203. By Víctor R. Ruiz

Accept filenames as arguments (Max Brustkern)

202. By Víctor R. Ruiz

- Python script to convert spread results to xunit format.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: