This is an archive of the discontinued Mercurial Phabricator instance.

Differential D8390

tests: move the phabricator auth token to the global config file
ClosedPublic

Authored by mharbison72 on Apr 8 2020, 5:52 PM.

Details

Reviewers
Alphare
Group Reviewers
hg-reviewers
Commits
rHGed81fa859426: tests: move the phabricator auth token to the global config file
Summary

The next commit introduces a new repo to simplify its development. This value
needs to be modified to record tests, so it doesn't make sense to have to do
that twice. The callsign and URL are *not* moved because there are tests that
fallback to the .arcconfig file when those aren't present.

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

mharbison72 created this revision.Apr 8 2020, 5:52 PM
Alphare accepted this revision.Apr 16 2020, 9:43 AM
mharbison72 added a commit: rHGed81fa859426: tests: move the phabricator auth token to the global config file.Apr 16 2020, 12:42 PM
This revision was not accepted when it landed; it landed in state Needs Review.
Closed by commit rHGed81fa859426: tests: move the phabricator auth token to the global config file (authored by mharbison72). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mtests/test-phabricator.t (18 lines)

Diff 21133

tests/test-phabricator.t

#require vcr #require vcr
$ cat >> $HGRCPATH <<EOF $ cat >> $HGRCPATH <<EOF
> [extensions] > [extensions]
> phabricator = > phabricator =
> >
> [auth]
> hgphab.schemes = https
> hgphab.prefix = phab.mercurial-scm.org
> # When working on the extension and making phabricator interaction
> # changes, edit this to be a real phabricator token. When done, edit
> # it back. The VCR transcripts will be auto-sanitised to replace your real
> # token with this value.
> hgphab.phabtoken = cli-hahayouwish
>
> [phabricator] > [phabricator]
> debug = True > debug = True
> EOF > EOF
$ hg init repo $ hg init repo
$ cd repo $ cd repo
$ cat >> .hg/hgrc <<EOF $ cat >> .hg/hgrc <<EOF
> [phabricator] > [phabricator]
> url = https://phab.mercurial-scm.org/ > url = https://phab.mercurial-scm.org/
> callsign = HG > callsign = HG
>
> [auth]
> hgphab.schemes = https
> hgphab.prefix = phab.mercurial-scm.org
> # When working on the extension and making phabricator interaction
> # changes, edit this to be a real phabricator token. When done, edit
> # it back. The VCR transcripts will be auto-sanitised to replace your real
> # token with this value.
> hgphab.phabtoken = cli-hahayouwish
> EOF > EOF
$ VCR="$TESTDIR/phabricator" $ VCR="$TESTDIR/phabricator"
Error is handled reasonably. We override the phabtoken here so that Error is handled reasonably. We override the phabtoken here so that
when you're developing changes to phabricator.py you can edit the when you're developing changes to phabricator.py you can edit the
above config and have a real token in the test but not have to edit above config and have a real token in the test but not have to edit
this test. this test.
$ hg phabread --config auth.hgphab.phabtoken=cli-notavalidtoken \ $ hg phabread --config auth.hgphab.phabtoken=cli-notavalidtoken \