{"action":"created","issue":{"url":"https://api.github.com/repos/droolsjbpm/jbpm-console-ng/issues/167","labels_url":"https://api.github.com/repos/droolsjbpm/jbpm-console-ng/issues/167/labels{/name}","comments_url":"https://api.github.com/repos/droolsjbpm/jbpm-console-ng/issues/167/comments","events_url":"https://api.github.com/repos/droolsjbpm/jbpm-console-ng/issues/167/events","html_url":"https://github.com/droolsjbpm/jbpm-console-ng/pull/167","id":93440192,"number":167,"title":"Refactor TaskCommentsView/Presenter for unit testability & add unit tests","user":{"login":"jhrcek","id":2716069,"avatar_url":"https://avatars.githubusercontent.com/u/2716069?v=3","gravatar_id":"","url":"https://api.github.com/users/jhrcek","html_url":"https://github.com/jhrcek","followers_url":"https://api.github.com/users/jhrcek/followers","following_url":"https://api.github.com/users/jhrcek/following{/other_user}","gists_url":"https://api.github.com/users/jhrcek/gists{/gist_id}","starred_url":"https://api.github.com/users/jhrcek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jhrcek/subscriptions","organizations_url":"https://api.github.com/users/jhrcek/orgs","repos_url":"https://api.github.com/users/jhrcek/repos","events_url":"https://api.github.com/users/jhrcek/events{/privacy}","received_events_url":"https://api.github.com/users/jhrcek/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/droolsjbpm/jbpm-console-ng/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":{"login":"Salaboy","id":271966,"avatar_url":"https://avatars.githubusercontent.com/u/271966?v=3","gravatar_id":"","url":"https://api.github.com/users/Salaboy","html_url":"https://github.com/Salaboy","followers_url":"https://api.github.com/users/Salaboy/followers","following_url":"https://api.github.com/users/Salaboy/following{/other_user}","gists_url":"https://api.github.com/users/Salaboy/gists{/gist_id}","starred_url":"https://api.github.com/users/Salaboy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Salaboy/subscriptions","organizations_url":"https://api.github.com/users/Salaboy/orgs","repos_url":"https://api.github.com/users/Salaboy/repos","events_url":"https://api.github.com/users/Salaboy/events{/privacy}","received_events_url":"https://api.github.com/users/Salaboy/received_events","type":"User","site_admin":false},"milestone":null,"comments":4,"created_at":"2015-07-07T05:13:46Z","updated_at":"2015-07-08T13:00:56Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/droolsjbpm/jbpm-console-ng/pulls/167","html_url":"https://github.com/droolsjbpm/jbpm-console-ng/pull/167","diff_url":"https://github.com/droolsjbpm/jbpm-console-ng/pull/167.diff","patch_url":"https://github.com/droolsjbpm/jbpm-console-ng/pull/167.patch"},"body":"Hello. This is my personal proof of concept in greater mission of migrating some QE's slow and hard-to-maintain selenium tests to faster and possibly more reliable Mockito based unit tests in community, performed at the level of UI controller. I also did some refactorings to remove some duplication and make the code a bit more readable (see comments inline). I tested the changes thoroughly. Please check it and provide some feedback.\r\n\r\nPS.: Sorry for changing the formatting, I'll try to avoid that in future commits."},"comment":{"url":"https://api.github.com/repos/droolsjbpm/jbpm-console-ng/issues/comments/119570379","html_url":"https://github.com/droolsjbpm/jbpm-console-ng/pull/167#issuecomment-119570379","issue_url":"https://api.github.com/repos/droolsjbpm/jbpm-console-ng/issues/167","id":119570379,"user":{"login":"jhrcek","id":2716069,"avatar_url":"https://avatars.githubusercontent.com/u/2716069?v=3","gravatar_id":"","url":"https://api.github.com/users/jhrcek","html_url":"https://github.com/jhrcek","followers_url":"https://api.github.com/users/jhrcek/followers","following_url":"https://api.github.com/users/jhrcek/following{/other_user}","gists_url":"https://api.github.com/users/jhrcek/gists{/gist_id}","starred_url":"https://api.github.com/users/jhrcek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jhrcek/subscriptions","organizations_url":"https://api.github.com/users/jhrcek/orgs","repos_url":"https://api.github.com/users/jhrcek/repos","events_url":"https://api.github.com/users/jhrcek/events{/privacy}","received_events_url":"https://api.github.com/users/jhrcek/received_events","type":"User","site_admin":false},"created_at":"2015-07-08T13:00:56Z","updated_at":"2015-07-08T13:00:56Z","body":"@yurloc Don't ask me, I just extracted existing implementation into static inner class :) What do you think @Salaboy ?"}}