This is an archive of the discontinued Mercurial Phabricator instance.

Differential D5973

tests: use raw string in test-impexp-branch.t
ClosedPublic

Authored by indygreg on Feb 15 2019, 4:09 PM.

Details

Reviewers
None
Group Reviewers
hg-reviewers
Commits
rHG7396508ad92b: tests: use raw string in test-impexp-branch.t
Summary

On first glance, the escaping of \s seems correct. However,
the shell eats the escape and we're left with '\s` in the
written file. Let's use a raw string so we don't have to
double escape.

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

indygreg created this revision.Feb 15 2019, 4:09 PM
Closed by commit rHG7396508ad92b: tests: use raw string in test-impexp-branch.t (authored by indygreg). · Explain WhyFeb 15 2019, 10:34 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mtests/test-impexp-branch.t (2 lines)

Diff 14116

tests/test-impexp-branch.t

$ echo '[extensions]' >> $HGRCPATH $ echo '[extensions]' >> $HGRCPATH
$ echo 'strip =' >> $HGRCPATH $ echo 'strip =' >> $HGRCPATH
$ cat >findbranch.py <<EOF $ cat >findbranch.py <<EOF
> from __future__ import absolute_import > from __future__ import absolute_import
> import re > import re
> import sys > import sys
> >
> head_re = re.compile('^#(?:(?:\\s+([A-Za-z][A-Za-z0-9_]*)(?:\\s.*)?)|(?:\\s*))$') > head_re = re.compile(r'^#(?:(?:\\s+([A-Za-z][A-Za-z0-9_]*)(?:\\s.*)?)|(?:\\s*))$')
> >
> for line in sys.stdin: > for line in sys.stdin:
> hmatch = head_re.match(line) > hmatch = head_re.match(line)
> if not hmatch: > if not hmatch:
> sys.exit(1) > sys.exit(1)
> if hmatch.group(1) == 'Branch': > if hmatch.group(1) == 'Branch':
> sys.exit(0) > sys.exit(0)
> sys.exit(1) > sys.exit(1)