6effcc0047e7b7a9e1df61210db4b3f4d16cd315: hgmaster: remove legacy path normalization
Gregory Szorc <gps@mozilla.com> - Thu, 05 Mar 2015 23:41:37 -0800 - rev 360499
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
hgmaster: remove legacy path normalization The line being modified was removed a while ago.
30fe2d69feab359ba1e2585477d3a19cf688f359: testing: establish vcttesting.util module
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 13:12:55 -0800 - rev 360498
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
testing: establish vcttesting.util module And seed it with a bunch of utility functions.
9f041e2cf97da5bb8fb8241ba998c687bf52b452: testing: make the admin user an admin in Review Board; r=smacleod
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 12:52:40 -0800 - rev 360497
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
testing: make the admin user an admin in Review Board; r=smacleod Up to this point, the admin user was only an admin user in Bugzilla. We had to explicitly make the user an admin in Review Board. This was for historical reasons, as tests were performing actions as the admin user. Now that all tests are using a regular user account, it is safe to make the admin user an admin in Review Board. So do it and make hacking on MozReview easier in the process.
8e3ce6b000aeb70a865a9085ec9f5ab12681ea22: bzpost: don't use admin user in tests; r=smacleod
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 12:52:28 -0800 - rev 360496
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
bzpost: don't use admin user in tests; r=smacleod With this patch, "admin@example.com" no longer occurs in any test output.
0a76fa261ef36cc98d224208a5e2d10280101d8d: bzexport: don't use admin user in tests; r=smacleod
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 12:52:14 -0800 - rev 360495
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
bzexport: don't use admin user in tests; r=smacleod bzexport tests, like mozreview's, were using the admin user as part of testing. That's a bad practice. This patch puts an end to the practice. The test output did change in one unexpected way. The bug status is now marked as ASSIGNED instead of NEW in a few locations. This appears to be purely a result of the differences between admin and non-admin users. But the change does warrant additional scrutiny in case it isn't.
720e1c3779b3bcc3191e70130cd30b7e1a4418f9: mozreview: create a default user as part of commonenv; r=smacleod
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 12:51:29 -0800 - rev 360494
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: create a default user as part of commonenv; r=smacleod Before we can make the admin user an actual admin user, we need to have tests stop using the admin user as part of regular Review Board interaction. This patch moves the creation of the default user into commonenv and updates all remaining tests to reflect the utilization of this new account.
3e8890f6b3fde6aa5fd0ead7ebaada5c1d4a2c03: mozreview: convert test-auth.t to have default user; r=smacleod
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 12:51:12 -0800 - rev 360493
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: convert test-auth.t to have default user; r=smacleod We have a goal to introduce a default, regular user to the common environment. test-auth.t is one of the more complicated tests involving users. Convert it to the new format first. As part of this, we also make the admin user an actual admin sooner, as this will also soon be done in commonenv.
ac7786f8952d54242cacfdc5c6f2941014bca7ba: testing: remove "reviewboard create" command; r=smacleod
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 12:50:57 -0800 - rev 360492
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
testing: remove "reviewboard create" command; r=smacleod It isn't possible to create Review Board instances outside the context of MozReview. This functionality has been replaced by functionality inside the MozReview APIs.
ef1074da77259bf5809f5bcbd119a99910c70492: mozreview: create and use adminbugzilla alias; r=smacleod
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 12:50:42 -0800 - rev 360491
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: create and use adminbugzilla alias; r=smacleod We have a goal to make the default admin user an actual admin in Review Board. Before this can be done, we need to change tests to stop using this user. To make the transition easier, we introduce an "adminbugzilla" alias that runs the Bugzilla helper command with the admin credentials. We switch invocations of "bugzilla" requiring admin access to this new alias.
284cc44767f7f9a802178f57a530dbe87a7385e3: mozreview: eliminate NO_BUGZILLA_AUTH variable; r=smacleod
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 12:50:13 -0800 - rev 360490
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: eliminate NO_BUGZILLA_AUTH variable; r=smacleod It isn't used any more.
59aadbfdf529b9b528f44d6476900b09a2fb1751: mozreview: print submitter username in dumpreview output; r=smacleod
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 12:49:54 -0800 - rev 360489
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: print submitter username in dumpreview output; r=smacleod Managing the active user in MozReview tests is a bit overwhelming. It can result in unexpected users being used. To help avoid confusion, this patch adds the submitter the the "dumpreview" command output. The new test output clearly demonstrates that we have the admin user performing a lot of actions. Fortunately, the admin user isn't actually a Review Board admin currently. But we want to change that. Upcoming patches will change these tests to use a different user and the new output will prove the admin user is no longer being used by tests.
24f8668e52f2246378af2b0f3a7d8eaf077803e6: testing: move make_admin into MozReviewBoard class; r=smacleod
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 12:49:34 -0800 - rev 360488
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
testing: move make_admin into MozReviewBoard class; r=smacleod We want to call this API from elsewhere. Move it out of an inline mach command and into a module. As part of this, we changed the _get_rb() function to take an optional path argument. It is a holdover from the days of old to pass a path into the reviewboard mach commands. This paves the road to getting rid of this pattern.
eb65e00a415aca2d28ee3c0f57df0e527a63e3dc: mozreview: add target_people to dumpreview output; r=smacleod
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 12:47:51 -0800 - rev 360487
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: add target_people to dumpreview output; r=smacleod This should have always been present. Not sure why it wasn't. This should help make reviewer modification testing easier to verify.
634af10201748bba5c4d9206d1d4a59e3cf20719: mozreview: don't create excessive bugs during tests; r=smacleod
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 12:47:09 -0800 - rev 360486
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: don't create excessive bugs during tests; r=smacleod Some tests were creating several bugs even though they didn't need them. This was causing tests to run slower than necessary. Stop the practice. On my machine, this was adding ~9s overhead to each test performing the practice.
8758b8bce90a853b97a23a026a42032fe4c0a886: docs: use internal link instead of absolute URL
Gregory Szorc <gps@mozilla.com> - Fri, 06 Mar 2015 12:44:45 -0800 - rev 360485
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
docs: use internal link instead of absolute URL
3b8a10644a71ffe21fc02b21c1baaab7c1bd2b56: autoland: create .t style tests for autoland (bug 1140155); r=gps
Dan Minor <dminor@mozilla.com> - Fri, 06 Mar 2015 06:47:17 -0500 - rev 360484
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
autoland: create .t style tests for autoland (bug 1140155); r=gps This adds mach commands for interacting with autoland and replaces the current unittest based integration test with a .t style test which uses these commands. This also fixes bad paths in the autoland.conf and start-autoland.sh commands which were introduced when the volume mapping was being used for debugging.
333d86874a6e810891106e1ce51fdb803ad4fec0: docs: add information on submitting changesets
Michael Comella <michael.l.comella@gmail.com> - Fri, 06 Mar 2015 11:47:33 -0800 - rev 360483
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
docs: add information on submitting changesets
aa4f74279f072665a0fee334c21474831e46334b: testing: prune old autoland images
Gregory Szorc <gps@mozilla.com> - Thu, 05 Mar 2015 17:08:49 -0800 - rev 360482
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
testing: prune old autoland images This change should have been made when support for autoland Docker images landed. I missed it as part of the review. Adding it now.
d6bc1f4e63372aeb81b8075543c12d29e6272c7e: testing: upgrade ecdsa and paramiko packages
Gregory Szorc <gps@mozilla.com> - Thu, 05 Mar 2015 16:12:05 -0800 - rev 360481
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
testing: upgrade ecdsa and paramiko packages
62e751dcaf84fe732f7700bf5ac52437513ef890: testing: use autoland images from unittest fixture
Gregory Szorc <gps@mozilla.com> - Thu, 05 Mar 2015 15:34:56 -0800 - rev 360480
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
testing: use autoland images from unittest fixture We need to pass the autoland Docker images into mr.start() or bootstrap will be performed, slowing down test execution. This is a very minor regression from 13287eb8684d.
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip