( )⚙ D1636 tests: use Python to write binary data in lfs test instead of shell

This is an archive of the discontinued Mercurial Phabricator instance.

Differential D1636

tests: use Python to write binary data in lfs test instead of shell
ClosedPublic

Authored by durin42 on Dec 9 2017, 8:50 PM.

Details

Reviewers
yuja
Group Reviewers
hg-reviewers
Commits
rHGc7d49b87c1ee: tests: use Python to write binary data in lfs test instead of shell
Summary

The shell construct here appears to be unevenly supported: it works in /bin/sh
on FreeBSD, but it doesn't seem to work when /bin/sh is dash. Using a Python
inline directive works fine, so let's just do that instead.

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

durin42 created this revision.Dec 9 2017, 8:50 PM
yuja accepted this revision.Dec 10 2017, 5:56 AM
This revision is now accepted and ready to land.Dec 10 2017, 5:56 AM
Closed by commit rHGc7d49b87c1ee: tests: use Python to write binary data in lfs test instead of shell (authored by durin42). · Explain WhyDec 10 2017, 6:09 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mtests/test-lfs.t (7 lines)

Diff 4340

tests/test-lfs.t

$ echo SMALL > a2 $ echo SMALL > a2
$ hg commit -m a -A a1 a2 $ hg commit -m a -A a1 a2
$ hg status $ hg status
$ hg mv a1 b1 $ hg mv a1 b1
$ hg mv a2 a1 $ hg mv a2 a1
$ hg mv b1 a2 $ hg mv b1 a2
$ hg commit -m b $ hg commit -m b
$ hg status $ hg status
$ HEADER=$'\1\n' >>> with open('a2', 'wb') as f:
$ printf '%sSTART-WITH-HG-FILELOG-METADATA' "$HEADER" > a2 ... f.write(b'\1\nSTART-WITH-HG-FILELOG-METADATA')
$ printf '%sMETA\n' "$HEADER" > a1 >>> with open('a1', 'wb') as f:
... f.write(b'\1\nMETA\n')
$ hg commit -m meta $ hg commit -m meta
$ hg status $ hg status
$ hg log -T '{rev}: {file_copies} | {file_dels} | {file_adds}\n' $ hg log -T '{rev}: {file_copies} | {file_dels} | {file_adds}\n'
2: | | 2: | |
1: a1 (a2)a2 (a1) | | 1: a1 (a2)a2 (a1) | |
0: | | a1 a2 0: | | a1 a2
$ for n in a1 a2; do $ for n in a1 a2; do