)]}'
{
  "commit": "681f711911eaf22c96f45df82a7ffde104ff2b72",
  "tree": "01e396305479ca05f31175376eb591f49f703d4c",
  "parents": [
    "5b8bfcbc4ff4fc669671ba640fc0387cb84170a5"
  ],
  "author": {
    "name": "Antonio Barone",
    "email": "syntonyze@gmail.com",
    "time": "Tue Sep 16 21:05:22 2025 +0200"
  },
  "committer": {
    "name": "Antonio Barone",
    "email": "syntonyze@gmail.com",
    "time": "Tue Sep 16 21:16:52 2025 +0200"
  },
  "message": "Drop tests asserting defaults on invalid time units\n\nParsing invalid time units now throws IllegalArgumentException (since\nchange I23f5586e1f6) instead of silently falling back to defaults. The\nhigh-availability configuration tests were still asserting that an\ninvalid value should yield the default, which no longer reflects the\ncurrent contract and caused brittle failures.\n\nThis change removes those assertions from ConfigurationTest for:\n- http.connectionTimeout\n- http.socketTimeout\n- http.retryInterval\n\nRationale:\n- These checks verify ConfigUtil’s parsing behavior, which is already\n  covered by its own test suite.\n- The plugin does not promise a default-on-error fallback; encoding that\n  expectation in its tests is misleading.\n- Dropping the assertions decouples the plugin tests from lower-level\n  parsing details and prevents false failures after the API change.\n\nBug: Issue 445404588\nChange-Id: Ie112b7dce05d35cdd4e4e3028c38b0d7d0d660b4\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "b51e1a022c4443388ddff5f3ed06ac893eeaea34",
      "old_mode": 33188,
      "old_path": "src/test/java/com/ericsson/gerrit/plugins/highavailability/ConfigurationTest.java",
      "new_id": "0fb7dadf6ab4a9504a50be2b9dd13da64a1ff4ef",
      "new_mode": 33188,
      "new_path": "src/test/java/com/ericsson/gerrit/plugins/highavailability/ConfigurationTest.java"
    }
  ]
}