Merge ~sylvain-pineau/checkbox-ng:fix-1811221 into checkbox-ng:master

Proposed by Sylvain Pineau
Status: Merged
Approved by: Sylvain Pineau
Approved revision: f20658fa72a2d28a0096e18c7c3ca5f1f7f99817
Merged at revision: 4b3771a5a067667334f2e2051a2dd0c6c289a37a
Proposed branch: ~sylvain-pineau/checkbox-ng:fix-1811221
Merge into: checkbox-ng:master
Diff against target: 13 lines (+1/-1)
1 file modified
plainbox/impl/exporter/tar.py (+1/-1)
Reviewer Review Type Date Requested Status
Jonathan Cave (community) Approve
Review via email: mp+373721@code.launchpad.net

Description of the change

Fixes linked bug by adding the junit report in the submission.xz report

To post a comment you must log in.
Revision history for this message
Jonathan Cave (jocave) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/plainbox/impl/exporter/tar.py b/plainbox/impl/exporter/tar.py
2index 72e1ff0..f9ba826 100644
3--- a/plainbox/impl/exporter/tar.py
4+++ b/plainbox/impl/exporter/tar.py
5@@ -66,7 +66,7 @@ class TARSessionStateExporter(SessionStateExporterBase):
6
7 job_state_map = manager.default_device_context.state.job_state_map
8 with tarfile.TarFile.open(None, 'w:xz', stream, preset=preset) as tar:
9- for fmt in ('html', 'json', 'xlsx'):
10+ for fmt in ('html', 'json', 'junit', 'xlsx'):
11 if fmt == 'xlsx':
12 options_list = [
13 XLSXSessionStateExporter.OPTION_WITH_SYSTEM_INFO,

Subscribers

People subscribed via source and target branches