{"action":"created","issue":{"url":"https://api.github.com/repos/mozilla/pontoon/issues/257","labels_url":"https://api.github.com/repos/mozilla/pontoon/issues/257/labels{/name}","comments_url":"https://api.github.com/repos/mozilla/pontoon/issues/257/comments","events_url":"https://api.github.com/repos/mozilla/pontoon/issues/257/events","html_url":"https://github.com/mozilla/pontoon/pull/257","id":116225044,"number":257,"title":"Support for coveralls.io","user":{"login":"jotes","id":8784972,"avatar_url":"https://avatars.githubusercontent.com/u/8784972?v=3","gravatar_id":"","url":"https://api.github.com/users/jotes","html_url":"https://github.com/jotes","followers_url":"https://api.github.com/users/jotes/followers","following_url":"https://api.github.com/users/jotes/following{/other_user}","gists_url":"https://api.github.com/users/jotes/gists{/gist_id}","starred_url":"https://api.github.com/users/jotes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jotes/subscriptions","organizations_url":"https://api.github.com/users/jotes/orgs","repos_url":"https://api.github.com/users/jotes/repos","events_url":"https://api.github.com/users/jotes/events{/privacy}","received_events_url":"https://api.github.com/users/jotes/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-11-10T23:18:57Z","updated_at":"2015-11-10T23:47:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mozilla/pontoon/pulls/257","html_url":"https://github.com/mozilla/pontoon/pull/257","diff_url":"https://github.com/mozilla/pontoon/pull/257.diff","patch_url":"https://github.com/mozilla/pontoon/pull/257.patch"},"body":"This service monitors tests coverage in project. Example report is here: https://coveralls.io/builds/4111747\r\nI think it would be nice addition to pontoon. The setup is very easy thing - all is documented @ coveralls.io .\r\n@Osmose @mathjazz could you review and share your thoughts?"},"comment":{"url":"https://api.github.com/repos/mozilla/pontoon/issues/comments/155605139","html_url":"https://github.com/mozilla/pontoon/pull/257#issuecomment-155605139","issue_url":"https://api.github.com/repos/mozilla/pontoon/issues/257","id":155605139,"user":{"login":"Osmose","id":193106,"avatar_url":"https://avatars.githubusercontent.com/u/193106?v=3","gravatar_id":"","url":"https://api.github.com/users/Osmose","html_url":"https://github.com/Osmose","followers_url":"https://api.github.com/users/Osmose/followers","following_url":"https://api.github.com/users/Osmose/following{/other_user}","gists_url":"https://api.github.com/users/Osmose/gists{/gist_id}","starred_url":"https://api.github.com/users/Osmose/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Osmose/subscriptions","organizations_url":"https://api.github.com/users/Osmose/orgs","repos_url":"https://api.github.com/users/Osmose/repos","events_url":"https://api.github.com/users/Osmose/events{/privacy}","received_events_url":"https://api.github.com/users/Osmose/received_events","type":"User","site_admin":false},"created_at":"2015-11-10T23:47:50Z","updated_at":"2015-11-10T23:47:50Z","body":"I have rather strong negative feelings about code coverage. The idea is that higher coverage indicates that your tests are running more code while they execute and thus can make guarantees about the behavior of a larger set of code, but I don't think that holds. Coverage says nothing about the quality of your tests, nor does it actually say whether the \"covered\" code is actually being tested as opposed to being incidentally executed during a test.\r\n\r\nIn practice, you end up gaming your tests to increase coverage instead of actually trying to make things better. Or, if you don't, you end up with a sub-100% number that drags on your motivation as it never rises. Or, if you hit 100%, you end up adding a barrier to new code that also drags you down.\r\n\r\nOf course a lot of this is based on personal experience, so I'm interested in hearing alternate viewpoints. :D"}}