{"action":"created","issue":{"url":"https://api.github.com/repos/yegor256/takes/issues/200","labels_url":"https://api.github.com/repos/yegor256/takes/issues/200/labels{/name}","comments_url":"https://api.github.com/repos/yegor256/takes/issues/200/comments","events_url":"https://api.github.com/repos/yegor256/takes/issues/200/events","html_url":"https://github.com/yegor256/takes/pull/200","id":69957699,"number":200,"title":"PsLinkedinTest.java:42-48 - Added PsLinkedinTest and refactor PsLinkedin...","user":{"login":"ikhvostenkov","id":861305,"avatar_url":"https://avatars.githubusercontent.com/u/861305?v=3","gravatar_id":"","url":"https://api.github.com/users/ikhvostenkov","html_url":"https://github.com/ikhvostenkov","followers_url":"https://api.github.com/users/ikhvostenkov/followers","following_url":"https://api.github.com/users/ikhvostenkov/following{/other_user}","gists_url":"https://api.github.com/users/ikhvostenkov/gists{/gist_id}","starred_url":"https://api.github.com/users/ikhvostenkov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ikhvostenkov/subscriptions","organizations_url":"https://api.github.com/users/ikhvostenkov/orgs","repos_url":"https://api.github.com/users/ikhvostenkov/repos","events_url":"https://api.github.com/users/ikhvostenkov/events{/privacy}","received_events_url":"https://api.github.com/users/ikhvostenkov/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/yegor256/takes/labels/15%20mins","name":"15 mins","color":"eb6421"},{"url":"https://api.github.com/repos/yegor256/takes/labels/@krzyk","name":"@krzyk","color":"009801"},{"url":"https://api.github.com/repos/yegor256/takes/labels/CR","name":"CR","color":"bfd4f3"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":14,"created_at":"2015-04-21T21:32:09Z","updated_at":"2015-04-26T05:32:03Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/yegor256/takes/pulls/200","html_url":"https://github.com/yegor256/takes/pull/200","diff_url":"https://github.com/yegor256/takes/pull/200.diff","patch_url":"https://github.com/yegor256/takes/pull/200.patch"},"body":"This is fix for https://github.com/yegor256/takes/issues/158.\r\nPsLinkedinTest.java:42-48 - Added PsLinkedinTest and refactor PsLinkedin to be able to mock calls to LinkedIn API. \r\n@yegor256 Could you please review my suggestion for mocking social network calls. I've created ```MemberProfileJson``` interface to avoid ```DesignForExtensionCheck``` rule. But by implementing this interface we can mock all classes with API calls to social networks. Initial idea was to combine ```PsLinkedin``` and ```PsGithub``` calls in one class, it still can be done with some modifications of ```PsGithub``` class and moving all social network calls functionality in one class hiding this under protected method to avoid ```DesignForExtensionCheck```violations."},"comment":{"url":"https://api.github.com/repos/yegor256/takes/issues/comments/96325274","html_url":"https://github.com/yegor256/takes/pull/200#issuecomment-96325274","issue_url":"https://api.github.com/repos/yegor256/takes/issues/200","id":96325274,"user":{"login":"krzyk","id":105730,"avatar_url":"https://avatars.githubusercontent.com/u/105730?v=3","gravatar_id":"","url":"https://api.github.com/users/krzyk","html_url":"https://github.com/krzyk","followers_url":"https://api.github.com/users/krzyk/followers","following_url":"https://api.github.com/users/krzyk/following{/other_user}","gists_url":"https://api.github.com/users/krzyk/gists{/gist_id}","starred_url":"https://api.github.com/users/krzyk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/krzyk/subscriptions","organizations_url":"https://api.github.com/users/krzyk/orgs","repos_url":"https://api.github.com/users/krzyk/repos","events_url":"https://api.github.com/users/krzyk/events{/privacy}","received_events_url":"https://api.github.com/users/krzyk/received_events","type":"User","site_admin":false},"created_at":"2015-04-26T05:32:03Z","updated_at":"2015-04-26T05:32:03Z","body":"@rultor merge please"}}