)]}' { "commit": "8b8b23c16f5dadb4b2ddb977cad615d8d38f9211", "tree": "aff5039c6bffc51dd6a098609312c1122ee080cb", "parents": [ "5cf981df330f1bb6ea4828e5945d123c2d9a670c" ], "author": { "name": "Hector Oswaldo Caballero", "email": "hector.caballero@ericsson.com", "time": "Wed Jan 31 17:21:41 2018 -0500" }, "committer": { "name": "Hector Oswaldo Caballero", "email": "hector.caballero@ericsson.com", "time": "Wed Jan 31 17:43:25 2018 -0500" }, "message": "Add tests to JiraItsFacade\n\nPassing the JiraClient dependency as parameter to the constructor\n(instead of creating it inside the class) helps making this class\ntestable. In the same sense, the Jira configuration can be injected now\ninto the JiraClient; the previous constructor is kept, though, to allow\nthe init step to test connectivity. This will be modified in a follow up\nchange to avoid passing parameters along the whole chain of calls.\n\nChange-Id: Iee6803aded52fff70e079630bed421748ee27d68\n", "tree_diff": [ { "type": "modify", "old_id": "72c5acc4c7f337512ffb95ae81d6d0f18f990e68", "old_mode": 33188, "old_path": "src/main/java/com/googlesource/gerrit/plugins/its/jira/JiraClient.java", "new_id": "8fcdc60214c041146275b077dedc2a25bb68c4b1", "new_mode": 33188, "new_path": "src/main/java/com/googlesource/gerrit/plugins/its/jira/JiraClient.java" }, { "type": "modify", "old_id": "790cefd322f370c152e69375b105935b0c448076", "old_mode": 33188, "old_path": "src/main/java/com/googlesource/gerrit/plugins/its/jira/JiraItsFacade.java", "new_id": "c14f2e62d60d17df579bb0230e88862b4f464fc0", "new_mode": 33188, "new_path": "src/main/java/com/googlesource/gerrit/plugins/its/jira/JiraItsFacade.java" }, { "type": "add", "old_id": "0000000000000000000000000000000000000000", "old_mode": 0, "old_path": "/dev/null", "new_id": "6d645c806d081194652e645213c9779585135425", "new_mode": 33188, "new_path": "src/test/java/com/googlesource/gerrit/plugins/its/jira/JiraItsFacadeTest.java" } ] }