Merge lp:~jelmer/brz/fix-broken-pack-test into lp:brz

Proposed by Jelmer Vernooij on 2018-09-20
Status: Merged
Approved by: Jelmer Vernooij on 2018-09-21
Approved revision: 7123
Merge reported by: The Breezy Bot
Merged at revision: not available
Proposed branch: lp:~jelmer/brz/fix-broken-pack-test
Merge into: lp:brz
Diff against target: 12 lines (+1/-1)
1 file modified
breezy/tests/blackbox/test_branch.py (+1/-1)
To merge this branch: bzr merge lp:~jelmer/brz/fix-broken-pack-test
Reviewer Review Type Date Requested Status
Martin Packman 2018-09-20 Approve on 2018-09-21
Review via email: mp+355457@code.launchpad.net

Description of the change

Fix byteness when we happen to write a xFF in the wrong place..

To post a comment you must log in.
Martin Packman (gz) wrote :

Ha, that's what you mean by byteness.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'breezy/tests/blackbox/test_branch.py'
2--- breezy/tests/blackbox/test_branch.py 2018-09-14 12:10:17 +0000
3+++ breezy/tests/blackbox/test_branch.py 2018-09-20 22:41:48 +0000
4@@ -121,7 +121,7 @@
5 f.seek(-5, os.SEEK_END)
6 # Make sure we inject a value different than the one we just read
7 if c == b'\xFF':
8- corrupt = 'b\x00'
9+ corrupt = b'\x00'
10 else:
11 corrupt = b'\xFF'
12 f.write(corrupt) # make sure we corrupt something

Subscribers

People subscribed via source and target branches