)]}'
{
  "commit": "33613895571b0e64a458eea498c8cc3e7d05724b",
  "tree": "b81c2cfc77448dbd2389821297ff176e305c12a3",
  "parents": [
    "f58c8c77ef14d478d26df897fb77cc40e89bf8fc"
  ],
  "author": {
    "name": "Dave Borowitz",
    "email": "dborowitz@google.com",
    "time": "Thu Feb 28 10:50:26 2019 -0800"
  },
  "committer": {
    "name": "Dave Borowitz",
    "email": "dborowitz@google.com",
    "time": "Fri Mar 01 07:54:42 2019 -0800"
  },
  "message": "Expand acceptance tests for ChangeMessageModifiers\n\nEnforce the current behavior of short-circuiting and failing the entire\noperation when a single ChangeMessageModifier fails. This is not the\nconvention elsewhere in Gerrit; typically we log and continue if a\nsingle plugin fails. This also means we can\u0027t switch to using\nPluginSetContext for the implementation. However, we know that the\nChromium git-numberer plugin depends on this specific behavior (their CI\ngets very unhappy if their custom commit footers are missing), so we\ncan\u0027t change it at the moment. In the future, we could extend the\nChangeMessageModifier interface so some plugins are capable of bringing\ndown the whole operation, while the default is to log-and-ignore. That\nis beyond the scope of this change, which, again, is just enforcing the\ncurrent behavior.\n\nChange-Id: I5fd1a074868f9248ab018924fb732cae2c30b604\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "31813e336a730e04aceed8b132017620cba9283f",
      "old_mode": 33188,
      "old_path": "javatests/com/google/gerrit/acceptance/rest/change/AbstractSubmitByRebase.java",
      "new_id": "36595cec97167b11900ce4598ff9ef529044946e",
      "new_mode": 33188,
      "new_path": "javatests/com/google/gerrit/acceptance/rest/change/AbstractSubmitByRebase.java"
    },
    {
      "type": "modify",
      "old_id": "7eec8548d39e953b1abbec660be35a636f00d743",
      "old_mode": 33188,
      "old_path": "javatests/com/google/gerrit/acceptance/rest/change/SubmitByRebaseAlwaysIT.java",
      "new_id": "9ed8dbc91378893fe263017bfc8cbabe0931c960",
      "new_mode": 33188,
      "new_path": "javatests/com/google/gerrit/acceptance/rest/change/SubmitByRebaseAlwaysIT.java"
    }
  ]
}