{"action":"created","issue":{"url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2156","labels_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2156/labels{/name}","comments_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2156/comments","events_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2156/events","html_url":"https://github.com/checkstyle/checkstyle/pull/2156","id":105924565,"number":2156,"title":"Issue #1292: AbstractHeaderCheck coverage has been increased to 100%","user":{"login":"sabaka","id":1004134,"avatar_url":"https://avatars.githubusercontent.com/u/1004134?v=3","gravatar_id":"","url":"https://api.github.com/users/sabaka","html_url":"https://github.com/sabaka","followers_url":"https://api.github.com/users/sabaka/followers","following_url":"https://api.github.com/users/sabaka/following{/other_user}","gists_url":"https://api.github.com/users/sabaka/gists{/gist_id}","starred_url":"https://api.github.com/users/sabaka/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sabaka/subscriptions","organizations_url":"https://api.github.com/users/sabaka/orgs","repos_url":"https://api.github.com/users/sabaka/repos","events_url":"https://api.github.com/users/sabaka/events{/privacy}","received_events_url":"https://api.github.com/users/sabaka/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":11,"created_at":"2015-09-11T00:38:45Z","updated_at":"2015-09-14T20:25:00Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/checkstyle/checkstyle/pulls/2156","html_url":"https://github.com/checkstyle/checkstyle/pull/2156","diff_url":"https://github.com/checkstyle/checkstyle/pull/2156.diff","patch_url":"https://github.com/checkstyle/checkstyle/pull/2156.patch"},"body":"Without changes in check's code"},"comment":{"url":"https://api.github.com/repos/checkstyle/checkstyle/issues/comments/140195985","html_url":"https://github.com/checkstyle/checkstyle/pull/2156#issuecomment-140195985","issue_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2156","id":140195985,"user":{"login":"mkordas","id":5467276,"avatar_url":"https://avatars.githubusercontent.com/u/5467276?v=3","gravatar_id":"","url":"https://api.github.com/users/mkordas","html_url":"https://github.com/mkordas","followers_url":"https://api.github.com/users/mkordas/followers","following_url":"https://api.github.com/users/mkordas/following{/other_user}","gists_url":"https://api.github.com/users/mkordas/gists{/gist_id}","starred_url":"https://api.github.com/users/mkordas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mkordas/subscriptions","organizations_url":"https://api.github.com/users/mkordas/orgs","repos_url":"https://api.github.com/users/mkordas/repos","events_url":"https://api.github.com/users/mkordas/events{/privacy}","received_events_url":"https://api.github.com/users/mkordas/received_events","type":"User","site_admin":false},"created_at":"2015-09-14T20:25:00Z","updated_at":"2015-09-14T20:25:00Z","body":"@romani, @sabaka, there are two issues in this commit:\r\n* Statically imported element 'fail' is unnecessarily qualified with 'Assert' at line 142 \r\n* 'new IOException()' without arguments at line 153\r\n\r\nAdditional comments:\r\n1) `public void setHeaderTwiceTest()` - we have convention to begin test names with `test` I believe\r\n2) `testIoExceptionWhenLoadHeader` -> `testIoExceptionWhenLoadingHeader`\r\n3) testIoExceptionWhenLoadHeader checks two things I believe - please split it into two tests"}}