Merge lp:~kissiel/checkbox/fix-1420352 into lp:checkbox

Proposed by Maciej Kisielewski
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 3575
Merged at revision: 3575
Proposed branch: lp:~kissiel/checkbox/fix-1420352
Merge into: lp:checkbox
Diff against target: 11 lines (+1/-1)
1 file modified
plainbox/plainbox/impl/secure/providers/v1.py (+1/-1)
To merge this branch: bzr merge lp:~kissiel/checkbox/fix-1420352
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+249271@code.launchpad.net

Description of the change

This MR makes WhiteListPlugIn delazify text content of file when creating
TestPlanUnit. Not doing so kept text as LazyFileContent which made plainbox not
guess the role of the file, breaking serialization of that unit which is
needed when comparing units.

540d8cc plainbox:secure:providers: delazify content when creating testplan unit

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

Hey, thanks for fixing this, +1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plainbox/plainbox/impl/secure/providers/v1.py'
--- plainbox/plainbox/impl/secure/providers/v1.py 2015-02-06 15:51:41 +0000
+++ plainbox/plainbox/impl/secure/providers/v1.py 2015-02-11 00:07:23 +0000
@@ -199,7 +199,7 @@
199 'unit': TestPlanUnit.Meta.name,199 'unit': TestPlanUnit.Meta.name,
200 'id': name,200 'id': name,
201 'name': name,201 'name': name,
202 'include': text,202 'include': str(text), # delazify content
203 }, origin=origin, provider=provider, field_offset_map=field_offset_map,203 }, origin=origin, provider=provider, field_offset_map=field_offset_map,
204 virtual=True)204 virtual=True)
205205

Subscribers

People subscribed via source and target branches