Closed Bug 1167283 Opened 10 years ago Closed 7 years ago

Add mach command to create local testing environment

Categories

(MozReview Graveyard :: General, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: dminor, Unassigned)

Details

Attachments

(5 files)

It looks like all the mozreview developers have different scripts to create an environment for local testing outside of running the unit tests. It would be beneficial to take the good parts of these and create a mach command for local testing.
testing: add Bugzilla group support to `mozreview create-user` By adding this, `mozreview create-user` should now be able to create both a Bugzilla and LDAP/system user in one command. Previously, we needed multiple commands to accomplish this.
Attachment #8623901 - Flags: review?(smacleod)
testing: use `mozreview create-user` to create users Before, we had multiple commands for creating a Bugzilla user and an LDAP/system user. Now, we use a single command. This will be the preferred user creation mechanism going forward, at least for users that require accounts in both systems.
Attachment #8623902 - Flags: review?(smacleod)
https://reviewboard.mozilla.org/r/11605/#review10025 This series is incomplete. But since it conflicts with stuff mcote is working on, I figure we should get it landed. Docs updates and other improvements will be added later.
testing: rewrite mozreview-ssh as a Python script We want to add functionality to this. That will be much easier once it is written in Python instead of shell. Support for SSH_USERNAME has been removed since it isn't used anywhere.
Attachment #8623953 - Flags: review?(smacleod)
testing: store credentials for created users We want the ability to easily look up credentials for created users. Store the credentials in plain text.
Attachment #8623954 - Flags: review?(smacleod)
testing: easily create configured clones of review repositories
Attachment #8623955 - Flags: review?(smacleod)
Comment on attachment 8623901 [details] MozReview Request: testing: add Bugzilla group support to `mozreview create-user` https://reviewboard.mozilla.org/r/11607/#review10109 Ship It!
Attachment #8623901 - Flags: review?(smacleod) → review+
Attachment #8623902 - Flags: review?(smacleod) → review+
Comment on attachment 8623902 [details] MozReview Request: testing: use `mozreview create-user` to create users https://reviewboard.mozilla.org/r/11609/#review10111 Ship It!
Comment on attachment 8623953 [details] MozReview Request: testing: rewrite mozreview-ssh as a Python script https://reviewboard.mozilla.org/r/11617/#review10113 Ship It!
Attachment #8623953 - Flags: review?(smacleod) → review+
Comment on attachment 8623954 [details] MozReview Request: testing: store credentials for created users https://reviewboard.mozilla.org/r/11619/#review10115 Ship It!
Attachment #8623954 - Flags: review?(smacleod) → review+
Comment on attachment 8623955 [details] MozReview Request: testing: easily create configured clones of review repositories https://reviewboard.mozilla.org/r/11621/#review10117 This stuff is __great__ - thanks for taking care of it. ::: docs/hacking-mozreview.rst:181 (Diff revision 1) > +Creating Configured Repository Clones Super bonus points request - allow me to point this at an existing clone of *some* hg repository, and have the command (possibly prompting before doing it) overwrite the hgrc. This makes it easy for me to reuse a testing repository with certain commits set up, rather than having to manually pull the changes over to the repo etc. ::: testing/vcttesting/mozreview.py:362 (Diff revision 1) > - def create_repository(self, name): > + def repo_urls(self, name): I'd love it if we could expose this as a command as well (or a command which just outputs *every* url - rbweb, bmo, hg [including all repos], etc.)
Attachment #8623955 - Flags: review?(smacleod) → review+
Landed the first series. There's still some more work to make this turnkey.
Product: Developer Services → MozReview
MozReview is now obsolete. Please use Phabricator instead. Closing this bug.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: