{"action":"created","issue":{"url":"https://api.github.com/repos/kenichi/angelo/issues/28","labels_url":"https://api.github.com/repos/kenichi/angelo/issues/28/labels{/name}","comments_url":"https://api.github.com/repos/kenichi/angelo/issues/28/comments","events_url":"https://api.github.com/repos/kenichi/angelo/issues/28/events","html_url":"https://github.com/kenichi/angelo/pull/28","id":54644052,"number":28,"title":"Add test coverage with SimpleCov on MRI","user":{"login":"kyledrake","id":28153,"avatar_url":"https://avatars.githubusercontent.com/u/28153?v=3","gravatar_id":"","url":"https://api.github.com/users/kyledrake","html_url":"https://github.com/kyledrake","followers_url":"https://api.github.com/users/kyledrake/followers","following_url":"https://api.github.com/users/kyledrake/following{/other_user}","gists_url":"https://api.github.com/users/kyledrake/gists{/gist_id}","starred_url":"https://api.github.com/users/kyledrake/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kyledrake/subscriptions","organizations_url":"https://api.github.com/users/kyledrake/orgs","repos_url":"https://api.github.com/users/kyledrake/repos","events_url":"https://api.github.com/users/kyledrake/events{/privacy}","received_events_url":"https://api.github.com/users/kyledrake/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-01-17T00:58:20Z","updated_at":"2015-01-17T01:58:24Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/kenichi/angelo/pulls/28","html_url":"https://github.com/kenichi/angelo/pull/28","diff_url":"https://github.com/kenichi/angelo/pull/28.diff","patch_url":"https://github.com/kenichi/angelo/pull/28.patch"},"body":"This adds SimpleCov testing. When the tests are run on MRI (does not run on RBX or JRuby), it will generate a `test/coverage` directory which is ignored in `.gitignore`, and gives a good way to check for missing tests and missed conditionals and stuff like that. Just point your browser to `angelo/test/coverage/index.html` to see locally. I have uploaded the tests to my Neocities account so you can inspect the current output: http://kyledrake.neocities.org/angelo-coverage/index.html\r\n\r\nIt would not be much more work to integrate [Coveralls](https://coveralls.io), giving you a nice button to put on the README, just like we did with [BitcoinJS](https://github.com/bitcoinjs/bitcoinjs-lib/blob/master/README.md).\r\n\r\nSuper useful for improving testing, never hurts to have another vector there."},"comment":{"url":"https://api.github.com/repos/kenichi/angelo/issues/comments/70349701","html_url":"https://github.com/kenichi/angelo/pull/28#issuecomment-70349701","issue_url":"https://api.github.com/repos/kenichi/angelo/issues/28","id":70349701,"user":{"login":"kenichi","id":933055,"avatar_url":"https://avatars.githubusercontent.com/u/933055?v=3","gravatar_id":"","url":"https://api.github.com/users/kenichi","html_url":"https://github.com/kenichi","followers_url":"https://api.github.com/users/kenichi/followers","following_url":"https://api.github.com/users/kenichi/following{/other_user}","gists_url":"https://api.github.com/users/kenichi/gists{/gist_id}","starred_url":"https://api.github.com/users/kenichi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kenichi/subscriptions","organizations_url":"https://api.github.com/users/kenichi/orgs","repos_url":"https://api.github.com/users/kenichi/repos","events_url":"https://api.github.com/users/kenichi/events{/privacy}","received_events_url":"https://api.github.com/users/kenichi/received_events","type":"User","site_admin":false},"created_at":"2015-01-17T01:58:24Z","updated_at":"2015-01-17T01:58:24Z","body":"Wow actually pretty good coverage already eh?"}}