phldef_conduit: add phid to canned user data

Make it possible to test the handling of phids by copying the
phids from the pre-recorded accounts into the canned data.

Test Plan: $ ./precommit.sh
phlcon_user: fix query limit

Fix an apparent copy and paste mistake, limit of single user query is 1,
not len(email_address)

Test Plan:
$ ./precommit.sh
$ ./testbed/arcyd/acceptance_tests.sh
arcyd: fix abort when author is also reviewer

Phabricator does not normally permit the author of a review to be a
reviewer, enforce this rule in Arcyd by removing the author from the
list of reviewers before sending the list of reviewers to Phabricator.

Notify the user that this has happened so that they may check for other
errors in the list of reviewers.

Test Plan:

added coverage to
$ ./testbed/arcyd/acceptance_tests.sh
reviewed error message to make sure it's suitable.

$ ./precommit.sh
arcyd: test for no exceptions in system tests

Make the system tests more effective by exiting with code 1 from
Arcyd if any operations failed during processing.

Test Plan:
Removed recent handler for 'author cant be reviewer' and ensure that
acceptance tests fail.
$ ./precommit.sh
$ ./testbed/arcyd/acceptance_tests.sh