{"action":"created","comment":{"url":"https://api.github.com/repos/HubTurbo/addressbook/pulls/comments/72225472","id":72225472,"diff_hunk":"@@ -104,18 +99,27 @@ public void editPerson_dataValidation() {\n \n     @Test\n     public void cancelOperation_usingAccelerator() {\n+        Person newAlice = new PersonBuilder(td.alice.copy()).withFirstName(\"Alicia\").withLastName(\"Brownstone\")\n+                .withStreet(\"Updated street\").withCity(\"Singapore\").withPostalCode(\"123123\")\n+                .withBirthday(\"01.01.1979\").withGithubUsername(\"alicebrown123\").withTags(td.colleagues, td.friends).build();","path":"src/test/java/guitests/PersonEditGuiTest.java","position":99,"original_position":99,"commit_id":"acae02a26b472c71d43b9ff87939bb6365f725ee","original_commit_id":"acae02a26b472c71d43b9ff87939bb6365f725ee","user":{"login":"damithc","id":1673303,"avatar_url":"https://avatars.githubusercontent.com/u/1673303?v=3","gravatar_id":"","url":"https://api.github.com/users/damithc","html_url":"https://github.com/damithc","followers_url":"https://api.github.com/users/damithc/followers","following_url":"https://api.github.com/users/damithc/following{/other_user}","gists_url":"https://api.github.com/users/damithc/gists{/gist_id}","starred_url":"https://api.github.com/users/damithc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/damithc/subscriptions","organizations_url":"https://api.github.com/users/damithc/orgs","repos_url":"https://api.github.com/users/damithc/repos","events_url":"https://api.github.com/users/damithc/events{/privacy}","received_events_url":"https://api.github.com/users/damithc/received_events","type":"User","site_admin":false},"body":"I know you are following my code, but on retrospect, this declaration should be done just before the newAlice is used. In general, declare variables close to where they are used first.","created_at":"2016-07-26T10:39:26Z","updated_at":"2016-07-26T10:39:26Z","html_url":"https://github.com/HubTurbo/addressbook/pull/557#discussion_r72225472","pull_request_url":"https://api.github.com/repos/HubTurbo/addressbook/pulls/557","_links":{"self":{"href":"https://api.github.com/repos/HubTurbo/addressbook/pulls/comments/72225472"},"html":{"href":"https://github.com/HubTurbo/addressbook/pull/557#discussion_r72225472"},"pull_request":{"href":"https://api.github.com/repos/HubTurbo/addressbook/pulls/557"}}},"pull_request":{"url":"https://api.github.com/repos/HubTurbo/addressbook/pulls/557","id":78643525,"html_url":"https://github.com/HubTurbo/addressbook/pull/557","diff_url":"https://github.com/HubTurbo/addressbook/pull/557.diff","patch_url":"https://github.com/HubTurbo/addressbook/pull/557.patch","issue_url":"https://api.github.com/repos/HubTurbo/addressbook/issues/557","number":557,"state":"open","locked":false,"title":"Improvement and more gui test cases.","user":{"login":"yl-coder","id":9070608,"avatar_url":"https://avatars.githubusercontent.com/u/9070608?v=3","gravatar_id":"","url":"https://api.github.com/users/yl-coder","html_url":"https://github.com/yl-coder","followers_url":"https://api.github.com/users/yl-coder/followers","following_url":"https://api.github.com/users/yl-coder/following{/other_user}","gists_url":"https://api.github.com/users/yl-coder/gists{/gist_id}","starred_url":"https://api.github.com/users/yl-coder/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yl-coder/subscriptions","organizations_url":"https://api.github.com/users/yl-coder/orgs","repos_url":"https://api.github.com/users/yl-coder/repos","events_url":"https://api.github.com/users/yl-coder/events{/privacy}","received_events_url":"https://api.github.com/users/yl-coder/received_events","type":"User","site_admin":false},"body":"As per title.\r\nFixes #565\r\nFixes #562 ","created_at":"2016-07-25T08:30:40Z","updated_at":"2016-07-26T10:39:26Z","closed_at":null,"merged_at":null,"merge_commit_sha":"1cad24a5b31362d33160aaefddad71026bbc5825","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/HubTurbo/addressbook/pulls/557/commits","review_comments_url":"https://api.github.com/repos/HubTurbo/addressbook/pulls/557/comments","review_comment_url":"https://api.github.com/repos/HubTurbo/addressbook/pulls/comments{/number}","comments_url":"https://api.github.com/repos/HubTurbo/addressbook/issues/557/comments","statuses_url":"https://api.github.com/repos/HubTurbo/addressbook/statuses/acae02a26b472c71d43b9ff87939bb6365f725ee","head":{"label":"HubTurbo:improvement-gui-testing","ref":"improvement-gui-testing","sha":"acae02a26b472c71d43b9ff87939bb6365f725ee","user":{"login":"HubTurbo","id":7955871,"avatar_url":"https://avatars.githubusercontent.com/u/7955871?v=3","gravatar_id":"","url":"https://api.github.com/users/HubTurbo","html_url":"https://github.com/HubTurbo","followers_url":"https://api.github.com/users/HubTurbo/followers","following_url":"https://api.github.com/users/HubTurbo/following{/other_user}","gists_url":"https://api.github.com/users/HubTurbo/gists{/gist_id}","starred_url":"https://api.github.com/users/HubTurbo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/HubTurbo/subscriptions","organizations_url":"https://api.github.com/users/HubTurbo/orgs","repos_url":"https://api.github.com/users/HubTurbo/repos","events_url":"https://api.github.com/users/HubTurbo/events{/privacy}","received_events_url":"https://api.github.com/users/HubTurbo/received_events","type":"Organization","site_admin":false},"repo":{"id":58317461,"name":"addressbook","full_name":"HubTurbo/addressbook","owner":{"login":"HubTurbo","id":7955871,"avatar_url":"https://avatars.githubusercontent.com/u/7955871?v=3","gravatar_id":"","url":"https://api.github.com/users/HubTurbo","html_url":"https://github.com/HubTurbo","followers_url":"https://api.github.com/users/HubTurbo/followers","following_url":"https://api.github.com/users/HubTurbo/following{/other_user}","gists_url":"https://api.github.com/users/HubTurbo/gists{/gist_id}","starred_url":"https://api.github.com/users/HubTurbo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/HubTurbo/subscriptions","organizations_url":"https://api.github.com/users/HubTurbo/orgs","repos_url":"https://api.github.com/users/HubTurbo/repos","events_url":"https://api.github.com/users/HubTurbo/events{/privacy}","received_events_url":"https://api.github.com/users/HubTurbo/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/HubTurbo/addressbook","description":"A small app for training and experimentation","fork":false,"url":"https://api.github.com/repos/HubTurbo/addressbook","forks_url":"https://api.github.com/repos/HubTurbo/addressbook/forks","keys_url":"https://api.github.com/repos/HubTurbo/addressbook/keys{/key_id}","collaborators_url":"https://api.github.com/repos/HubTurbo/addressbook/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/HubTurbo/addressbook/teams","hooks_url":"https://api.github.com/repos/HubTurbo/addressbook/hooks","issue_events_url":"https://api.github.com/repos/HubTurbo/addressbook/issues/events{/number}","events_url":"https://api.github.com/repos/HubTurbo/addressbook/events","assignees_url":"https://api.github.com/repos/HubTurbo/addressbook/assignees{/user}","branches_url":"https://api.github.com/repos/HubTurbo/addressbook/branches{/branch}","tags_url":"https://api.github.com/repos/HubTurbo/addressbook/tags","blobs_url":"https://api.github.com/repos/HubTurbo/addressbook/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/HubTurbo/addressbook/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/HubTurbo/addressbook/git/refs{/sha}","trees_url":"https://api.github.com/repos/HubTurbo/addressbook/git/trees{/sha}","statuses_url":"https://api.github.com/repos/HubTurbo/addressbook/statuses/{sha}","languages_url":"https://api.github.com/repos/HubTurbo/addressbook/languages","stargazers_url":"https://api.github.com/repos/HubTurbo/addressbook/stargazers","contributors_url":"https://api.github.com/repos/HubTurbo/addressbook/contributors","subscribers_url":"https://api.github.com/repos/HubTurbo/addressbook/subscribers","subscription_url":"https://api.github.com/repos/HubTurbo/addressbook/subscription","commits_url":"https://api.github.com/repos/HubTurbo/addressbook/commits{/sha}","git_commits_url":"https://api.github.com/repos/HubTurbo/addressbook/git/commits{/sha}","comments_url":"https://api.github.com/repos/HubTurbo/addressbook/comments{/number}","issue_comment_url":"https://api.github.com/repos/HubTurbo/addressbook/issues/comments{/number}","contents_url":"https://api.github.com/repos/HubTurbo/addressbook/contents/{+path}","compare_url":"https://api.github.com/repos/HubTurbo/addressbook/compare/{base}...{head}","merges_url":"https://api.github.com/repos/HubTurbo/addressbook/merges","archive_url":"https://api.github.com/repos/HubTurbo/addressbook/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/HubTurbo/addressbook/downloads","issues_url":"https://api.github.com/repos/HubTurbo/addressbook/issues{/number}","pulls_url":"https://api.github.com/repos/HubTurbo/addressbook/pulls{/number}","milestones_url":"https://api.github.com/repos/HubTurbo/addressbook/milestones{/number}","notifications_url":"https://api.github.com/repos/HubTurbo/addressbook/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/HubTurbo/addressbook/labels{/name}","releases_url":"https://api.github.com/repos/HubTurbo/addressbook/releases{/id}","deployments_url":"https://api.github.com/repos/HubTurbo/addressbook/deployments","created_at":"2016-05-08T15:05:27Z","updated_at":"2016-05-08T23:35:38Z","pushed_at":"2016-07-26T10:10:37Z","git_url":"git://github.com/HubTurbo/addressbook.git","ssh_url":"git@github.com:HubTurbo/addressbook.git","clone_url":"https://github.com/HubTurbo/addressbook.git","svn_url":"https://github.com/HubTurbo/addressbook","homepage":null,"size":4947,"stargazers_count":1,"watchers_count":1,"language":"Java","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"open_issues_count":62,"forks":3,"open_issues":62,"watchers":1,"default_branch":"master"}},"base":{"label":"HubTurbo:master","ref":"master","sha":"5ee70a3ffe7c13392b7d754d5c258c36d8a5b5d1","user":{"login":"HubTurbo","id":7955871,"avatar_url":"https://avatars.githubusercontent.com/u/7955871?v=3","gravatar_id":"","url":"https://api.github.com/users/HubTurbo","html_url":"https://github.com/HubTurbo","followers_url":"https://api.github.com/users/HubTurbo/followers","following_url":"https://api.github.com/users/HubTurbo/following{/other_user}","gists_url":"https://api.github.com/users/HubTurbo/gists{/gist_id}","starred_url":"https://api.github.com/users/HubTurbo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/HubTurbo/subscriptions","organizations_url":"https://api.github.com/users/HubTurbo/orgs","repos_url":"https://api.github.com/users/HubTurbo/repos","events_url":"https://api.github.com/users/HubTurbo/events{/privacy}","received_events_url":"https://api.github.com/users/HubTurbo/received_events","type":"Organization","site_admin":false},"repo":{"id":58317461,"name":"addressbook","full_name":"HubTurbo/addressbook","owner":{"login":"HubTurbo","id":7955871,"avatar_url":"https://avatars.githubusercontent.com/u/7955871?v=3","gravatar_id":"","url":"https://api.github.com/users/HubTurbo","html_url":"https://github.com/HubTurbo","followers_url":"https://api.github.com/users/HubTurbo/followers","following_url":"https://api.github.com/users/HubTurbo/following{/other_user}","gists_url":"https://api.github.com/users/HubTurbo/gists{/gist_id}","starred_url":"https://api.github.com/users/HubTurbo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/HubTurbo/subscriptions","organizations_url":"https://api.github.com/users/HubTurbo/orgs","repos_url":"https://api.github.com/users/HubTurbo/repos","events_url":"https://api.github.com/users/HubTurbo/events{/privacy}","received_events_url":"https://api.github.com/users/HubTurbo/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/HubTurbo/addressbook","description":"A small app for training and experimentation","fork":false,"url":"https://api.github.com/repos/HubTurbo/addressbook","forks_url":"https://api.github.com/repos/HubTurbo/addressbook/forks","keys_url":"https://api.github.com/repos/HubTurbo/addressbook/keys{/key_id}","collaborators_url":"https://api.github.com/repos/HubTurbo/addressbook/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/HubTurbo/addressbook/teams","hooks_url":"https://api.github.com/repos/HubTurbo/addressbook/hooks","issue_events_url":"https://api.github.com/repos/HubTurbo/addressbook/issues/events{/number}","events_url":"https://api.github.com/repos/HubTurbo/addressbook/events","assignees_url":"https://api.github.com/repos/HubTurbo/addressbook/assignees{/user}","branches_url":"https://api.github.com/repos/HubTurbo/addressbook/branches{/branch}","tags_url":"https://api.github.com/repos/HubTurbo/addressbook/tags","blobs_url":"https://api.github.com/repos/HubTurbo/addressbook/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/HubTurbo/addressbook/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/HubTurbo/addressbook/git/refs{/sha}","trees_url":"https://api.github.com/repos/HubTurbo/addressbook/git/trees{/sha}","statuses_url":"https://api.github.com/repos/HubTurbo/addressbook/statuses/{sha}","languages_url":"https://api.github.com/repos/HubTurbo/addressbook/languages","stargazers_url":"https://api.github.com/repos/HubTurbo/addressbook/stargazers","contributors_url":"https://api.github.com/repos/HubTurbo/addressbook/contributors","subscribers_url":"https://api.github.com/repos/HubTurbo/addressbook/subscribers","subscription_url":"https://api.github.com/repos/HubTurbo/addressbook/subscription","commits_url":"https://api.github.com/repos/HubTurbo/addressbook/commits{/sha}","git_commits_url":"https://api.github.com/repos/HubTurbo/addressbook/git/commits{/sha}","comments_url":"https://api.github.com/repos/HubTurbo/addressbook/comments{/number}","issue_comment_url":"https://api.github.com/repos/HubTurbo/addressbook/issues/comments{/number}","contents_url":"https://api.github.com/repos/HubTurbo/addressbook/contents/{+path}","compare_url":"https://api.github.com/repos/HubTurbo/addressbook/compare/{base}...{head}","merges_url":"https://api.github.com/repos/HubTurbo/addressbook/merges","archive_url":"https://api.github.com/repos/HubTurbo/addressbook/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/HubTurbo/addressbook/downloads","issues_url":"https://api.github.com/repos/HubTurbo/addressbook/issues{/number}","pulls_url":"https://api.github.com/repos/HubTurbo/addressbook/pulls{/number}","milestones_url":"https://api.github.com/repos/HubTurbo/addressbook/milestones{/number}","notifications_url":"https://api.github.com/repos/HubTurbo/addressbook/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/HubTurbo/addressbook/labels{/name}","releases_url":"https://api.github.com/repos/HubTurbo/addressbook/releases{/id}","deployments_url":"https://api.github.com/repos/HubTurbo/addressbook/deployments","created_at":"2016-05-08T15:05:27Z","updated_at":"2016-05-08T23:35:38Z","pushed_at":"2016-07-26T10:10:37Z","git_url":"git://github.com/HubTurbo/addressbook.git","ssh_url":"git@github.com:HubTurbo/addressbook.git","clone_url":"https://github.com/HubTurbo/addressbook.git","svn_url":"https://github.com/HubTurbo/addressbook","homepage":null,"size":4947,"stargazers_count":1,"watchers_count":1,"language":"Java","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"open_issues_count":62,"forks":3,"open_issues":62,"watchers":1,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/HubTurbo/addressbook/pulls/557"},"html":{"href":"https://github.com/HubTurbo/addressbook/pull/557"},"issue":{"href":"https://api.github.com/repos/HubTurbo/addressbook/issues/557"},"comments":{"href":"https://api.github.com/repos/HubTurbo/addressbook/issues/557/comments"},"review_comments":{"href":"https://api.github.com/repos/HubTurbo/addressbook/pulls/557/comments"},"review_comment":{"href":"https://api.github.com/repos/HubTurbo/addressbook/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/HubTurbo/addressbook/pulls/557/commits"},"statuses":{"href":"https://api.github.com/repos/HubTurbo/addressbook/statuses/acae02a26b472c71d43b9ff87939bb6365f725ee"}}}}
{"actor":{"display_login":"damithc"}}