This is an archive of the discontinued Mercurial Phabricator instance.

Differential D619

tests: use python instead of echo -e
ClosedPublic

Authored by simonfar on Sep 4 2017, 8:22 AM.
Tags
None
Subscribers
None

Details

Reviewers
ryanmce
Group Reviewers
Restricted Project
Commits
rFBHGX3216f5c73f0c: tests: use python instead of echo -e
Summary

BSD echo doesn't support -e. The lint rule says to use Python instead,
so follow its advice

Test Plan

Ran tests locally on Mac and Linux.

Diff Detail

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

Event Timeline

simonfar created this revision.Sep 4 2017, 8:22 AM
ryanmce accepted this revision.Sep 4 2017, 8:25 AM
This revision is now accepted and ready to land.Sep 4 2017, 8:25 AM
Closed by commit rFBHGX3216f5c73f0c: tests: use python instead of echo -e (authored by simonfar). · Explain WhySep 6 2017, 9:54 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mtests/test-scm-prompt-hg.t (2 lines)

Diff 1647

tests/test-scm-prompt-hg.t

$ cmd $ cmd
(97af35b|remote/remote9...) (97af35b|remote/remote9...)
$ echo 97af35b3648c0098cbd8114ae1b1bafab997ac20 bookmarks remote/@ >> .hg/remotenames $ echo 97af35b3648c0098cbd8114ae1b1bafab997ac20 bookmarks remote/@ >> .hg/remotenames
$ cmd $ cmd
(97af35b|remote/@...) (97af35b|remote/@...)
Test eden snapshots Test eden snapshots
$ mkdir -p .eden/client/ $ mkdir -p .eden/client/
$ echo -e "eden\x00\x00\x00\x01\xde\xad\xbe\xef\xca\xfe\xfa\xce\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00" > .eden/client/SNAPSHOT $ python -c 'print("eden\x00\x00\x00\x01\xde\xad\xbe\xef\xca\xfe\xfa\xce\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00")' > .eden/client/SNAPSHOT
$ rm .hg/dirstate $ rm .hg/dirstate
$ cmd $ cmd
(deadbee) (deadbee)