mirror of
https://github.com/facebook/sapling.git
synced 2024-10-11 09:17:30 +03:00
1059b7253f
Summary: I pretty much stole this from our libfb.py.conduit client, but removed the python 2.7 and 3 specific aspects of it. This is an HTTP client for conduit, rather than shelling out to arcanist. I've added a very simple mechanism for replaying conduit results in the test harness and used this to build out some tests for the `arcdiff.py` and `phabstatus.py` extensions. Test Plan: ``` $ ../../hg-crew/tests/run-tests.py -j8 ``` In addition to the new tests, manually tested the actual HTTP functionality: ``` $ /data/users/wez/facebook-hg-rpms/hg-crew/hg --config extensions.phabstatus=/data/users/wez/facebook-hg-rpms/fb-hgext/phabstatus.py --config extensions.errorredirect=! ssl ``` Does not error out and shows the diff status. Reviewers: #sourcecontrol, ttung Subscribers: mjpieters Differential Revision: https://phabricator.fb.com/D3200713
60 lines
1.6 KiB
Perl
60 lines
1.6 KiB
Perl
Setup
|
|
|
|
$ PYTHONPATH=$TESTDIR/..:$PYTHONPATH
|
|
$ export PYTHONPATH
|
|
$ cat >> $HGRCPATH << EOF
|
|
> [extensions]
|
|
> arcconfig=$TESTDIR/../phabricator/arcconfig.py
|
|
> phabstatus=$TESTDIR/../phabstatus.py
|
|
> smartlog=$TESTDIR/../smartlog.py
|
|
> EOF
|
|
$ hg init repo
|
|
$ cd repo
|
|
$ touch foo
|
|
$ hg ci -qAm 'Differential Revision: https://phabricator.fb.com/D1'
|
|
|
|
With an invalid arc configuration
|
|
|
|
$ hg log -T '{phabstatus}\n' -r .
|
|
arcconfig configuration problem. No diff information can be provided.
|
|
Error info: no .arcconfig foundError
|
|
|
|
Configure arc...
|
|
|
|
$ echo '{}' > .arcconfig
|
|
$ echo '{}' > .arcrc
|
|
|
|
And now with bad responses:
|
|
|
|
$ cat > $TESTTMP/mockduit << EOF
|
|
> [{"cmd": ["differential.query", {"ids": ["1"]}], "result": {}}]
|
|
> EOF
|
|
$ HG_ARC_CONDUIT_MOCK=$TESTTMP/mockduit hg log -T '{phabstatus}\n' -r .
|
|
Error
|
|
|
|
$ cat > $TESTTMP/mockduit << EOF
|
|
> [{"cmd": ["differential.query", {"ids": ["1"]}], "error_info": "failed, yo"}]
|
|
> EOF
|
|
$ HG_ARC_CONDUIT_MOCK=$TESTTMP/mockduit hg log -T '{phabstatus}\n' -r .
|
|
Error talking to phabricator. No diff information can be provided.
|
|
Error info: failed, yoError
|
|
|
|
Missing id field is treated as an error
|
|
|
|
$ cat > $TESTTMP/mockduit << EOF
|
|
> [{"cmd": ["differential.query", {"ids": ["1"]}],
|
|
> "result": [{"statusName": "Needs Review"}]}]
|
|
> EOF
|
|
$ HG_ARC_CONDUIT_MOCK=$TESTTMP/mockduit hg log -T '{phabstatus}\n' -r .
|
|
Error
|
|
|
|
And finally, the success case
|
|
|
|
$ cat > $TESTTMP/mockduit << EOF
|
|
> [{"cmd": ["differential.query", {"ids": ["1"]}],
|
|
> "result": [{"id": 1, "statusName": "Needs Review"}]}]
|
|
> EOF
|
|
$ HG_ARC_CONDUIT_MOCK=$TESTTMP/mockduit hg log -T '{phabstatus}\n' -r .
|
|
Needs Review
|
|
|