)]}'
{
  "commit": "115ecd224b8a90d2c5ebe9e3f097f9a9a88b5e67",
  "tree": "b8798ee263619b9f704943b87ba767376fe20cdf",
  "parents": [
    "cacac00870e75b578253f11c10042e3af4c45a82"
  ],
  "author": {
    "name": "David Pursehouse",
    "email": "david.pursehouse@sonymobile.com",
    "time": "Fri Mar 11 13:16:42 2016 +0900"
  },
  "committer": {
    "name": "David Pursehouse",
    "email": "david.pursehouse@sonymobile.com",
    "time": "Fri Mar 11 11:06:13 2016 +0000"
  },
  "message": "RevisionIT: Extend \u0027submit\u0027 and \u0027submit on behalf of\u0027 tests\n\nThe existing test named \u0027submitOnBehalfOf\u0027 tests the case where the\nuser does not have permission.\n\nRename it to \u0027submitOnBehalfOfNotPermitted\u0027 to better reflect its\npurpose.\n\nUse ExpectedException rather than the \u0027expected\u0027 argument on the @Test\nannotation, and add an additional check for the expected message.\n\nAdd a new test \u0027submitOnBehalfOf\u0027 that tests the case where the user\ndoes have permission.\n\nAdd a new test that tests the case where the \u0027on behalf of\u0027 user is\nnot a valid account.\n\nAdd an assertion in the submit test that the change was merged.\n\nThere are still some cases that don\u0027t have coverage, but they can be\ndone in separate commits later:\n\n- Caller does not have submit permission\n- Destination branch does not exist\n- Destination branch is not visible to on-behalf-of user\n- Revision is not the current revision\n\nChange-Id: Iad0df1b9262b9031293c7f2b42eb5fd1a9c95ecb\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "7add9a2b8bc2817940858ecab74c7ed09667c143",
      "old_mode": 33188,
      "old_path": "gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/revision/RevisionIT.java",
      "new_id": "def831731c442149d14185ec1d54d842de45bf51",
      "new_mode": 33188,
      "new_path": "gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/revision/RevisionIT.java"
    }
  ]
}