{"action":"created","issue":{"url":"https://api.github.com/repos/openlayers/ol3/issues/3518","labels_url":"https://api.github.com/repos/openlayers/ol3/issues/3518/labels{/name}","comments_url":"https://api.github.com/repos/openlayers/ol3/issues/3518/comments","events_url":"https://api.github.com/repos/openlayers/ol3/issues/3518/events","html_url":"https://github.com/openlayers/ol3/pull/3518","id":67068690,"number":3518,"title":"Include all source files in coverage","user":{"login":"tsauerwein","id":266474,"avatar_url":"https://avatars.githubusercontent.com/u/266474?v=3","gravatar_id":"","url":"https://api.github.com/users/tsauerwein","html_url":"https://github.com/tsauerwein","followers_url":"https://api.github.com/users/tsauerwein/followers","following_url":"https://api.github.com/users/tsauerwein/following{/other_user}","gists_url":"https://api.github.com/users/tsauerwein/gists{/gist_id}","starred_url":"https://api.github.com/users/tsauerwein/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tsauerwein/subscriptions","organizations_url":"https://api.github.com/users/tsauerwein/orgs","repos_url":"https://api.github.com/users/tsauerwein/repos","events_url":"https://api.github.com/users/tsauerwein/events{/privacy}","received_events_url":"https://api.github.com/users/tsauerwein/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-04-08T07:32:37Z","updated_at":"2015-04-08T10:17:03Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openlayers/ol3/pulls/3518","html_url":"https://github.com/openlayers/ol3/pull/3518","diff_url":"https://github.com/openlayers/ol3/pull/3518.diff","patch_url":"https://github.com/openlayers/ol3/pull/3518.patch"},"body":"After merging #3470 our coverage increased by 4%. This wasn't *caused* by the new rendering tests (which are not included in the coverage), but due to the fact that the file loader used in the tests now only included sources that were required by the tests. Because istanbul only considers files that are loaded, all untested files were not included in the coverage.\r\n\r\nThis PR aims to fix the issue by loading all source files that have a `goog.provide`.\r\n\r\nSee also #3517 and #3514"},"comment":{"url":"https://api.github.com/repos/openlayers/ol3/issues/comments/90871425","html_url":"https://github.com/openlayers/ol3/pull/3518#issuecomment-90871425","issue_url":"https://api.github.com/repos/openlayers/ol3/issues/3518","id":90871425,"user":{"login":"tschaub","id":41094,"avatar_url":"https://avatars.githubusercontent.com/u/41094?v=3","gravatar_id":"","url":"https://api.github.com/users/tschaub","html_url":"https://github.com/tschaub","followers_url":"https://api.github.com/users/tschaub/followers","following_url":"https://api.github.com/users/tschaub/following{/other_user}","gists_url":"https://api.github.com/users/tschaub/gists{/gist_id}","starred_url":"https://api.github.com/users/tschaub/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tschaub/subscriptions","organizations_url":"https://api.github.com/users/tschaub/orgs","repos_url":"https://api.github.com/users/tschaub/repos","events_url":"https://api.github.com/users/tschaub/events{/privacy}","received_events_url":"https://api.github.com/users/tschaub/received_events","type":"User","site_admin":false},"created_at":"2015-04-08T10:17:03Z","updated_at":"2015-04-08T10:17:03Z","body":"> What about removing the npm tasks test and test-coverage and requiring that the tests are run with ./build.py test?\r\n\r\n-1"}}