This is an archive of the discontinued Mercurial Phabricator instance.

Differential D8171

tests: handle In-Reply-To headers for line wrapping
ClosedPublic

Authored by joerg.sonnenberger on Feb 26 2020, 4:26 PM.

Details

Reviewers
marmoute
Group Reviewers
hg-reviewers
Commits
rHGa1908951ca42: tests: handle In-Reply-To headers for line wrapping
Summary

Python 3 tends to insert a newline for both Message-ID and In-Reply-To
headers, so unwrap both. Just check the wrapped line format explicitly
without regular expression.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

marmoute accepted this revision.Feb 28 2020, 10:56 AM
marmoute added a subscriber: marmoute.

If the test are happy, I guess we are.

joerg.sonnenberger added a commit: rHGa1908951ca42: tests: handle In-Reply-To headers for line wrapping.Feb 28 2020, 2:14 PM
This revision was not accepted when it landed; it landed in state Needs Review.
Closed by commit rHGa1908951ca42: tests: handle In-Reply-To headers for line wrapping (authored by joerg.sonnenberger). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mtests/unwrap-message-id.py (6 lines)

Diff 20397

tests/unwrap-message-id.py

from __future__ import absolute_import, print_function from __future__ import absolute_import, print_function
import re
import sys import sys
print(re.sub(r"(?<=Message-Id:) \n ", " ", sys.stdin.read()), end="") for line in sys.stdin:
if line.lower() in ("message-id: \n", "in-reply-to: \n"):
line = line[:-2]
print(line, end="")